Bug 51745

Summary:

Web Inspector: breakpoint is not disabled when clicking on breakpoints sidebar pane checkbox.

Product:

WebKit

Reporter:

Pavel Podivilov <podivilov>

Component:

Web Inspector (Deprecated)

Assignee:

Pavel Podivilov <podivilov>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

apavlov, bweinstein, joepeck, keishi, loislo, pfeldman, pmuellr, rik, timothy, yurys

Priority:

Version:

528+ (Nightly build)

Hardware:

All

OS:

All

Attachments:

Description	Flags
Patch.	pfeldman: review+

Pavel Podivilov

Reported 2010-12-30 05:36:20 PST

Web Inspector: breakpoint is not disabled when clicking on breakpoints sidebar pane checkbox.

Attachments
Patch. (1.18 KB, patch) 2010-12-30 05:37 PST, Pavel Podivilov	pfeldman: review+	Details Formatted Diff Diff
View All Add attachment proposed patch, testcase, etc.

Pavel Podivilov

Comment 1 2010-12-30 05:37:11 PST

Created attachment 77675 [details] Patch.

Pavel Feldman

Comment 2 2010-12-30 05:59:07 PST

Comment on attachment 77675 [details] Patch. View in context: https://bugs.webkit.org/attachment.cgi?id=77675&action=review > WebCore/inspector/front-end/Breakpoint.js:55 > + this.remove(); It would be great if breakpoint had two states: "resolved" and "unresolved". In that case, setting enabled to false would simply make it unresolved unless backend confirms disabling.

Pavel Podivilov

Comment 3 2010-12-30 07:33:32 PST

Committed r74798: <http://trac.webkit.org/changeset/74798>

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