.
Created attachment 458188 [details] Patch
Created attachment 458201 [details] Patch
Comment on attachment 458201 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=458201&action=review r=me > Source/WebCore/inspector/InspectorInstrumentation.cpp:848 > +bool InspectorInstrumentation::shouldInterceptRequestImpl(InstrumentingAgents& instrumentingAgents, ResourceLoader& loader) As far as I can tell, everything we do with the ResourceLoader can be done with a const ref – is there a reason we aren't passing `const ResourceLoader&` throughout instead? > Source/WebCore/inspector/agents/InspectorNetworkAgent.cpp:1121 > + return shouldIntercept(loader.request().url(), Protocol::Network::NetworkStage::Request); Nit: I think we can just `loader.url()`
Created attachment 458285 [details] Patch
Committed r293368 (249968@main): <https://commits.webkit.org/249968@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 458285 [details].
<rdar://problem/92300532>