| Summary: | StorageProcess::deleteWebsiteData() should ensure there is a SWServer for the given sessionID | ||||||||
|---|---|---|---|---|---|---|---|---|---|
| Product: | WebKit | Reporter: | Chris Dumez <cdumez> | ||||||
| Component: | Service Workers | Assignee: | Chris Dumez <cdumez> | ||||||
| Status: | RESOLVED FIXED | ||||||||
| Severity: | Normal | CC: | beidson, commit-queue, ggaren, webkit-bug-importer, youennf | ||||||
| Priority: | P2 | Keywords: | InRadar | ||||||
| Version: | WebKit Nightly Build | ||||||||
| Hardware: | Unspecified | ||||||||
| OS: | Unspecified | ||||||||
| See Also: | https://bugs.webkit.org/show_bug.cgi?id=180794 | ||||||||
| Attachments: |
|
||||||||
|
Description
Chris Dumez
2017-12-13 16:40:35 PST
Created attachment 329284 [details]
Patch
Created attachment 329285 [details]
Patch
Work towards <rdar://problem/36034667>. Comment on attachment 329285 [details]
Patch
I guess we could try to write an API test for this one somehow. Probably need to require filling some service worker, hence the need for an HTTP server...
(In reply to youenn fablet from comment #4) > Comment on attachment 329285 [details] > Patch > > I guess we could try to write an API test for this one somehow. Probably > need to require filling some service worker, hence the need for an HTTP > server... I hear Brady is working on allowing API testing of Service Workers. Once his work lands, I'll look into testing this. Comment on attachment 329285 [details] Patch Clearing flags on attachment: 329285 Committed r225914: <https://trac.webkit.org/changeset/225914> All reviewed patches have been landed. Closing bug. |