Skip to content

Commit

Permalink
Bug 1644897 [wpt PR 24100] - Reland "Remove Document::IsFeatureEnable…
Browse files Browse the repository at this point in the history
…d, use the ExecutionContext variants instead.", a=testonly

Automatic update from web-platform-tests
Reland "Remove Document::IsFeatureEnabled, use the ExecutionContext variants instead."

Was reverted because of callsites where GetDocument() was guaranteed
non-null, but GetExecutionContext() might be null if the Document is
detached. Fixed and added regression tests for the places where we got
crash reports.

Bug: 1029822
Change-Id: I2fb16bb8a90613b92a4603289bd7df20ffa992a0
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2238891
Commit-Queue: Nate Chapin <[email protected]>
Reviewed-by: Kentaro Hara <[email protected]>
Cr-Commit-Position: refs/heads/master@{#777449}

--

wpt-commits: c3514a1ed476f5e2b9e0d55ddc1cc151f655acc8
wpt-pr: 24100
  • Loading branch information
natechapin authored and moz-wptsync-bot committed Jun 17, 2020
1 parent 145c393 commit 6425049
Show file tree
Hide file tree
Showing 3 changed files with 21 additions and 0 deletions.
Original file line number Diff line number Diff line change
@@ -0,0 +1,8 @@
<audio id="a"></audio>
<iframe id="i"></iframe>
<script>
var a = document.getElementById("a");
i.contentDocument.documentElement.appendChild(a);
i.remove();
a.play();
</script>
Original file line number Diff line number Diff line change
@@ -0,0 +1,6 @@
<iframe id="i"></iframe>
<script>
var doc = i.contentDocument;
i.remove();
doc.createElement("img");
</script>
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
<iframe id="i"></iframe>
<video id="v"></video>
<script>
var doc = i.contentDocument;
i.remove();
doc.importNode(v);
</script>

0 comments on commit 6425049

Please sign in to comment.