RESOLVED INVALID 180053
Add Internals.terminateServiceWorker
https://bugs.webkit.org/show_bug.cgi?id=180053
Summary Add Internals.terminateServiceWorker
Brady Eidson
Reported 2017-11-27 13:38:16 PST
Add Internals.terminateServiceWorker Useful for our own testing and importing other engine's tests
Attachments
Patch (10.54 KB, patch)
2017-11-27 13:41 PST, Brady Eidson
no flags
Patch (10.33 KB, patch)
2017-11-27 14:25 PST, Brady Eidson
cdumez: review+
commit-queue: commit-queue-
Archive of layout-test-results from ews107 for mac-elcapitan-wk2 (2.79 MB, application/zip)
2017-11-27 15:48 PST, EWS Watchlist
no flags
Brady Eidson
Comment 1 2017-11-27 13:41:01 PST
youenn fablet
Comment 2 2017-11-27 13:49:21 PST
Sounds really cool! There are some chromium specific tests that are using the same terminateServiceWorker test runner API. We could try to import these. See https://github.com/w3c/web-platform-tests/issues/6866 for discussion on uploading it to WPT.
Brady Eidson
Comment 3 2017-11-27 13:57:03 PST
(In reply to youenn fablet from comment #2) > Sounds really cool! > There are some chromium specific tests that are using the same > terminateServiceWorker test runner API. > > We could try to import these. > See https://github.com/w3c/web-platform-tests/issues/6866 for discussion on > uploading it to WPT. Yup, that's why I am doing this! :)
Brady Eidson
Comment 4 2017-11-27 14:25:12 PST
EWS Watchlist
Comment 5 2017-11-27 15:48:52 PST
Comment on attachment 327681 [details] Patch Attachment 327681 [details] did not pass mac-wk2-ews (mac-wk2): Output: http://webkit-queues.webkit.org/results/5380415 New failing tests: imported/w3c/web-platform-tests/service-workers/cache-storage/serviceworker/cache-match.https.html
EWS Watchlist
Comment 6 2017-11-27 15:48:54 PST
Created attachment 327705 [details] Archive of layout-test-results from ews107 for mac-elcapitan-wk2 The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews. Bot: ews107 Port: mac-elcapitan-wk2 Platform: Mac OS X 10.11.6
Brady Eidson
Comment 7 2017-11-27 15:56:47 PST
cache-match is a flake, this patch did not change things.
WebKit Commit Bot
Comment 8 2017-11-27 15:58:44 PST
Comment on attachment 327681 [details] Patch Rejecting attachment 327681 [details] from commit-queue. Failed to run "['/Volumes/Data/EWS/WebKit/Tools/Scripts/webkit-patch', '--status-host=webkit-queues.webkit.org', '--bot-id=webkit-cq-03', 'apply-attachment', '--no-update', '--non-interactive', 327681, '--port=mac']" exit_code: 2 cwd: /Volumes/Data/EWS/WebKit Last 500 characters of output: at 86. 1 out of 1 hunk FAILED -- saving rejects to file Source/WebCore/workers/service/server/SWServer.h.rej patching file Source/WebKit/StorageProcess/ServiceWorker/WebSWServerConnection.messages.in patching file Source/WebKit/WebProcess/Storage/WebSWClientConnection.cpp patching file Source/WebKit/WebProcess/Storage/WebSWClientConnection.h Failed to run "[u'/Volumes/Data/EWS/WebKit/Tools/Scripts/svn-apply', '--force', '--reviewer', u'Chris Dumez']" exit_code: 1 cwd: /Volumes/Data/EWS/WebKit Full output: http://webkit-queues.webkit.org/results/5381327
Brady Eidson
Comment 9 2017-11-27 16:27:38 PST
is it just me or has git/svn gotten stupider lately WRT resolving conflicts? Forget it, I'll land this with changes that use it.
Note You need to log in before you can comment on or make changes to this bug.