Layout Test imported/wpt/html/browsers/origin/cross-origin-objects/cross-origin-objects.html is failing |
|||
Issue descriptionThe following layout test is failing on all platforms: imported/wpt/html/browsers/origin/cross-origin-objects/cross-origin-objects.html It looks like it was passing yesterday, and then started failing one way when tkent's change to enable wptserve landed in https://codereview.chromium.org/2482793002/, and then started failing a different way when I reverted that change in https://codereview.chromium.org/2495613002/ It's not immediately obvious to me what changed, but it's possible some CL landed while wptserve was enabled that changed the non-wptserve behavior. I'm filing this bug to track down what that was. In addition, it looks like the layout-test-results actual results .json file might be busted, because if you click on the "expected failures" checkbox in the results.html UI, you'll see a bunch of "expected failures" that are listed as Pass/Pass and that don't seem to be failing: https://storage.googleapis.com/chromium-layout-test-archives/WebKit_Linux_Precise/2241/layout-test-results/results.html this is probably a separate bug, but I'm mentioning it here because it's adding to my confusion :).
,
Nov 10 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/9c9e46be729f430ea0f6f0c50e70d19c8fd8f2a7 commit 9c9e46be729f430ea0f6f0c50e70d19c8fd8f2a7 Author: dpranke <dpranke@chromium.org> Date: Thu Nov 10 19:32:54 2016 Suppress weird WPT cross-origin-objects.html layout test failure. TBR=qyearsley@chromium.org, tkent@chromium.org BUG= 664256 NOTRY=true Review-Url: https://codereview.chromium.org/2486373003 Cr-Commit-Position: refs/heads/master@{#431316} [modify] https://crrev.com/9c9e46be729f430ea0f6f0c50e70d19c8fd8f2a7/third_party/WebKit/LayoutTests/TestExpectations
,
Nov 10 2016
On WebKit Linux Precise, build 2248 is the first one that has the new diff:
-FAIL [[SetPrototypeOf]] should throw assert_throws: proto set on cross-origin Window function "function () { C.__proto__ = new Object(); }" did not throw
+FAIL [[SetPrototypeOf]] should throw assert_throws: proto set on cross-origin Location function "function () { C.location.__proto__ = new Object(); }" did not throw
Before:
https://storage.googleapis.com/chromium-layout-test-archives/WebKit_Linux_Precise/2247/layout-test-results/imported/wpt/html/browsers/origin/cross-origin-objects/cross-origin-objects-pretty-diff.html
After:
https://storage.googleapis.com/chromium-layout-test-archives/WebKit_Linux_Precise/2248/layout-test-results/imported/wpt/html/browsers/origin/cross-origin-objects/cross-origin-objects-pretty-diff.html
Blamelist:
https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Linux%20Precise/builds/2248
Most interesting commit in that range is c65700010f3c557867e23b24373499170959f845, but I'm not sure whether that change explains this change in imported/wpt/html/browsers/origin/cross-origin-objects/cross-origin-objects.html yet.
One way to investigate would be to locally run this test with and without a revert of that change; if the test result changes, then that indicates that that change is the cause. Then, the most obvious fix would be to rebaseline the test.
,
Nov 10 2016
Yup, I expect c6570001 is to blame.
,
Nov 10 2016
Yep, it's my patch. Reverting it shows that the old expectations are found. It looks like the change resulted in a test which was previously failing to pass. The assertion on line 131 of html/browsers/origin/cross-origin-objects/cross-origin-objects.html was previously failing, but it now throws as expected, and this leads to the assertion on 132 being reached (and then failing as it would have done before). I think the appropriate thing to do here is to change the test expectations to what is output now. What I don't understand is why there are so many failing lines in this test. It seems like it would be a good idea for someone to look into that.
,
Nov 11 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/6e4c3b357983e3d687d135297ed9f0f7c44031f5 commit 6e4c3b357983e3d687d135297ed9f0f7c44031f5 Author: littledan <littledan@chromium.org> Date: Fri Nov 11 04:26:45 2016 Update test expectations for a test which passes slightly more Freezing the prototype chain of the global object made a line of a test pass which was checking that, cross-origin, this chain cannot be manipulated. This patch updates the test expectations for the test doing better. BUG= 664256 , v8:5149 Review-Url: https://codereview.chromium.org/2486943006 Cr-Commit-Position: refs/heads/master@{#431496} [modify] https://crrev.com/6e4c3b357983e3d687d135297ed9f0f7c44031f5/third_party/WebKit/LayoutTests/TestExpectations [modify] https://crrev.com/6e4c3b357983e3d687d135297ed9f0f7c44031f5/third_party/WebKit/LayoutTests/imported/wpt/html/browsers/origin/cross-origin-objects/cross-origin-objects-expected.txt
,
Nov 11 2016
Thanks littledan@! |
|||
►
Sign in to add a comment |
|||
Comment 1 by dpranke@chromium.org
, Nov 10 2016