Bug 116649
| Summary: | [Mac] [WK2] http/tests/security/cross-frame-access-put.html fails | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Ryosuke Niwa <rniwa> |
| Component: | DOM | Assignee: | Nobody <webkit-unassigned> |
| Status: | RESOLVED WORKSFORME | ||
| Severity: | Normal | CC: | ap, beidson, simon.fraser, thorton, zarvai |
| Priority: | P2 | ||
| Version: | 528+ (Nightly build) | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
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
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
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>.