Bug 128785 - Web Inspector: add context menu item for quickly adding probes
Summary: Web Inspector: add context menu item for quickly adding probes
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: Web Inspector (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Nobody
URL:
Keywords: InRadar
: 66314 (view as bug list)
Depends on:
Blocks: WebReplayUI
  Show dependency tree / graph
 
Reported: 2014-02-13 17:47 PST by BJ Burg
Modified: 2016-12-13 15:37 PST (History)
5 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description BJ Burg 2014-02-13 17:47:18 PST 
It's especially tedious to add a probe to a line that does not already have a breakpoint.

I propose we have "Add Probe Expression" as a context menu item when right clicking in the text editor gutter, and the same for right clicking a breakpoint in the navigation sidebar.

We could also add Option+LeftClick as a shortcut for the text editor gutter which will set up the breakpoint action editor to add a probe action.
Comment 1 Radar WebKit Bug Importer 2014-02-13 17:53:55 PST 
<rdar://problem/16066541>
Comment 2 Timothy Hatcher 2014-02-14 12:14:12 PST 
Add Probe Expression in the context menu sounds good.

I'd like to reserve modifier clicks for a feature where the previous breakpoint config is copied, so option-click would copy the previous setup to the new breakpoint. Which would be useful for probes or other actions, like play sound and auto continue.

Maybe have a checkbox in the popover to set the config as the default breakpoint config.
Comment 3 Brian Burg 2014-12-15 21:05:28 PST 
Would also be great to have context menu to create probe from a highlighted expression in the text editor.
Comment 4 Brian Burg 2014-12-15 21:06:02 PST 
*** Bug 66314 has been marked as a duplicate of this bug. ***