Bug 190810 - Turn on Conic Gradients
Summary: Turn on Conic Gradients
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Megan Gardner
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2018-10-22 15:28 PDT by Megan Gardner
Modified: 2018-10-24 15:28 PDT (History)
8 users (show)

See Also:


Attachments
Patch (53.63 KB, patch)
2018-10-22 15:39 PDT, Megan Gardner
no flags Details | Formatted Diff | Diff
Archive of layout-test-results from ews102 for mac-sierra (2.32 MB, application/zip)
2018-10-22 16:58 PDT, EWS Watchlist
no flags Details
Patch (55.64 KB, patch)
2018-10-22 17:22 PDT, Megan Gardner
no flags Details | Formatted Diff | Diff
Archive of layout-test-results from ews100 for mac-sierra (2.76 MB, application/zip)
2018-10-22 18:38 PDT, EWS Watchlist
no flags Details
Archive of layout-test-results from ews107 for mac-sierra-wk2 (3.27 MB, application/zip)
2018-10-22 19:11 PDT, EWS Watchlist
no flags Details
Archive of layout-test-results from ews115 for mac-sierra (3.15 MB, application/zip)
2018-10-22 19:47 PDT, EWS Watchlist
no flags Details
Archive of layout-test-results from ews123 for ios-simulator-wk2 (2.81 MB, application/zip)
2018-10-22 19:55 PDT, EWS Watchlist
no flags Details
Archive of layout-test-results from ews202 for win-future (12.74 MB, application/zip)
2018-10-22 20:18 PDT, EWS Watchlist
no flags Details
Archive of layout-test-results from ews114 for mac-sierra (3.03 MB, application/zip)
2018-10-22 21:46 PDT, EWS Watchlist
no flags Details
Patch (54.90 KB, patch)
2018-10-23 10:11 PDT, Megan Gardner
no flags Details | Formatted Diff | Diff
Archive of layout-test-results from ews103 for mac-sierra (2.39 MB, application/zip)
2018-10-23 11:31 PDT, EWS Watchlist
no flags Details
Archive of layout-test-results from ews104 for mac-sierra-wk2 (2.82 MB, application/zip)
2018-10-23 11:40 PDT, EWS Watchlist
no flags Details
Archive of layout-test-results from ews113 for mac-sierra (3.10 MB, application/zip)
2018-10-23 12:20 PDT, EWS Watchlist
no flags Details
Archive of layout-test-results from ews206 for win-future (12.72 MB, application/zip)
2018-10-23 12:34 PDT, EWS Watchlist
no flags Details
Archive of layout-test-results from ews117 for mac-sierra (3.06 MB, application/zip)
2018-10-23 14:23 PDT, EWS Watchlist
no flags Details
Archive of layout-test-results from ews205 for win-future (12.69 MB, application/zip)
2018-10-23 14:33 PDT, EWS Watchlist
no flags Details
Patch (57.10 KB, patch)
2018-10-23 15:25 PDT, Megan Gardner
no flags Details | Formatted Diff | Diff
Patch (57.58 KB, patch)
2018-10-24 14:18 PDT, Megan Gardner
no flags Details | Formatted Diff | Diff
Patch for landing (58.36 KB, patch)
2018-10-24 14:51 PDT, Megan Gardner
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Megan Gardner 2018-10-22 15:28:25 PDT
Turn on Conic Gradients
Comment 1 Megan Gardner 2018-10-22 15:39:13 PDT
Created attachment 352918 [details]
Patch
Comment 2 Megan Gardner 2018-10-22 15:39:34 PDT
<rdar://problem/44576827>
Comment 3 EWS Watchlist 2018-10-22 16:58:04 PDT
Comment on attachment 352918 [details]
Patch

Attachment 352918 [details] did not pass mac-ews (mac):
Output: https://webkit-queues.webkit.org/results/9698649

New failing tests:
http/wpt/css/css-images-4/conic-gradient-parsing.html
Comment 4 EWS Watchlist 2018-10-22 16:58:06 PDT
Created attachment 352928 [details]
Archive of layout-test-results from ews102 for mac-sierra

The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: ews102  Port: mac-sierra  Platform: Mac OS X 10.12.6
Comment 5 Megan Gardner 2018-10-22 17:22:28 PDT
Created attachment 352931 [details]
Patch
Comment 6 EWS Watchlist 2018-10-22 18:38:25 PDT
Comment on attachment 352931 [details]
Patch

Attachment 352931 [details] did not pass mac-ews (mac):
Output: https://webkit-queues.webkit.org/results/9700254

New failing tests:
http/wpt/css/css-images-4/conic-gradient-parsing.html
Comment 7 EWS Watchlist 2018-10-22 18:38:27 PDT
Created attachment 352942 [details]
Archive of layout-test-results from ews100 for mac-sierra

The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: ews100  Port: mac-sierra  Platform: Mac OS X 10.12.6
Comment 8 EWS Watchlist 2018-10-22 19:11:25 PDT
Comment on attachment 352931 [details]
Patch

Attachment 352931 [details] did not pass mac-wk2-ews (mac-wk2):
Output: https://webkit-queues.webkit.org/results/9700506

New failing tests:
http/wpt/css/css-images-4/conic-gradient-parsing.html
Comment 9 EWS Watchlist 2018-10-22 19:11:27 PDT
Created attachment 352945 [details]
Archive of layout-test-results from ews107 for mac-sierra-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews107  Port: mac-sierra-wk2  Platform: Mac OS X 10.12.6
Comment 10 EWS Watchlist 2018-10-22 19:47:30 PDT
Comment on attachment 352931 [details]
Patch

Attachment 352931 [details] did not pass mac-debug-ews (mac):
Output: https://webkit-queues.webkit.org/results/9700525

New failing tests:
http/wpt/css/css-images-4/conic-gradient-parsing.html
Comment 11 EWS Watchlist 2018-10-22 19:47:32 PDT
Created attachment 352949 [details]
Archive of layout-test-results from ews115 for mac-sierra

The attached test failures were seen while running run-webkit-tests on the mac-debug-ews.
Bot: ews115  Port: mac-sierra  Platform: Mac OS X 10.12.6
Comment 12 EWS Watchlist 2018-10-22 19:55:08 PDT
Comment on attachment 352931 [details]
Patch

Attachment 352931 [details] did not pass ios-sim-ews (ios-simulator-wk2):
Output: https://webkit-queues.webkit.org/results/9700578

New failing tests:
http/wpt/css/css-images-4/conic-gradient-parsing.html
media/video-aspect-ratio.html
Comment 13 EWS Watchlist 2018-10-22 19:55:10 PDT
Created attachment 352950 [details]
Archive of layout-test-results from ews123 for ios-simulator-wk2

The attached test failures were seen while running run-webkit-tests on the ios-sim-ews.
Bot: ews123  Port: ios-simulator-wk2  Platform: Mac OS X 10.13.6
Comment 14 EWS Watchlist 2018-10-22 20:18:01 PDT
Comment on attachment 352931 [details]
Patch

Attachment 352931 [details] did not pass win-ews (win):
Output: https://webkit-queues.webkit.org/results/9701054

New failing tests:
http/wpt/css/css-images-4/conic-gradient-parsing.html
Comment 15 EWS Watchlist 2018-10-22 20:18:13 PDT
Created attachment 352952 [details]
Archive of layout-test-results from ews202 for win-future

The attached test failures were seen while running run-webkit-tests on the win-ews.
Bot: ews202  Port: win-future  Platform: CYGWIN_NT-6.1-2.10.0-0.325-5-3-x86_64-64bit
Comment 16 EWS Watchlist 2018-10-22 21:46:09 PDT
Comment on attachment 352931 [details]
Patch

Attachment 352931 [details] did not pass mac-debug-ews (mac):
Output: https://webkit-queues.webkit.org/results/9701604

New failing tests:
http/wpt/css/css-images-4/conic-gradient-parsing.html
Comment 17 EWS Watchlist 2018-10-22 21:46:11 PDT
Created attachment 352955 [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 18 Megan Gardner 2018-10-23 10:11:49 PDT
Created attachment 352980 [details]
Patch
Comment 19 EWS Watchlist 2018-10-23 11:31:02 PDT
Comment on attachment 352980 [details]
Patch

Attachment 352980 [details] did not pass mac-ews (mac):
Output: https://webkit-queues.webkit.org/results/9707645

New failing tests:
http/wpt/css/css-images-4/conic-gradient-parsing.html
Comment 20 EWS Watchlist 2018-10-23 11:31:04 PDT
Created attachment 352984 [details]
Archive of layout-test-results from ews103 for mac-sierra

The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: ews103  Port: mac-sierra  Platform: Mac OS X 10.12.6
Comment 21 EWS Watchlist 2018-10-23 11:40:31 PDT
Comment on attachment 352980 [details]
Patch

Attachment 352980 [details] did not pass mac-wk2-ews (mac-wk2):
Output: https://webkit-queues.webkit.org/results/9707655

New failing tests:
http/wpt/css/css-images-4/conic-gradient-parsing.html
Comment 22 EWS Watchlist 2018-10-23 11:40:33 PDT
Created attachment 352985 [details]
Archive of layout-test-results from ews104 for mac-sierra-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews104  Port: mac-sierra-wk2  Platform: Mac OS X 10.12.6
Comment 23 EWS Watchlist 2018-10-23 12:20:35 PDT
Comment on attachment 352980 [details]
Patch

Attachment 352980 [details] did not pass mac-debug-ews (mac):
Output: https://webkit-queues.webkit.org/results/9707727

New failing tests:
http/wpt/css/css-images-4/conic-gradient-parsing.html
Comment 24 EWS Watchlist 2018-10-23 12:20:37 PDT
Created attachment 352986 [details]
Archive of layout-test-results from ews113 for mac-sierra

The attached test failures were seen while running run-webkit-tests on the mac-debug-ews.
Bot: ews113  Port: mac-sierra  Platform: Mac OS X 10.12.6
Comment 25 EWS Watchlist 2018-10-23 12:34:24 PDT
Comment on attachment 352980 [details]
Patch

Attachment 352980 [details] did not pass win-ews (win):
Output: https://webkit-queues.webkit.org/results/9707830

New failing tests:
http/wpt/css/css-images-4/conic-gradient-parsing.html
Comment 26 EWS Watchlist 2018-10-23 12:34:35 PDT
Created attachment 352989 [details]
Archive of layout-test-results from ews206 for win-future

The attached test failures were seen while running run-webkit-tests on the win-ews.
Bot: ews206  Port: win-future  Platform: CYGWIN_NT-6.1-2.9.0-0.318-5-3-x86_64-64bit
Comment 27 EWS Watchlist 2018-10-23 14:23:32 PDT
Comment on attachment 352980 [details]
Patch

Attachment 352980 [details] did not pass mac-debug-ews (mac):
Output: https://webkit-queues.webkit.org/results/9708672

New failing tests:
http/wpt/css/css-images-4/conic-gradient-parsing.html
Comment 28 EWS Watchlist 2018-10-23 14:23:33 PDT
Created attachment 352994 [details]
Archive of layout-test-results from ews117 for mac-sierra

The attached test failures were seen while running run-webkit-tests on the mac-debug-ews.
Bot: ews117  Port: mac-sierra  Platform: Mac OS X 10.12.6
Comment 29 EWS Watchlist 2018-10-23 14:33:21 PDT
Comment on attachment 352980 [details]
Patch

Attachment 352980 [details] did not pass win-ews (win):
Output: https://webkit-queues.webkit.org/results/9708884

New failing tests:
http/wpt/css/css-images-4/conic-gradient-parsing.html
Comment 30 EWS Watchlist 2018-10-23 14:33:33 PDT
Created attachment 352995 [details]
Archive of layout-test-results from ews205 for win-future

The attached test failures were seen while running run-webkit-tests on the win-ews.
Bot: ews205  Port: win-future  Platform: CYGWIN_NT-6.1-2.9.0-0.318-5-3-x86_64-64bit
Comment 31 Megan Gardner 2018-10-23 15:25:34 PDT
Created attachment 352997 [details]
Patch
Comment 32 Tim Horton 2018-10-24 12:27:43 PDT
Comment on attachment 352997 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=352997&action=review

> Source/WebCore/Configurations/FeatureDefines.xcconfig:101
> +ENABLE_CSS_CONIC_GRADIENT_iphonesimulator = ENABLE_CSS_CONIC_GRADIENT;

Where's iosmac?

Also, since that means you have to revise the patch, can it be ENABLE_CSS_CONIC_GRADIENT*S* instead?
Comment 33 Megan Gardner 2018-10-24 14:18:19 PDT
Created attachment 353053 [details]
Patch
Comment 34 Tim Horton 2018-10-24 14:37:06 PDT
Comment on attachment 353053 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=353053&action=review

> Source/WebKit/Configurations/FeatureDefines.xcconfig:103
> +ENABLE_CSS_CONIC_GRADIENTS_iosmac = ENABLE_CSS_CONIC_GRADIENTS;

What about watchOS and watch simulator?
Comment 35 Megan Gardner 2018-10-24 14:51:19 PDT
Created attachment 353056 [details]
Patch for landing
Comment 36 WebKit Commit Bot 2018-10-24 15:28:52 PDT
Comment on attachment 353056 [details]
Patch for landing

Clearing flags on attachment: 353056

Committed r237402: <https://trac.webkit.org/changeset/237402>
Comment 37 WebKit Commit Bot 2018-10-24 15:28:54 PDT
All reviewed patches have been landed.  Closing bug.