diff --git a/components/net/http_loader.rs b/components/net/http_loader.rs index c3e6a5dfb3c..9b81de49f8c 100644 --- a/components/net/http_loader.rs +++ b/components/net/http_loader.rs @@ -14,6 +14,7 @@ use hyper::client::Request; use hyper::header::{ContentLength, ContentType, Host, Location}; use hyper::HttpError; use hyper::method::Method; +use hyper::mime::{Mime, TopLevel, SubLevel}; use hyper::net::HttpConnector; use hyper::status::{StatusCode, StatusClass}; use std::error::Error; @@ -59,6 +60,26 @@ fn load(mut load_data: LoadData, start_chan: Sender, cooki eventual_consumer: load_data.consumer }; + // If the URL is a view-source scheme then the scheme data contains the + // real URL that should be used for which the source is to be viewed. + // Change our existing URL to that and keep note that we are viewing + // the source rather than rendering the contents of the URL. + let viewing_source = if url.scheme == "view-source" { + let inner_url = load_data.url.non_relative_scheme_data().unwrap(); + url = Url::parse(inner_url).unwrap(); + match url.scheme.as_slice() { + "http" | "https" => {} + _ => { + let s = format!("The {} scheme with view-source is not supported", url.scheme); + send_error(url, s, senders); + return; + } + }; + true + } else { + false + }; + // Loop to handle redirects. loop { iters = iters + 1; @@ -259,12 +280,16 @@ reason: \"certificate verify failed\" }]"; } } + let mut adjusted_headers = response.headers.clone(); + if viewing_source { + adjusted_headers.set(ContentType(Mime(TopLevel::Text, SubLevel::Plain, vec![]))); + } let mut metadata = Metadata::default(url); - metadata.set_content_type(match response.headers.get() { + metadata.set_content_type(match adjusted_headers.get() { Some(&ContentType(ref mime)) => Some(mime), None => None }); - metadata.headers = Some(response.headers.clone()); + metadata.headers = Some(adjusted_headers); metadata.status = Some(response.status_raw().clone()); let progress_chan = match start_sending_opt(senders, metadata) { diff --git a/components/net/resource_task.rs b/components/net/resource_task.rs index d0a65975d35..8ceb0d82306 100644 --- a/components/net/resource_task.rs +++ b/components/net/resource_task.rs @@ -333,7 +333,7 @@ impl ResourceManager { let loader = match load_data.url.scheme.as_slice() { "file" => from_factory(file_loader::factory), - "http" | "https" => http_loader::factory(self.resource_task.clone()), + "http" | "https" | "view-source" => http_loader::factory(self.resource_task.clone()), "data" => from_factory(data_loader::factory), "about" => from_factory(about_loader::factory), _ => { diff --git a/components/script/parse/html.rs b/components/script/parse/html.rs index 6da3dac1ec8..836eac6dfba 100644 --- a/components/script/parse/html.rs +++ b/components/script/parse/html.rs @@ -180,6 +180,22 @@ pub fn parse_html(document: JSRef, task_state::enter(IN_HTML_PARSER); } + fn parse_progress(parser: &JSRef, url: &Url, load_response: &LoadResponse) { + for msg in load_response.progress_port.iter() { + match msg { + ProgressMsg::Payload(data) => { + // FIXME: use Vec (html5ever #34) + let data = UTF_8.decode(data.as_slice(), DecoderTrap::Replace).unwrap(); + parser.parse_chunk(data); + } + ProgressMsg::Done(Err(err)) => { + panic!("Failed to load page URL {}, error: {}", url.serialize(), err); + } + ProgressMsg::Done(Ok(())) => break, + } + } + }; + match input { HTMLInput::InputString(s) => { parser.parse_chunk(s); @@ -190,20 +206,22 @@ pub fn parse_html(document: JSRef, let page = format!("", url.serialize()); parser.parse_chunk(page); }, + Some((ref t, ref st)) if t.as_slice().eq_ignore_ascii_case("text") && + st.as_slice().eq_ignore_ascii_case("plain") => { + // FIXME: When servo/html5ever#109 is fixed remove usage and + // replace with fix from that issue. + + // text/plain documents require setting the tokenizer into PLAINTEXT mode. + // This is done by using a <plaintext> element as the html5ever tokenizer + // provides no other way to change to that state. + // Spec for text/plain handling is: + // https://html.spec.whatwg.org/multipage/browsers.html#read-text + let page = format!("<pre>\u{000A}<plaintext>"); + parser.parse_chunk(page); + parse_progress(&parser, url, &load_response); + }, _ => { - for msg in load_response.progress_port.iter() { - match msg { - ProgressMsg::Payload(data) => { - // FIXME: use Vec<u8> (html5ever #34) - let data = UTF_8.decode(data.as_slice(), DecoderTrap::Replace).unwrap(); - parser.parse_chunk(data); - } - ProgressMsg::Done(Err(err)) => { - panic!("Failed to load page URL {}, error: {}", url.serialize(), err); - } - ProgressMsg::Done(Ok(())) => break, - } - } + parse_progress(&parser, url, &load_response); } } } diff --git a/components/script/script_task.rs b/components/script/script_task.rs index 02c504804ce..3b17c49e45c 100644 --- a/components/script/script_task.rs +++ b/components/script/script_task.rs @@ -83,6 +83,7 @@ use js; use url::Url; use libc; +use std::ascii::AsciiExt; use std::any::Any; use std::borrow::ToOwned; use std::cell::{Cell, RefCell}; @@ -978,10 +979,18 @@ impl ScriptTask { headers.get().map(|&LastModified(ref tm)| dom_last_modified(tm)) }); + let content_type = match response.metadata.content_type { + Some((ref t, ref st)) if t.as_slice().eq_ignore_ascii_case("text") && + st.as_slice().eq_ignore_ascii_case("plain") => { + Some("text/plain".to_owned()) + } + _ => None + }; + let document = Document::new(window.r(), Some(final_url.clone()), IsHTMLDocument::HTMLDocument, - None, + content_type, last_modified, DocumentSource::FromParser).root(); diff --git a/tests/wpt/metadata/html/browsers/browsing-the-web/read-text/load-text-plain.html.ini b/tests/wpt/metadata/html/browsers/browsing-the-web/read-text/load-text-plain.html.ini index cc9779501b8..ee851470532 100644 --- a/tests/wpt/metadata/html/browsers/browsing-the-web/read-text/load-text-plain.html.ini +++ b/tests/wpt/metadata/html/browsers/browsing-the-web/read-text/load-text-plain.html.ini @@ -1,11 +1,5 @@ [load-text-plain.html] type: testharness - [Checking document metadata for text file] - expected: FAIL - - [Checking DOM for text file] - expected: FAIL - [Checking contents for text file] expected: FAIL