Bug 116649 – [Mac] [WK2] http/tests/security/cross-frame-access-put.html fails

• Status: RESOLVED
• Resolution: WORKSFORME
• Component: DOM

[Ryosuke Niwa]

http/tests/security/cross-frame-access-put.html has been failing:
http://webkit-test-results.appspot.com/dashboards/flakiness_dashboard.html#tests=http%2Ftests%2Fsecurity%2Fcross-frame-access-put.html

e.g.
http://build.webkit.org/results/Apple%20MountainLion%20Release%20WK2%20(Tests)/r150569%20(9124)/results.html

[Ryosuke Niwa]

Note that this test only fails on WK2.

[Alexey Proskuryakov]

The test is about JS bindings, but the failure is about an incorrect value of window.screenX. Not sure where that's taken from, and why it would be flaky.

[Zoltan Arvai]

Test also fails on Qt WK2

--- /ramdisk/qt-linux-32-release-webkit2/build/layout-test-results/http/tests/security/cross-frame-access-put-expected.txt
+++ /ramdisk/qt-linux-32-release-webkit2/build/layout-test-results/http/tests/security/cross-frame-access-put-actual.txt
@@ -332,9 +332,9 @@
 ALERT: PASS: window.personalbar should be '[object BarProp]' and is.
 ALERT: PASS: window.plugins should be 'undefined' and is.
 ALERT: PASS: window.screen should be '[object Screen]' and is.
-ALERT: PASS: window.screenLeft should be '112' and is.
+ALERT: PASS: window.screenLeft should be '0' and is.
 ALERT: PASS: window.screenTop matched the expected value.
-ALERT: PASS: window.screenX should be '112' and is.
+ALERT: PASS: window.screenX should be '0' and is.
 ALERT: PASS: window.screenY matched the expected value.
 ALERT: PASS: window.scrollbars should be '[object BarProp]' and is.
 ALERT: PASS: window.scrollX should be '0' and is.

http://build.webkit.sed.hu/results/x86-32%20Linux%20Qt%20Release%20WebKit2/r150579%20%2835716%29/results.html

[Zoltan Arvai]

Skipped on Qt WK2 in http://trac.webkit.org/changeset/150584.
Please unskip it with proper fix.

[Alexey Proskuryakov]

Looks like this test got fixed long ago (and broken again everywhere last week for an entirely different reason).

[Alexey Proskuryakov]

Filed bug 126238 for the new brokenness. Removed this old expectation in <http://trac.webkit.org/r161075>.