[webkit-reviews] review granted: [Bug 222635] Service worker loads are not marked as app-bound : [Attachment 422726] Patch

[bugzilla-daemon at webkit.org]

youenn fablet <youennf at gmail.com> has granted katherine_cheney at apple.com's
request for review:
Bug 222635: Service worker loads are not marked as app-bound
https://bugs.webkit.org/show_bug.cgi?id=222635

Attachment 422726: Patch

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




--- Comment #6 from youenn fablet <youennf at gmail.com> ---
Comment on attachment 422726
  --> https://bugs.webkit.org/attachment.cgi?id=422726
Patch

Can you add a test where service worker is serving two pages, one being app
bound and the other being not app bound.
Also maybe do some loads outside of the fetch event handler (async fetch for
instance).