Bug 38194 - Web Inspector: multiple breakpoints shown for multiple instances of same script
Summary: Web Inspector: multiple breakpoints shown for multiple instances of same script
Status: RESOLVED INVALID
Alias: None
Product: WebKit
Classification: Unclassified
Component: Web Inspector (Deprecated) (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-04-27 05:37 PDT by Andrey Kosyakov
Modified: 2014-12-09 10:44 PST (History)
8 users (show)

See Also:


Attachments
patch (1.14 KB, patch)
2010-04-27 08:56 PDT, Andrey Kosyakov
pfeldman: review-
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Andrey Kosyakov 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.
Comment 1 Andrey Kosyakov 2010-04-27 08:56:50 PDT
Created attachment 54421 [details]
patch
Comment 2 Pavel Feldman 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.