Bug 180534 - Activate IDB and WebSockets in service workers
Summary: Activate IDB and WebSockets in service workers
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Service Workers (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: youenn fablet
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2017-12-07 10:22 PST by youenn fablet
Modified: 2019-07-18 16:36 PDT (History)
6 users (show)

See Also:


Attachments
Patch (12.67 KB, patch)
2017-12-07 10:27 PST, youenn fablet
no flags Details | Formatted Diff | Diff
Patch for landing (15.26 KB, patch)
2017-12-07 13:09 PST, youenn fablet
no flags Details | Formatted Diff | Diff
Patch for landing (15.26 KB, patch)
2017-12-07 13:12 PST, youenn fablet
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description youenn fablet 2017-12-07 10:22:12 PST
Activate IDB and WebSockets in service workers
Comment 1 youenn fablet 2017-12-07 10:27:17 PST
Created attachment 328702 [details]
Patch
Comment 2 EWS Watchlist 2017-12-07 10:28:33 PST
Attachment 328702 [details] did not pass style-queue:


ERROR: Source/WebCore/workers/service/context/ServiceWorkerThread.cpp:75:  Missing spaces around :  [whitespace/init] [4]
Total errors found: 1 in 11 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 3 Chris Dumez 2017-12-07 10:30:47 PST
Cannot we unskip the websocket in WPT?
Comment 4 youenn fablet 2017-12-07 10:44:30 PST
(In reply to Chris Dumez from comment #3)
> Cannot we unskip the websocket in WPT?

I can try but this test relies on WPT ws socket server.
One of the service worker web socket test is flaky due to certificate issues.
Comment 5 youenn fablet 2017-12-07 11:00:37 PST
(In reply to youenn fablet from comment #4)
> (In reply to Chris Dumez from comment #3)
> > Cannot we unskip the websocket in WPT?
> 
> I can try but this test relies on WPT ws socket server.
> One of the service worker web socket test is flaky due to certificate issues.

WPT test of web socket in service worker is running locally consistently (still failing though, getting an onerror event).
I will unskip it at landing time or in another version of this patch.
Comment 6 youenn fablet 2017-12-07 13:09:13 PST
Created attachment 328721 [details]
Patch for landing
Comment 7 EWS Watchlist 2017-12-07 13:10:40 PST
Attachment 328721 [details] did not pass style-queue:


ERROR: Source/WebCore/workers/service/context/ServiceWorkerThread.cpp:75:  Missing spaces around :  [whitespace/init] [4]
Total errors found: 1 in 13 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 8 WebKit Commit Bot 2017-12-07 13:10:49 PST
Comment on attachment 328721 [details]
Patch for landing

Rejecting attachment 328721 [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-02', 'validate-changelog', '--check-oops', '--non-interactive', 328721, '--port=mac']" exit_code: 1 cwd: /Volumes/Data/EWS/WebKit

ChangeLog entry in LayoutTests/imported/w3c/ChangeLog contains OOPS!.

Full output: http://webkit-queues.webkit.org/results/5533024
Comment 9 youenn fablet 2017-12-07 13:12:16 PST
Created attachment 328723 [details]
Patch for landing
Comment 10 EWS Watchlist 2017-12-07 13:13:47 PST
Attachment 328723 [details] did not pass style-queue:


ERROR: Source/WebCore/workers/service/context/ServiceWorkerThread.cpp:75:  Missing spaces around :  [whitespace/init] [4]
Total errors found: 1 in 13 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 11 WebKit Commit Bot 2017-12-07 13:46:14 PST
Comment on attachment 328723 [details]
Patch for landing

Clearing flags on attachment: 328723

Committed r225644: <https://trac.webkit.org/changeset/225644>
Comment 12 WebKit Commit Bot 2017-12-07 13:46:16 PST
All reviewed patches have been landed.  Closing bug.
Comment 13 Radar WebKit Bug Importer 2017-12-07 13:48:31 PST
<rdar://problem/35918045>