[webkit-reviews] review denied: [Bug 69015] Web Inspector: Add support for search in script content. : [Attachment 109659] Patch

[bugzilla-daemon at webkit.org]

Ryosuke Niwa <rniwa at webkit.org> has denied Vsevolod Vlasov
<vsevik at chromium.org>'s request for review:
Bug 69015: Web Inspector: Add support for search in script content.
https://bugs.webkit.org/show_bug.cgi?id=69015

Attachment 109659: Patch
https://bugs.webkit.org/attachment.cgi?id=109659&action=review

------- Additional Comments from Ryosuke Niwa <rniwa at webkit.org>
View in context: https://bugs.webkit.org/attachment.cgi?id=109659&action=review


>> Source/WebCore/inspector/InspectorDebuggerAgent.cpp:324
>> +void InspectorDebuggerAgent::searchInContent(ErrorString* error, const
String& scriptId, const String& query, RefPtr<InspectorArray>* results)
> 
> The parameter type should use PassRefPtr instead of RefPtr. 
[readability/pass_ptr] [5]

We don't use pointers to get values out of a function. Please use a reference
instead. r- because of this.