LayoutTests/platform/wk2/TestExpectations contains expectations for all (or Apple-only?) WebKit2 ports, so they are less-specific than LayoutTests/platform/ios-simulator/TestExpectations, which are shared test expectations for both WK1 and WK2 tests on iOS Simulator.
The current order for ios-simulator-wk2 is this: ios-simulator-wk2, wk2, ios-simulator
I propose changing it to this: ios-simulator-wk2, ios-simulator, wk2
This has the additional benefit of fixing the expected results for this test because it's marked as [ Failure ] in wk2, but [ Skip ] in ios-simulator:
Created attachment 247770 [details]
(In reply to comment #0)
> This has the additional benefit of fixing the expected results for this test
> because it's marked as [ Failure ] in wk2, but [ Skip ] in ios-simulator:
And I think switching the order of test expectations for ios-simulator-wk2 is better than trying to move the [ Failure ] expectation from platform/wk2/TestExpectations to somewhere else (namely each port-specific TestExpectations file).
What is the order on Mac?
For expected results, it's "mac-wk2 -> wk2 -> mac-mavericks -> mac -> generic" (on Mavericks), but I'm not quite sure what the order is for TestExpectations.
(In reply to comment #3)
> What is the order on Mac?
> For expected results, it's "mac-wk2 -> wk2 -> mac-mavericks -> mac ->
> generic" (on Mavericks), but I'm not quite sure what the order is for
For mac-yosemite-wk2, it is: mac-wk2, wk2, mac
Which is basically the same as the "Baseline search path" without "generic" at the end.
I assume the TestExpectations files are parsed in the same order as the test results, or nothing would really make sense. :)
What am I missing here? Should the [ Failure ] result for fast/dynamic/window-resize-scrollbars-test.html move out of wk2/TestExpectations (and into mac-wk2/TestExpectations?) so the [ Skip ] result in ios-simulator/TestExpectations is recognized?
Oops, I didn’t see Alexey’s question.
Comment on attachment 247770 [details]
Clearing flags on attachment: 247770
Committed r181131: <http://trac.webkit.org/changeset/181131>
All reviewed patches have been landed. Closing bug.