Bug 222155

Summary: Support modules in service workers
Product: WebKit Reporter: Yusuke Suzuki <ysuzuki>
Component: Service WorkersAssignee: Yusuke Suzuki <ysuzuki>
Status: RESOLVED FIXED    
Severity: Normal CC: beidson, saam, webkit-bug-importer, youennf, ysuzuki
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch
none
Patch
none
Patch saam: review+

Yusuke Suzuki
Reported 2021-02-18 19:58:33 PST
...
Attachments
Patch (21.85 KB, patch)
2021-02-20 14:36 PST, Yusuke Suzuki 		no flags
DetailsFormatted DiffDiff
Patch (27.34 KB, patch)
2021-02-20 23:23 PST, Yusuke Suzuki 		no flags
DetailsFormatted DiffDiff
Patch (27.53 KB, patch)
2021-02-20 23:27 PST, Yusuke Suzuki 		no flags
DetailsFormatted DiffDiff
Patch (26.42 KB, patch)
2021-02-20 23:30 PST, Yusuke Suzuki 		saam: review+
DetailsFormatted DiffDiff
Show Obsolete (3) View All Add attachment proposed patch, testcase, etc.
Yusuke Suzuki
Comment 1 2021-02-20 14:36:35 PST
Created attachment 421114 [details] Patch
Yusuke Suzuki
Comment 2 2021-02-20 23:23:05 PST
Created attachment 421130 [details] Patch
Yusuke Suzuki
Comment 3 2021-02-20 23:27:56 PST
Created attachment 421131 [details] Patch
Yusuke Suzuki
Comment 4 2021-02-20 23:30:30 PST
Created attachment 421132 [details] Patch
Radar WebKit Bug Importer
Comment 5 2021-02-20 23:49:09 PST
<rdar://problem/74564317>
Saam Barati
Comment 6 2021-02-21 08:55:36 PST
Comment on attachment 421132 [details] Patch LGTM. Should we add some of our own tests too that aren’t imported?
Yusuke Suzuki
Comment 7 2021-02-21 13:27:39 PST
(In reply to Saam Barati from comment #6) > Comment on attachment 421132 [details] > Patch > > LGTM. Should we add some of our own tests too that aren’t imported? I think, for now, imported tests cover the things we would like to check. So, I will add tests if issue is found.
Yusuke Suzuki
Comment 8 2021-02-21 13:54:35 PST
Committed r273224 (234410@main): <https://commits.webkit.org/234410@main>
Note You need to log in before you can comment on or make changes to this bug.