[webkit-reviews] review granted: [Bug 74088] Web Inspector: auto detect source map url. : [Attachment 118571] Patch

[bugzilla-daemon at webkit.org]

Pavel Feldman <pfeldman at chromium.org> has granted Pavel Podivilov
<podivilov at chromium.org>'s request for review:
Bug 74088: Web Inspector: auto detect source map url.
https://bugs.webkit.org/show_bug.cgi?id=74088

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

------- Additional Comments from Pavel Feldman <pfeldman at chromium.org>
View in context: https://bugs.webkit.org/attachment.cgi?id=118571&action=review


>> Source/WebCore/inspector/Inspector.json:1994
>> +			{ "name": "sourceMapURL", "type": "string",
"description": "URL of source map associated with script (if any)." },
> 
> We should not process sourceMapURL on the backend (node.js won't have it).
User should listen to the network and get it from there.

You should at least make this optional.