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

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

Issue 517043003: Move Frame to the Oilpan heap. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Comments + fix fast/events/message-port-gc-closed.html 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..2da37c53a7ad49752303f38048246ffb777fac70 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 GCed (=> window.closed = true.)
+ collectProperties(true);
frame.parentNode.removeChild(frame);
window.setTimeout(verifyResults, 0);
}

Powered by Google App Engine
This is Rietveld 408576698