[webkit-reviews] review granted: [Bug 93271] Web Inspector: Use and process the actual ScriptId in the protocol EventListener object : [Attachment 164854] Patch

[bugzilla-daemon at webkit.org]

Yury Semikhatsky <yurys at chromium.org> has granted Alexander Pavlov (apavlov)
<apavlov at chromium.org>'s request for review:
Bug 93271: Web Inspector: Use and process the actual ScriptId in the protocol
EventListener object
https://bugs.webkit.org/show_bug.cgi?id=93271

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

------- Additional Comments from Yury Semikhatsky <yurys at chromium.org>
View in context: https://bugs.webkit.org/attachment.cgi?id=164854&action=review


> Source/WebCore/bindings/js/ScriptEventListener.cpp:127
> +    scriptId = funcSourceId == SourceProvider::nullID ? "" :
String::number(funcSourceId);

I wonder when (funcSourceId == SourceProvider::nullID) can happen.