RESOLVED FIXED 180784
StorageProcess::deleteWebsiteData() should ensure there is a SWServer for the given sessionID 
https://bugs.webkit.org/show_bug.cgi?id=180784
Summary StorageProcess::deleteWebsiteData() should ensure there is a SWServer for the...
Chris Dumez
Reported 2017-12-13 16:40:35 PST
StorageProcess::deleteWebsiteData() should ensure there is a SWServer for the given sessionID instead of not clearing anything when no such SWServer exists.
Attachments
Patch (4.49 KB, patch)
2017-12-13 16:42 PST, Chris Dumez 		no flags
DetailsFormatted DiffDiff
Patch (2.81 KB, patch)
2017-12-13 16:45 PST, Chris Dumez 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Chris Dumez
Comment 1 2017-12-13 16:42:13 PST
Created attachment 329284 [details] Patch
Chris Dumez
Comment 2 2017-12-13 16:45:08 PST
Created attachment 329285 [details] Patch
Chris Dumez
Comment 3 2017-12-14 09:03:46 PST
Work towards <rdar://problem/36034667>.
youenn fablet
Comment 4 2017-12-14 10:51:14 PST
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...
Chris Dumez
Comment 5 2017-12-14 10:52:37 PST
(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.
WebKit Commit Bot
Comment 6 2017-12-14 11:14:00 PST
Comment on attachment 329285 [details] Patch Clearing flags on attachment: 329285 Committed r225914: <https://trac.webkit.org/changeset/225914>
WebKit Commit Bot
Comment 7 2017-12-14 11:14:02 PST
All reviewed patches have been landed. Closing bug.
Radar WebKit Bug Importer
Comment 8 2017-12-14 11:14:32 PST
<rdar://problem/36052966>
Note You need to log in before you can comment on or make changes to this bug.