[webkit-reviews] review granted: [Bug 236271] [iOS] Take adequate process assertion for the SharedWorker process : [Attachment 451406] Patch

[bugzilla-daemon at webkit.org]

Brent Fulgham <bfulgham at webkit.org> has granted Chris Dumez
<cdumez at apple.com>'s request for review:
Bug 236271: [iOS] Take adequate process assertion for the SharedWorker process
https://bugs.webkit.org/show_bug.cgi?id=236271

Attachment 451406: Patch

https://bugs.webkit.org/attachment.cgi?id=451406&action=review




--- Comment #9 from Brent Fulgham <bfulgham at webkit.org> ---
Comment on attachment 451406
  --> https://bugs.webkit.org/attachment.cgi?id=451406
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=451406&action=review

> Source/WebKit/NetworkProcess/SharedWorker/WebSharedWorkerServer.cpp:76
> +	       auto* contextConnection = sharedWorker->contextConnection();

Did SharedWorker always have the correct contextConnection, and we've been
wasting time looking it up?