WebSWServerConnection should register/unregister itself with the StorageProcess instead of relying on StorageToWebProcessConnection to do so on its behalf. This is less error-prone.
Created attachment 327482 [details] Patch
Created attachment 327483 [details] Patch
Comment on attachment 327483 [details] Patch Looks good. I noticed that we delete the copy constructor but not the assignment operator from WebSWServerConnection; not sure if that’s good. I don’t absolutely love WTF_MAKE_NONCOPYABLE, but one of its benefits is that it deletes both. By the way, I just noticed that the "private:" in Noncopyable.h is no longer needed, now that we use "delete". I should remove that since it would make the macro easier to use.
Comment on attachment 327483 [details] Patch Clearing flags on attachment: 327483 Committed r225118: <https://trac.webkit.org/changeset/225118>
All reviewed patches have been landed. Closing bug.
<rdar://problem/35677379>