Bug 87370

Summary:

Web Inspector: "Deactivate/Activate All Breakpoints" should be disabled when there are no breakpoints

Product:

WebKit

Reporter:

sam <dsam2912>

Component:

Web Inspector (Deprecated)

Assignee:

Nobody <webkit-unassigned>

Status:

RESOLVED WONTFIX

Severity:

Normal

CC:

apavlov, burg, bweinstein, joepeck, keishi, loislo, pfeldman, pmuellr, rik, vivekgalatage, yurys

Priority:

Version:

528+ (Nightly build)

Hardware:

All

OS:

All

Attachments:

Description	Flags
Illustration for problem description.	none

sam

Reported 2012-05-24 04:38:05 PDT

Created attachment 143784 [details] Illustration for problem description. "Deactivate/Activate All Breakpoints" button on Source panel should be disbaled when there are no breakpoints.

Attachments
Illustration for problem description. (63.70 KB, image/png) 2012-05-24 04:38 PDT, sam	no flags	Details
View All Add attachment proposed patch, testcase, etc.

Pavel Feldman

Comment 1 2012-05-24 04:42:15 PDT

Breakpoints can be dynamically added as scripts are parsed and removed as pages are reloaded. Blocking activate / deactivate during these transitions does not make UI more intuitive. I would be oppose to such change.

sam

Comment 2 2012-05-24 05:53:41 PDT

(In reply to comment #1) > Breakpoints can be dynamically added as scripts are parsed and removed as pages are reloaded. Blocking activate / deactivate during these transitions does not make UI more intuitive. I would be oppose to such change. Thanks for your feedback Pavel, I think its a bit of confusing to keep activate / deactivate enabled when there are no breakpoints. Also when breakpoints are getting updated dynamically as you pointed out, the button would become disabled iff there are no breakpoints. I am not very sure of occurrences of the cases where we have breakpoints/no-breakpoints in quick successions. Plz let me know your thoughts.

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