[webkit-reviews] review granted: [Bug 201761] Add release logging in WebLoaderStrategy to check that a load is going through service worker : [Attachment 378728] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Fri Sep 13 08:25:42 PDT 2019


Chris Dumez <cdumez at apple.com> has granted youenn fablet <youennf at gmail.com>'s
request for review:
Bug 201761: Add release logging in WebLoaderStrategy to check that a load is
going through service worker
https://bugs.webkit.org/show_bug.cgi?id=201761

Attachment 378728: Patch

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




--- Comment #2 from Chris Dumez <cdumez at apple.com> ---
Comment on attachment 378728
  --> https://bugs.webkit.org/attachment.cgi?id=378728
Patch

Can we add RELEASE_LOGGING to scheduleLoadFromNetworkProcess() also? Otherwise,
we cannot tell that the service worker did not handle the request and that we
went to the network. It looks like we only have debug logging at the moment.


More information about the webkit-reviews mailing list