Bug 38194

Summary:

Web Inspector: multiple breakpoints shown for multiple instances of same script

Product:

WebKit

Reporter:

Andrey Kosyakov <caseq>

Component:

Web Inspector (Deprecated)

Assignee:

Nobody <webkit-unassigned>

Status:

RESOLVED INVALID

Severity:

Normal

CC:

burg, bweinstein, joepeck, keishi, pfeldman, pmuellr, rik, yurys

Priority:

Version:

528+ (Nightly build)

Hardware:

All

OS:

All

Attachments:

Description	Flags
patch	pfeldman: review-

Andrey Kosyakov

Reported 2010-04-27 05:37:14 PDT

1. Load a page that contains multiple instances of the same script (e.g. within iframes) 2. Set a breakpoint. 3. Observe one breakpoint on breakpoint sidebar. 4. Reload page 5. Observe multiple breakpoints This is counter-intuitive -- if user sets breakpoint once, it should be shown once, and we do not distinguish (at least, visually) the instances of breakpoints.

Attachments
patch (1.14 KB, patch) 2010-04-27 08:56 PDT, Andrey Kosyakov	pfeldman: review-	Details Formatted Diff Diff
View All Add attachment proposed patch, testcase, etc.

Andrey Kosyakov

Comment 1 2010-04-27 08:56:50 PDT

Created attachment 54421 [details] patch

Pavel Feldman

Comment 2 2010-05-02 05:28:52 PDT

Comment on attachment 54421 [details] patch 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.

Note You need to log in before you can comment on or make changes to this bug.