[webkit-reviews] review granted: [Bug 203388] [iOS] Regression(r251067) WebProcesses with service workers no longer keep their network process alive : [Attachment 381860] Patch

[bugzilla-daemon at webkit.org]

Geoffrey Garen <ggaren at apple.com> has granted Chris Dumez <cdumez at apple.com>'s
request for review:
Bug 203388: [iOS] Regression(r251067) WebProcesses with service workers no
longer keep their network process alive
https://bugs.webkit.org/show_bug.cgi?id=203388

Attachment 381860: Patch

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




--- Comment #4 from Geoffrey Garen <ggaren at apple.com> ---
Comment on attachment 381860
  --> https://bugs.webkit.org/attachment.cgi?id=381860
Patch

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

r=me

I'm surprised that our existing service worker tests don't exercise the case
where you have a service worker and need to do a network load. I wonder if
there's a way to test this.

> Source/WebKit/ChangeLog:14
> +	   However, after r251067, there is no longer a concept of service
worker process per say. Service workers

per se