Title: [196002] trunk/LayoutTests
Revision
196002
Author
[email protected]
Date
2016-02-01 22:12:38 -0800 (Mon, 01 Feb 2016)

Log Message

Unreviewed, fix http/tests/security/cross-origin-window-property-access.html

Update http/tests/security/cross-origin-window-property-access.html that was
introduced in r196001 so that it does not assert on Mac DRT. Mac DRT asserts
that testRunner.waitUntilDone() must be called *BEFORE* calling window.open().

* http/tests/security/cross-origin-window-property-access.html:

Modified Paths

Diff

Modified: trunk/LayoutTests/ChangeLog (196001 => 196002)


--- trunk/LayoutTests/ChangeLog	2016-02-02 03:47:51 UTC (rev 196001)
+++ trunk/LayoutTests/ChangeLog	2016-02-02 06:12:38 UTC (rev 196002)
@@ -1,5 +1,15 @@
 2016-02-01  Chris Dumez  <[email protected]>
 
+        Unreviewed, fix http/tests/security/cross-origin-window-property-access.html
+
+        Update http/tests/security/cross-origin-window-property-access.html that was
+        introduced in r196001 so that it does not assert on Mac DRT. Mac DRT asserts
+        that testRunner.waitUntilDone() must be called *BEFORE* calling window.open().
+
+        * http/tests/security/cross-origin-window-property-access.html:
+
+2016-02-01  Chris Dumez  <[email protected]>
+
         Native Bindings Descriptors are Incomplete
         https://bugs.webkit.org/show_bug.cgi?id=140575
         <rdar://problem/19506502>

Modified: trunk/LayoutTests/http/tests/security/cross-origin-window-property-access.html (196001 => 196002)


--- trunk/LayoutTests/http/tests/security/cross-origin-window-property-access.html	2016-02-02 03:47:51 UTC (rev 196001)
+++ trunk/LayoutTests/http/tests/security/cross-origin-window-property-access.html	2016-02-02 06:12:38 UTC (rev 196002)
@@ -3,18 +3,21 @@
 <head>
 <script src=""
 </head>
-<body>
+<body _onload_="runTest()">
 <script>
 description("Tests that using another window's property getter does not bypass cross-origin checks.");
 jsTestIsAsync = true;
 if (window.testRunner)
     testRunner.setCanOpenWindows();
 
-var crossOriginWindow = window.open("http://127.0.0.1:8000/security/resources/blank.html");
-crossOriginWindow._onload_ = function() {
-    shouldThrow('Object.getOwnPropertyDescriptor(window, "document").get.call(crossOriginWindow)');
-    shouldThrow('Object.getOwnPropertyDescriptor(window, "location").get.call(crossOriginWindow)');
-    finishJSTest();
+function runTest()
+{
+    var crossOriginWindow = window.open("http://127.0.0.1:8000/security/resources/blank.html");
+    crossOriginWindow._onload_ = function() {
+        shouldThrow('Object.getOwnPropertyDescriptor(window, "document").get.call(crossOriginWindow)');
+        shouldThrow('Object.getOwnPropertyDescriptor(window, "location").get.call(crossOriginWindow)');
+        finishJSTest();
+    }
 }
 </script>
 </body>
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to