[webkit-reviews] review denied: [Bug 38194] Web Inspector: multiple breakpoints shown for multiple instances of same script : [Attachment 54421] patch

[bugzilla-daemon at webkit.org]

Pavel Feldman <pfeldman at chromium.org> has denied Andrey Kosyakov
<caseq at chromium.org>'s request for review:
Bug 38194: Web Inspector: multiple breakpoints shown for multiple instances of
same script
https://bugs.webkit.org/show_bug.cgi?id=38194

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

------- Additional Comments from Pavel Feldman <pfeldman at chromium.org>
After your change, disabling this 'single' breakpoint in the breakpoints
sidebar will only disable one of its 'instances'. I think we should come up
with a more solid approach where scripts are matched based on the url + content
either on the backend or the front-end side.