Do not search for service workers in case of non HTTP main frame navigation loads.
Created attachment 329765 [details] Patch
Created attachment 329766 [details] Patch
Created attachment 329837 [details] Patch
Comment on attachment 329837 [details] Patch Clearing flags on attachment: 329837 Committed r226185: <https://trac.webkit.org/changeset/226185>
All reviewed patches have been landed. Closing bug.
<rdar://problem/36157322>
Comment on attachment 329837 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=329837&action=review > Source/WebCore/loader/DocumentLoader.cpp:1573 > + auto tryLoadingThroughServiceWorker = !frameLoader()->isReloadingFromOrigin() && m_frame->page() && RuntimeEnabledFeatures::sharedFeatures().serviceWorkerEnabled() && SchemeRegistry::canServiceWorkersHandleURLScheme(request.url().protocol().toStringWithoutCopying()); What’s going on with all the weird spaces?
Comment on attachment 329837 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=329837&action=review >> Source/WebCore/loader/DocumentLoader.cpp:1573 >> + auto tryLoadingThroughServiceWorker = !frameLoader()->isReloadingFromOrigin() && m_frame->page() && RuntimeEnabledFeatures::sharedFeatures().serviceWorkerEnabled() && SchemeRegistry::canServiceWorkersHandleURLScheme(request.url().protocol().toStringWithoutCopying()); > > What’s going on with all the weird spaces? Will fix it.
Reopening to attach new patch.
Created attachment 329949 [details] Fixing spaces
Comment on attachment 329949 [details] Fixing spaces Clearing flags on attachment: 329949 Committed r226200: <https://trac.webkit.org/changeset/226200>