Fix document.write check for activity.

Alan Jeffrey · Alan Jeffrey · commit ca9cee084e5e · 2017-01-27T17:25:40.000-06:00
diff --git a/components/script/dom/document.rs b/components/script/dom/document.rs
@@ -3271,8 +3271,7 @@ impl DocumentMethods for Document {
 
         // Step 2.
         // TODO: handle throw-on-dynamic-markup-insertion counter.
-        // FIXME: this should check for being active rather than fully active
-        if !self.is_fully_active() {
+        if !self.is_active() {
             // Step 3.
             return Ok(());
         }
diff --git a/components/script/script_thread.rs b/components/script/script_thread.rs
@@ -1304,6 +1304,7 @@ impl ScriptThread {
 
     /// Handles activity change message
     fn handle_set_document_activity_msg(&self, id: PipelineId, activity: DocumentActivity) {
+        debug!("Setting activity of {} to be {:?}.", id, activity);
         let document = self.documents.borrow().find_document(id);
         if let Some(document) = document {
             document.set_activity(activity);
diff --git a/tests/wpt/metadata/MANIFEST.json b/tests/wpt/metadata/MANIFEST.json
@@ -45873,6 +45873,12 @@
             "url": "/cssom/stylesheet-same-origin.sub.html"
           }
         ],
+        "html/dom/dynamic-markup-insertion/document-write/write-active-document.html": [
+          {
+            "path": "html/dom/dynamic-markup-insertion/document-write/write-active-document.html",
+            "url": "/html/dom/dynamic-markup-insertion/document-write/write-active-document.html"
+          }
+        ],
         "html/semantics/embedded-content/the-iframe-element/iframe-synchronously-discard.html": [
           {
             "path": "html/semantics/embedded-content/the-iframe-element/iframe-synchronously-discard.html",
diff --git a/tests/wpt/metadata/html/dom/dynamic-markup-insertion/document-write/write-active-document.html.ini b/tests/wpt/metadata/html/dom/dynamic-markup-insertion/document-write/write-active-document.html.ini
@@ -0,0 +1,4 @@
+[write-active-document.html]
+  type: testharness
+  [document.write only writes to active documents]
+    expected: FAIL
diff --git a/tests/wpt/web-platform-tests/html/dom/dynamic-markup-insertion/document-write/empty.html b/tests/wpt/web-platform-tests/html/dom/dynamic-markup-insertion/document-write/empty.html
@@ -0,0 +1 @@
+<html><body></body></html>
diff --git a/tests/wpt/web-platform-tests/html/dom/dynamic-markup-insertion/document-write/write-active-document.html b/tests/wpt/web-platform-tests/html/dom/dynamic-markup-insertion/document-write/write-active-document.html
@@ -0,0 +1,35 @@
+<!doctype html>
+<title>document.write only writes to active documents</title>
+<script src="/resources/testharness.js"></script>
+<script src="/resources/testharnessreport.js"></script>
+<body><div id="log"></div></body>
+<script>
+  async_test(function(t) {
+    var child = document.createElement("iframe");
+    child.src = "empty.html?1";
+    child.onload = t.step_func(function() {
+      var child1 = child.contentDocument;
+      var link = child1.createElement("a");
+      link.href = "data:text/html,Clicked.";
+      link.innerText = "Link.";
+      child1.body.appendChild(link);
+      var grandchild = child1.createElement("iframe");
+      grandchild.src = "empty.html?2";
+      grandchild.onload = t.step_func(function() {
+        var grandchild1 = grandchild.contentDocument;
+        child.onload = t.step_func(function() {
+          // This is a write to an inactive document
+          child1.write('WRITE HAPPENED');
+          assert_equals(child1.body.lastChild.tagName, "IFRAME");
+          // This is a write to an active but not fully active document
+          grandchild1.write('WRITE HAPPENED');
+          assert_equals(grandchild1.body.innerHTML, "WRITE HAPPENED");
+          t.done();
+        });
+        link.click();
+      });
+      child1.body.appendChild(grandchild);
+    });
+    document.body.appendChild(child);
+  });
+</script>

Original file line number	Diff line number	Diff line change
`@@ -3271,8 +3271,7 @@ impl DocumentMethods for Document {`
`3271`	`3271`
`3272`	`3272`	`// Step 2.`
`3273`	`3273`	`// TODO: handle throw-on-dynamic-markup-insertion counter.`
`3274`		`- // FIXME: this should check for being active rather than fully active`
`3275`		`- if !self.is_fully_active() {`
	`3274`	`+ if !self.is_active() {`
`3276`	`3275`	`// Step 3.`
`3277`	`3276`	`return Ok(());`
`3278`	`3277`	`}`
Original file line number	Diff line number	Diff line change
`@@ -45873,6 +45873,12 @@`
`45873`	`45873`	`"url": "/cssom/stylesheet-same-origin.sub.html"`
`45874`	`45874`	`}`
`45875`	`45875`	`],`
	`45876`	`+ "html/dom/dynamic-markup-insertion/document-write/write-active-document.html": [`
	`45877`	`+ {`
	`45878`	`+ "path": "html/dom/dynamic-markup-insertion/document-write/write-active-document.html",`
	`45879`	`+ "url": "/html/dom/dynamic-markup-insertion/document-write/write-active-document.html"`
	`45880`	`+ }`
	`45881`	`+ ],`
`45876`	`45882`	`"html/semantics/embedded-content/the-iframe-element/iframe-synchronously-discard.html": [`
`45877`	`45883`	`{`
`45878`	`45884`	`"path": "html/semantics/embedded-content/the-iframe-element/iframe-synchronously-discard.html",`