Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Migrate iOS 16 expectations to OpenSource #2352

Conversation

robert-jenner
Copy link
Contributor

@robert-jenner robert-jenner commented Jul 13, 2022

4a63e72

Migrate iOS 16 expectations to OpenSource
https://bugs.webkit.org/show_bug.cgi?id=242650

Reviewed by NOBODY (OOPS!).

* LayoutTests/platform/ios/TestExpectations:
* LayoutTests/platform/ios/imported/w3c/web-platform-tests/cookies/value/value-ctl-expected.txt: Added.
* LayoutTests/platform/ios/imported/w3c/web-platform-tests/html/dom/documents/resource-metadata-management/document-cookie-expected.txt: Added.
* LayoutTests/platform/ios/svg/W3C-SVG-1.1/masking-path-04-b-expected.txt:
* LayoutTests/platform/ios/svg/batik/text/textStyles-expected.txt:
* LayoutTests/platform/ios/svg/custom/use-invalid-pattern-expected.txt: Added.
* LayoutTests/platform/ios/svg/custom/use-invalid-style-expected.txt: Added.
* LayoutTests/platform/ios/svg/hixie/error/012-expected.txt:
* Tools/Scripts/webkitpy/port/ios.py:
(IOSPort):
* Tools/Scripts/webkitpy/port/ios_device_unittest.py:
* Tools/Scripts/webkitpy/port/ios_simulator_unittest.py:
(IOSSimulatorTest.make_port):
(IOSSimulatorTest.test_layout_test_searchpath_with_apple_additions):
(IOSSimulatorTest.test_layout_test_searchpath_without_apple_additions):
(IOSSimulatorTest.test_layout_searchpath_wih_device_type):
* Tools/Scripts/webkitpy/port/ios_testcase.py:
(IOSTest.make_port):

@robert-jenner robert-jenner self-assigned this Jul 13, 2022
@robert-jenner robert-jenner added Tools / Tests Tools in the Tools directory, build issues, test infrastructure, and bugs in test cases WebKit Nightly Build labels Jul 13, 2022
@webkit-early-warning-system webkit-early-warning-system added the merging-blocked Applied to prevent a change from being merged label Jul 13, 2022
@JonWBedard
Copy link
Member

Code looks good, @ryanhaddad, are we ready to land this now?

@JonWBedard JonWBedard removed the merging-blocked Applied to prevent a change from being merged label Jul 13, 2022
Comment on lines 3680 to 3681
# rdar://93063355 ([ iOS16 ] 8x imported/w3c/web-platform-tests/mathml/relations/* layout tests are a constant text failure)
imported/w3c/web-platform-tests/mathml/relations/css-styling/display-2.html [ Pass Failure ]
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As these are a constant failure, why is this expectation flaky? Is your goal to avoid adding an ios-15 expectations directory?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have fixed this in my latest edits to this PR. I'm having it go through EWS again just to verify my changes worked.

@ryanhaddad
Copy link
Contributor

Code looks good, @ryanhaddad, are we ready to land this now?

Not yet, we need an ios-15 folder to house baselines to keep existing bots happy.

@robert-jenner robert-jenner force-pushed the eng/Migrate-iOS-16-expectations-to-OpenSource branch 2 times, most recently from 77b57d8 to 5aa249a Compare July 25, 2022 22:56
@webkit-early-warning-system webkit-early-warning-system added the merging-blocked Applied to prevent a change from being merged label Jul 25, 2022
@robert-jenner robert-jenner removed the merging-blocked Applied to prevent a change from being merged label Jul 26, 2022
@robert-jenner robert-jenner changed the title Migrate iOS 16 expectations to OpenSource The background of the main frame's transparent custom scrollbar should be filled with the view background color if !HAVE(RUBBER_BANDING) Jul 26, 2022
@robert-jenner robert-jenner force-pushed the eng/Migrate-iOS-16-expectations-to-OpenSource branch from 3aa3f63 to 54f7d65 Compare July 26, 2022 00:09
@robert-jenner robert-jenner added Compositing Bugs related to layerization for transforms, video, scrolling etc and removed Tools / Tests Tools in the Tools directory, build issues, test infrastructure, and bugs in test cases labels Jul 26, 2022
https://bugs.webkit.org/show_bug.cgi?id=242650

Reviewed by NOBODY (OOPS!).

* LayoutTests/platform/ios/TestExpectations:
* LayoutTests/platform/ios/imported/w3c/web-platform-tests/cookies/value/value-ctl-expected.txt: Added.
* LayoutTests/platform/ios/imported/w3c/web-platform-tests/html/dom/documents/resource-metadata-management/document-cookie-expected.txt: Added.
* LayoutTests/platform/ios/svg/W3C-SVG-1.1/masking-path-04-b-expected.txt:
* LayoutTests/platform/ios/svg/batik/text/textStyles-expected.txt:
* LayoutTests/platform/ios/svg/custom/use-invalid-pattern-expected.txt: Added.
* LayoutTests/platform/ios/svg/custom/use-invalid-style-expected.txt: Added.
* LayoutTests/platform/ios/svg/hixie/error/012-expected.txt:
* Tools/Scripts/webkitpy/port/ios.py:
(IOSPort):
* Tools/Scripts/webkitpy/port/ios_device_unittest.py:
* Tools/Scripts/webkitpy/port/ios_simulator_unittest.py:
(IOSSimulatorTest.make_port):
(IOSSimulatorTest.test_layout_test_searchpath_with_apple_additions):
(IOSSimulatorTest.test_layout_test_searchpath_without_apple_additions):
(IOSSimulatorTest.test_layout_searchpath_wih_device_type):
* Tools/Scripts/webkitpy/port/ios_testcase.py:
(IOSTest.make_port):
@robert-jenner robert-jenner removed Compositing Bugs related to layerization for transforms, video, scrolling etc WebKit Nightly Build labels Jul 26, 2022
@robert-jenner robert-jenner changed the title The background of the main frame's transparent custom scrollbar should be filled with the view background color if !HAVE(RUBBER_BANDING) Migrate iOS 16 expectations to OpenSource Jul 26, 2022
@robert-jenner robert-jenner force-pushed the eng/Migrate-iOS-16-expectations-to-OpenSource branch from 54f7d65 to 4a63e72 Compare July 26, 2022 00:18
@robert-jenner robert-jenner added Tools / Tests Tools in the Tools directory, build issues, test infrastructure, and bugs in test cases WebKit Nightly Build labels Jul 26, 2022
@webkit-early-warning-system webkit-early-warning-system added the merging-blocked Applied to prevent a change from being merged label Jul 26, 2022
@robert-jenner robert-jenner deleted the eng/Migrate-iOS-16-expectations-to-OpenSource branch July 26, 2022 00:31
@robert-jenner robert-jenner restored the eng/Migrate-iOS-16-expectations-to-OpenSource branch July 26, 2022 00:40
@robert-jenner robert-jenner deleted the eng/Migrate-iOS-16-expectations-to-OpenSource branch July 26, 2022 00:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merging-blocked Applied to prevent a change from being merged Tools / Tests Tools in the Tools directory, build issues, test infrastructure, and bugs in test cases
Projects
None yet
6 participants