Update test expectations. Mark script loads as complete when the resp…

…onse is complete.

components/script/document_loader.rs

@@ -100,6 +100,7 @@ impl DocumentLoader {
     }
 
     pub fn is_blocked(&self) -> bool {
+        //TODO: Ensure that we report blocked if parsing is still ongoing.
         !self.blocking_loads.is_empty()
     }
 

components/script/dom/htmlscriptelement.rs

@@ -167,6 +167,8 @@ struct ScriptContext {
     metadata: RefCell<Option<Metadata>>,
     /// Whether the owning document's parser should resume once the response completes.
     resume_on_completion: bool,
+    /// The initial URL requested.
+    url: Url,
 }
 
 impl AsyncResponseListener for ScriptContext {
@@ -175,7 +177,8 @@ impl AsyncResponseListener for ScriptContext {
     }
 
     fn data_available(&self, payload: Vec<u8>) {
-        self.data.borrow_mut().extend(payload.into_iter());
+        let mut payload = payload;
+        self.data.borrow_mut().append(&mut payload);
     }
 
     fn response_complete(&self, status: Result<(), String>) {
@@ -188,8 +191,10 @@ impl AsyncResponseListener for ScriptContext {
 
         elem.r().execute(ScriptOrigin::External(load));
 
+        let document = document_from_node(elem.r()).root();
+        document.r().finish_load(LoadType::Script(self.url.clone()));
+
         if self.resume_on_completion {
-            let document = document_from_node(elem.r()).root();
             document.r().get_current_parser().unwrap().root().r().resume();
         }
     }
@@ -316,6 +321,7 @@ impl<'a> HTMLScriptElementHelpers for JSRef<'a, HTMLScriptElement> {
                             data: RefCell::new(vec!()),
                             metadata: RefCell::new(None),
                             resume_on_completion: self.parser_inserted.get(),
+                            url: url.clone(),
                         }));
 
                         let listener = box NetworkListener {

tests/wpt/metadata/old-tests/submission/Opera/script_scheduling/122.html.ini

@@ -1,8 +1,5 @@
 [122.html]
   type: testharness
-  [scheduler: altering the type attribute and adding/removing external script ]
-    expected: FAIL
-
   [Reinserted script async IDL attribute]
     expected: FAIL
 

tests/wpt/metadata/old-tests/submission/Opera/script_scheduling/134.html.ini

@@ -0,0 +1,5 @@
+[134.html]
+  type: testharness
+  [scheduler: external HTML script added by SVG script]
+    expected: FAIL
+