Bug 180209

Summary: Update RGB/RGBA parsing to match CSS Color 4
Product: WebKit Reporter: Chris Nardi <cnardi>
Component: CSSAssignee: Nobody <webkit-unassigned>
Status: NEW ---    
Severity: Normal CC: dino, ews-watchlist, rniwa, simon.fraser
Priority: P2    
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Archive of layout-test-results from ews101 for mac-sierra
none
Archive of layout-test-results from ews106 for mac-sierra-wk2
none
Archive of layout-test-results from ews114 for mac-sierra
none
Archive of layout-test-results from ews122 for ios-simulator-wk2
none
Update tests cnardi: review?

Description Chris Nardi 2017-11-30 11:29:39 PST
In CSS Color 4, there have been several changes to the grammar of the RGB/RGBA functions. See https://drafts.csswg.org/css-color/#rgb-functions. WebKit does not appear to have implemented any of the changes (e.g. percent alpha values, comma-less syntax, and making the two functions synonyms). As a result, Safari is failing all of the RGB/RGBA tests at https://wpt.fyi/css/css-color.
Comment 1 Chris Nardi 2018-05-21 14:48:46 PDT
Created attachment 340897 [details]
Patch
Comment 2 EWS Watchlist 2018-05-21 16:00:31 PDT
Comment on attachment 340897 [details]
Patch

Attachment 340897 [details] did not pass mac-ews (mac):
Output: http://webkit-queues.webkit.org/results/7757962

New failing tests:
imported/w3c/web-platform-tests/css/css-color/rgb-006.html
imported/w3c/web-platform-tests/css/css-color/rgba-002.html
canvas/philip/tests/2d.fillStyle.parse.invalid.rgba-1.html
fast/canvas/gradient-addColorStop-with-invalid-color.html
imported/w3c/web-platform-tests/css/css-color/rgb-004.html
imported/w3c/canvas/2d.fillStyle.parse.invalid.rgb-7.html
imported/w3c/web-platform-tests/css/css-color/rgba-008.html
fast/css/rgb-float.html
imported/w3c/web-platform-tests/css/css-color/rgb-002.html
canvas/philip/tests/2d.fillStyle.parse.invalid.rgba-2.html
canvas/philip/tests/2d.fillStyle.parse.invalid.rgb-2.html
css2.1/t040306-syntax-01-f.html
canvas/philip/tests/2d.fillStyle.parse.invalid.rgb-1.html
imported/w3c/canvas/2d.fillStyle.parse.invalid.rgba-1.html
canvas/philip/tests/2d.fillStyle.parse.invalid.rgb-7.html
canvas/philip/tests/2d.fillStyle.parse.invalid.rgb-5.html
canvas/philip/tests/2d.fillStyle.parse.invalid.rgba-4.html
imported/w3c/canvas/2d.fillStyle.parse.invalid.rgba-2.html
imported/w3c/web-platform-tests/css/css-color/rgba-006.html
imported/w3c/canvas/2d.fillStyle.parse.invalid.rgb-2.html
imported/w3c/web-platform-tests/css/css-color/rgba-004.html
imported/w3c/canvas/2d.fillStyle.parse.invalid.rgba-4.html
imported/w3c/canvas/2d.fillStyle.parse.invalid.rgb-5.html
imported/w3c/canvas/2d.fillStyle.parse.invalid.rgb-1.html
imported/w3c/web-platform-tests/css/css-color/rgb-008.html
Comment 3 EWS Watchlist 2018-05-21 16:00:32 PDT
Created attachment 340914 [details]
Archive of layout-test-results from ews101 for mac-sierra

The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: ews101  Port: mac-sierra  Platform: Mac OS X 10.12.6
Comment 4 EWS Watchlist 2018-05-21 16:10:31 PDT
Comment on attachment 340897 [details]
Patch

Attachment 340897 [details] did not pass mac-wk2-ews (mac-wk2):
Output: http://webkit-queues.webkit.org/results/7757995

New failing tests:
imported/w3c/web-platform-tests/css/css-color/rgb-006.html
imported/w3c/web-platform-tests/css/css-color/rgba-002.html
canvas/philip/tests/2d.fillStyle.parse.invalid.rgba-1.html
fast/canvas/gradient-addColorStop-with-invalid-color.html
imported/w3c/web-platform-tests/css/css-color/rgb-004.html
imported/w3c/canvas/2d.fillStyle.parse.invalid.rgb-7.html
imported/w3c/web-platform-tests/css/css-color/rgba-008.html
fast/css/rgb-float.html
imported/w3c/web-platform-tests/css/css-color/rgb-002.html
canvas/philip/tests/2d.fillStyle.parse.invalid.rgba-2.html
canvas/philip/tests/2d.fillStyle.parse.invalid.rgb-2.html
css2.1/t040306-syntax-01-f.html
canvas/philip/tests/2d.fillStyle.parse.invalid.rgb-1.html
imported/w3c/canvas/2d.fillStyle.parse.invalid.rgba-1.html
canvas/philip/tests/2d.fillStyle.parse.invalid.rgb-7.html
canvas/philip/tests/2d.fillStyle.parse.invalid.rgb-5.html
canvas/philip/tests/2d.fillStyle.parse.invalid.rgba-4.html
imported/w3c/canvas/2d.fillStyle.parse.invalid.rgba-2.html
imported/w3c/web-platform-tests/css/css-color/rgba-006.html
imported/w3c/canvas/2d.fillStyle.parse.invalid.rgb-2.html
imported/w3c/web-platform-tests/css/css-color/rgba-004.html
imported/w3c/canvas/2d.fillStyle.parse.invalid.rgba-4.html
imported/w3c/canvas/2d.fillStyle.parse.invalid.rgb-5.html
imported/w3c/canvas/2d.fillStyle.parse.invalid.rgb-1.html
imported/w3c/web-platform-tests/css/css-color/rgb-008.html
Comment 5 EWS Watchlist 2018-05-21 16:10:33 PDT
Created attachment 340917 [details]
Archive of layout-test-results from ews106 for mac-sierra-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews106  Port: mac-sierra-wk2  Platform: Mac OS X 10.12.6
Comment 6 EWS Watchlist 2018-05-21 16:37:07 PDT
Comment on attachment 340897 [details]
Patch

Attachment 340897 [details] did not pass mac-debug-ews (mac):
Output: http://webkit-queues.webkit.org/results/7758175

New failing tests:
imported/w3c/web-platform-tests/css/css-color/rgb-006.html
imported/w3c/web-platform-tests/css/css-color/rgba-002.html
canvas/philip/tests/2d.fillStyle.parse.invalid.rgba-1.html
fast/canvas/gradient-addColorStop-with-invalid-color.html
imported/w3c/web-platform-tests/css/css-color/rgb-004.html
imported/w3c/canvas/2d.fillStyle.parse.invalid.rgb-7.html
imported/w3c/web-platform-tests/css/css-color/rgba-008.html
fast/css/rgb-float.html
imported/w3c/web-platform-tests/css/css-color/rgb-002.html
canvas/philip/tests/2d.fillStyle.parse.invalid.rgba-2.html
canvas/philip/tests/2d.fillStyle.parse.invalid.rgb-2.html
css2.1/t040306-syntax-01-f.html
canvas/philip/tests/2d.fillStyle.parse.invalid.rgb-1.html
imported/w3c/canvas/2d.fillStyle.parse.invalid.rgba-1.html
canvas/philip/tests/2d.fillStyle.parse.invalid.rgb-7.html
canvas/philip/tests/2d.fillStyle.parse.invalid.rgb-5.html
canvas/philip/tests/2d.fillStyle.parse.invalid.rgba-4.html
imported/w3c/canvas/2d.fillStyle.parse.invalid.rgba-2.html
imported/w3c/web-platform-tests/css/css-color/rgba-006.html
imported/w3c/canvas/2d.fillStyle.parse.invalid.rgb-2.html
imported/w3c/web-platform-tests/css/css-color/rgba-004.html
imported/w3c/canvas/2d.fillStyle.parse.invalid.rgba-4.html
imported/w3c/canvas/2d.fillStyle.parse.invalid.rgb-5.html
imported/w3c/canvas/2d.fillStyle.parse.invalid.rgb-1.html
imported/w3c/web-platform-tests/css/css-color/rgb-008.html
Comment 7 EWS Watchlist 2018-05-21 16:37:09 PDT
Created attachment 340922 [details]
Archive of layout-test-results from ews114 for mac-sierra

The attached test failures were seen while running run-webkit-tests on the mac-debug-ews.
Bot: ews114  Port: mac-sierra  Platform: Mac OS X 10.12.6
Comment 8 EWS Watchlist 2018-05-21 16:38:32 PDT
Comment on attachment 340897 [details]
Patch

Attachment 340897 [details] did not pass ios-sim-ews (ios-simulator-wk2):
Output: http://webkit-queues.webkit.org/results/7758062

New failing tests:
imported/w3c/web-platform-tests/css/css-color/rgb-006.html
imported/w3c/web-platform-tests/css/css-color/rgba-002.html
canvas/philip/tests/2d.fillStyle.parse.invalid.rgba-1.html
fast/canvas/gradient-addColorStop-with-invalid-color.html
canvas/philip/tests/2d.fillStyle.parse.invalid.rgb-7.html
imported/w3c/canvas/2d.fillStyle.parse.invalid.rgb-7.html
imported/w3c/web-platform-tests/css/css-color/rgba-008.html
fast/css/rgb-float.html
imported/w3c/web-platform-tests/css/css-color/rgb-002.html
canvas/philip/tests/2d.fillStyle.parse.invalid.rgba-2.html
canvas/philip/tests/2d.fillStyle.parse.invalid.rgb-2.html
css2.1/t040306-syntax-01-f.html
canvas/philip/tests/2d.fillStyle.parse.invalid.rgb-1.html
imported/w3c/canvas/2d.fillStyle.parse.invalid.rgba-1.html
imported/w3c/web-platform-tests/css/css-color/rgb-004.html
canvas/philip/tests/2d.fillStyle.parse.invalid.rgb-5.html
canvas/philip/tests/2d.fillStyle.parse.invalid.rgba-4.html
imported/w3c/canvas/2d.fillStyle.parse.invalid.rgba-2.html
imported/w3c/web-platform-tests/css/css-color/rgba-006.html
imported/w3c/canvas/2d.fillStyle.parse.invalid.rgb-2.html
imported/w3c/web-platform-tests/css/css-color/rgba-004.html
imported/w3c/canvas/2d.fillStyle.parse.invalid.rgba-4.html
imported/w3c/canvas/2d.fillStyle.parse.invalid.rgb-5.html
imported/w3c/canvas/2d.fillStyle.parse.invalid.rgb-1.html
imported/w3c/web-platform-tests/css/css-color/rgb-008.html
Comment 9 EWS Watchlist 2018-05-21 16:38:33 PDT
Created attachment 340923 [details]
Archive of layout-test-results from ews122 for ios-simulator-wk2

The attached test failures were seen while running run-webkit-tests on the ios-sim-ews.
Bot: ews122  Port: ios-simulator-wk2  Platform: Mac OS X 10.13.4
Comment 10 Chris Nardi 2018-05-21 16:52:26 PDT
Created attachment 340926 [details]
Update tests
Comment 11 Chris Nardi 2018-06-12 09:47:49 PDT
Simon: have you had a chance to take a look at this patch?