Skip to content

Commit

Permalink
Fixes #5215 Serializing the thread name for the page load task to mak…
Browse files Browse the repository at this point in the history
…e it a simple url
  • Loading branch information
mattnenterprise committed Mar 17, 2015
1 parent b8e87ea commit f083583
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion components/script/script_task.rs
Expand Up @@ -1201,7 +1201,7 @@ impl ScriptTask {
let script_chan = self.chan.clone();
let resource_task = self.resource_task.clone();

spawn_named(format!("fetch for {:?}", load_data.url), move || {
spawn_named(format!("fetch for {:?}", load_data.url.serialize()), move || {
if load_data.url.scheme.as_slice() == "javascript" {
load_data.url = Url::parse("about:blank").unwrap();
}
Expand Down

5 comments on commit f083583

@bors-servo
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

saw approval from jdm
at mattnenterprise@f083583

@bors-servo
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

merging mattnenterprise/servo/page_load_thread_name = f083583 into auto

@bors-servo
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

mattnenterprise/servo/page_load_thread_name = f083583 merged ok, testing candidate = 990a965

@bors-servo
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@bors-servo
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fast-forwarding master to auto = 990a965

Please sign in to comment.