[Webkit-unassigned] [Bug 52879] New: Web Inspector: refactor debugger api

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=52879

           Summary: Web Inspector: refactor debugger api
           Product: WebKit
           Version: 528+ (Nightly build)
          Platform: All
        OS/Version: All
            Status: UNCONFIRMED
          Severity: Normal
          Priority: P2
         Component: Web Inspector
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: podivilov at chromium.org
                CC: timothy at apple.com, rik at webkit.org, keishi at webkit.org,
                    pmuellr at yahoo.com, joepeck at webkit.org,
                    pfeldman at chromium.org, yurys at chromium.org,
                    bweinstein at apple.com, apavlov at chromium.org,
                    loislo at chromium.org


The are two problems with setting breakpoints using our API:
1. Clients have to manage the list of sticky breakpoints and constantly call setStickyBreakpoints to sync that list with backend.
2. There is no way to set JavaScript breakpoint by url.


To fix the first problem, we should update sticky breakpoints on backend each time client calls set/remove breakpoint method.
Since we have two independent debugger agents (InspectorDebuggerAgent and InspectorBrowserDebuggerAgent), sticky breakpoints management should be implemented in both.

Client should have to sync all breakpoints just once at the beginning using the following methods:
[domain=Inspector] setAllJavaScriptBreakpoints(breakpoints)
[domain=Inspector] setAllBrowserBreakpoints(breakpoints)


To fix the second problem, JavaScript debugger API should allow setting breakpoint by both url and sourceID.
[domain=Debugger] setJavaScriptBreakpoint(url, lineNumber, condition, enabled, out breakpointId)
[domain=Debugger] setJavaScriptBreakpointBySourceID(sourceID, lineNumber, condition, enabled, out breakpointId)
[domain=Debugger] removeJavaScriptBreakpoint(breakpointId)

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.