[webkit-reviews] review denied: [Bug 195266] Web Inspector: Audit: provide a way to get the contents of resources : [Attachment 363713] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Mar 6 22:35:44 PST 2019

Joseph Pecoraro <joepeck at webkit.org> has denied Devin Rousso
<drousso at apple.com>'s request for review:
Bug 195266: Web Inspector: Audit: provide a way to get the contents of

Attachment 363713: Patch


--- Comment #18 from Joseph Pecoraro <joepeck at webkit.org> ---
Comment on attachment 363713
  --> https://bugs.webkit.org/attachment.cgi?id=363713

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

>> Source/WebCore/inspector/InspectorAuditResourcesObject.h:58
>> +	ExceptionOr<Vector<Resource>> getContents(Document&,
> One drawback of having this be one API instead of two is that if you want all
the resources you get an enormous JSON result with the contents of all
resources. In fact, building that JSON might blow up the memory of inspected
> We could have this be two APIs:
>     getResources(query?)
>     getResourceContent(url|id)
> This is also nicer for developers to get the complete list of URLs/MIMETypes
and filter them by RegExp on their own.
> What do you think about this approach?

I'm going to r- while we discuss this aspect, I think this is a better API.

Also maybe we should make these asynchronous, but everyone enjoys the
simplicity of synchronous APIs.

More information about the webkit-reviews mailing list