Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(545)

Unified Diff: LayoutTests/fast/dom/Window/property-access-on-cached-properties-after-frame-removed-and-gced.html

Issue 538323003: Have window.closed return true when frame is closed. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Un-inline and improve formatting of Frame::setHasBeenClosed() Created 6 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: LayoutTests/fast/dom/Window/property-access-on-cached-properties-after-frame-removed-and-gced.html
diff --git a/LayoutTests/fast/dom/Window/property-access-on-cached-properties-after-frame-removed-and-gced.html b/LayoutTests/fast/dom/Window/property-access-on-cached-properties-after-frame-removed-and-gced.html
index 45490b58dabeb1c25957597e892419db62f74790..96b703393fdb1afa6fd4a705d6d4a27e42ddf7c8 100644
--- a/LayoutTests/fast/dom/Window/property-access-on-cached-properties-after-frame-removed-and-gced.html
+++ b/LayoutTests/fast/dom/Window/property-access-on-cached-properties-after-frame-removed-and-gced.html
@@ -26,7 +26,8 @@ function runTest()
description("Tests access of cached DOMWindow properties after the associated frame is removed from a web page and garbage collected. Test should not crash and properties should be set to sane defaults.");
var frame = document.getElementById("frame");
childWindow = frame.contentWindow;
- collectProperties();
+ // Have expected results assume that the frame has been closed (=> window.closed = true.)
+ collectProperties(true);
frame.parentNode.removeChild(frame);
window.setTimeout(verifyResults, 0);
}

Powered by Google App Engine
This is Rietveld 408576698