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

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

Issue 2706923002: Rework security checks to be based on Window rather than Frame. (Closed)
Patch Set: Do not hardcode V8Window::wrapperTypeInfo Created 3 years, 7 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: third_party/WebKit/LayoutTests/fast/dom/Window/property-access-on-cached-window-after-frame-removed-and-gced.html
diff --git a/third_party/WebKit/LayoutTests/fast/dom/Window/property-access-on-cached-window-after-frame-removed-and-gced.html b/third_party/WebKit/LayoutTests/fast/dom/Window/property-access-on-cached-window-after-frame-removed-and-gced.html
index 73497457d17cca8adca23cc5062a9b064ed7f15b..1afb262aa06513c99f1cc04bae32fc041b86a866 100644
--- a/third_party/WebKit/LayoutTests/fast/dom/Window/property-access-on-cached-window-after-frame-removed-and-gced.html
+++ b/third_party/WebKit/LayoutTests/fast/dom/Window/property-access-on-cached-window-after-frame-removed-and-gced.html
@@ -10,7 +10,10 @@ var propertiesToVerify = [];
function insertExpectedResult(path, expected)
{
- propertiesToVerify.push({"path": path.slice(0), "property": path.join("."), "expected": expected});
+ var property = path.join(".");
+ if (property == "location.ancestorOrigins.length")
+ expected = '0';
+ propertiesToVerify.push({"path": path.slice(0), "property": property, "expected": expected});
}
function runTest()

Powered by Google App Engine
This is Rietveld 408576698