1214857 - getElementsByTagName should snapshot the HTMLness of the document at call time

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, defect)

Comment 1

[Boris Zbarsky [:bzbarsky]]

Attachment: Store the document-is-HTML state directly in nsContentList instead of refetching from the node being matched

Comment 2

[Olli Pettay [:smaug][[e-mail]]]

Comment on attachment 8674271 [details] [diff] [review]
Store the document-is-HTML state directly in nsContentList instead of refetching from the node being matched

>+  // We have to take an aIsHTMLDocument arg for two reasons:
>+  // 1)  We don't want to include nsIDocument.h in this header.
>+  // 2) We need to do that to make nsContentList::RemoveFromHashtable
Extra space after 1)

>+++ b/testing/web-platform/mozilla/meta/MANIFEST.json
>@@ -422,16 +422,22 @@
>           }
>         ],
>         "service-workers/service-worker/resources/invalid-header-iframe.https.html": [
>           {
>             "path": "service-workers/service-worker/resources/invalid-header-iframe.https.html",
>             "url": "/_mozilla/service-workers/service-worker/resources/invalid-header-iframe.https.html"
>           }
>         ],
>+        "service-workers/service-worker/resources/worker-interception-iframe.https.html": [
>+          {
>+            "path": "service-workers/service-worker/resources/worker-interception-iframe.https.html",
>+            "url": "/_mozilla/service-workers/service-worker/resources/worker-interception-iframe.https.html"
>+          }
>+        ],

This is some unrelated change

Comment 3

[Boris Zbarsky [:bzbarsky]]

> This is some unrelated change

This change is due to running the manifest update script after adding the new test.  Basically, someone added that serviceworker test but didn't update the manifest so the test is not running right now....

Comment 4

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/413327d4c524

Comment 5

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/413327d4c524