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

Unified Diff: LayoutTests/http/tests/security/cross-frame-access-delete.html

Issue 19932002: Throw exceptions on all failed cross-origin access checks. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: test. Created 7 years, 5 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/http/tests/security/cross-frame-access-delete.html
diff --git a/LayoutTests/http/tests/security/cross-frame-access-delete.html b/LayoutTests/http/tests/security/cross-frame-access-delete.html
index 2f11139f847fa42605a8f237adedb025c90328fc..deb6a8a8f3d0011ff9cdc465ad64ccbc75e88d21 100644
--- a/LayoutTests/http/tests/security/cross-frame-access-delete.html
+++ b/LayoutTests/http/tests/security/cross-frame-access-delete.html
@@ -19,8 +19,8 @@
{
targetWindow = frames[0];
- shouldBe("eval('delete targetWindow.existingProperty')", "false");
- shouldBe("eval('delete targetWindow[1]')", "false");
+ shouldBeTrue("deletionThrowsException('targetWindow.existingProperty')");
+ shouldBeTrue("deletionThrowsException('targetWindow[1]')");
shouldBeTrue("deletionThrowsException('targetWindow.location.existingProperty')");
shouldBeTrue("deletionThrowsException('targetWindow.location[1]')", "false");

Powered by Google App Engine
This is Rietveld 408576698