Bug 195418

Summary: Delete WebMetal implementation in favor of WebGPU
Product: WebKit Reporter: Myles C. Maxfield <mmaxfield>
Component: New BugsAssignee: Myles C. Maxfield <mmaxfield>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, dino, ews-watchlist, hi, jonlee, justin_fan, rniwa, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
dino: review+
Patch
none
Patch for committing
none
Patch for committing
none
Patch for committing
none
Patch for committing
ews-watchlist: commit-queue-
Archive of layout-test-results from ews102 for mac-highsierra
none
Archive of layout-test-results from ews105 for mac-highsierra-wk2
none
Archive of layout-test-results from ews114 for mac-highsierra
none
Patch for committing
none
WIP
none
Patch for committing
none
Patch for committing none

Description Myles C. Maxfield 2019-03-07 10:41:53 PST
Delete WebMetal implementation in favor of WebGPU
Comment 1 Myles C. Maxfield 2019-03-07 10:44:39 PST
Created attachment 363896 [details]
Patch
Comment 2 Dean Jackson 2019-03-07 10:47:12 PST
Comment on attachment 363896 [details]
Patch

I couldn't find them, but I assume the tests in Tools are also removed.
Comment 3 Myles C. Maxfield 2019-03-07 10:57:21 PST
Created attachment 363899 [details]
Patch
Comment 4 Myles C. Maxfield 2019-03-07 10:59:22 PST
Created attachment 363900 [details]
Patch for committing
Comment 5 Myles C. Maxfield 2019-03-07 10:59:53 PST
I will defer committing until we have compute facilities in WebGPU.
Comment 6 Myles C. Maxfield 2019-03-07 16:14:45 PST
Created attachment 363948 [details]
Patch for committing
Comment 7 Myles C. Maxfield 2019-03-07 17:16:34 PST
Created attachment 363962 [details]
Patch for committing
Comment 8 Myles C. Maxfield 2019-03-07 17:35:19 PST
Created attachment 363963 [details]
Patch for committing
Comment 9 EWS Watchlist 2019-03-07 18:49:21 PST
Comment on attachment 363963 [details]
Patch for committing

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

New failing tests:
inspector/canvas/resolveCanvasContext-webmetal.html
fast/canvas/webmetal/webmetal-dispatch.html
fast/canvas/webmetal/webmetal-runtime-flag.html
Comment 10 EWS Watchlist 2019-03-07 18:49:23 PST
Created attachment 363971 [details]
Archive of layout-test-results from ews102 for mac-highsierra

The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: ews102  Port: mac-highsierra  Platform: Mac OS X 10.13.6
Comment 11 EWS Watchlist 2019-03-07 19:08:37 PST
Comment on attachment 363963 [details]
Patch for committing

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

New failing tests:
inspector/canvas/resolveCanvasContext-webmetal.html
fast/canvas/webmetal/webmetal-dispatch.html
fast/canvas/webmetal/webmetal-runtime-flag.html
Comment 12 EWS Watchlist 2019-03-07 19:08:39 PST
Created attachment 363973 [details]
Archive of layout-test-results from ews105 for mac-highsierra-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews105  Port: mac-highsierra-wk2  Platform: Mac OS X 10.13.6
Comment 13 EWS Watchlist 2019-03-08 07:17:11 PST
Comment on attachment 363963 [details]
Patch for committing

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

New failing tests:
inspector/canvas/resolveCanvasContext-webmetal.html
fast/canvas/webmetal/webmetal-dispatch.html
fast/canvas/webmetal/webmetal-runtime-flag.html
Comment 14 EWS Watchlist 2019-03-08 07:17:13 PST
Created attachment 364010 [details]
Archive of layout-test-results from ews114 for mac-highsierra

The attached test failures were seen while running run-webkit-tests on the mac-debug-ews.
Bot: ews114  Port: mac-highsierra  Platform: Mac OS X 10.13.6
Comment 15 Myles C. Maxfield 2019-03-11 15:02:12 PDT
Created attachment 364293 [details]
Patch for committing
Comment 16 Myles C. Maxfield 2019-03-25 21:42:33 PDT
Created attachment 365942 [details]
WIP
Comment 17 Myles C. Maxfield 2019-03-25 21:43:48 PDT
Created attachment 365944 [details]
Patch for committing
Comment 18 Myles C. Maxfield 2019-03-29 12:01:37 PDT
Created attachment 366290 [details]
Patch for committing
Comment 19 WebKit Commit Bot 2019-03-29 15:19:08 PDT
Comment on attachment 366290 [details]
Patch for committing

Clearing flags on attachment: 366290

Committed r243666: <https://trac.webkit.org/changeset/243666>
Comment 20 Devin Rousso 2019-03-29 15:58:16 PDT
Comment on attachment 366290 [details]
Patch for committing

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

> Source/WebInspectorUI/UserInterface/Models/Canvas.js:-78
> -        case CanvasAgent.ContextType.WebMetal:

We actually still want this to exist, so that we support older backends (just like how we "still" have WebGPU (meaning the older WebGPU, which was renamed to WebMetal)).

<https://webkit.org/b/196412>
Comment 21 Radar WebKit Bug Importer 2019-03-29 18:36:38 PDT
<rdar://problem/49443093>