Bug 27502 – Web Inspector: Add ability to evaluate selection while on break point.

• Status: RESOLVED
• Resolution: FIXED
• Component: Web Inspector (Deprecated)

[Pavel Feldman]

While on a break point, there should be a way to evaluate selected text in the source frame.
There should be a separate request to evaluate text on hover, but it is beyond the scope of this one.

I'd suggest the following user scenario:
1. User hits a breakpoint
2. User selects text he wants to evaluate
3. Presses Meta + Shift + E shortcut (or optionally chooses item in Popup menu?)
4. Result is printed in the console.

[Pavel Feldman]

Created attachment 33177 [details]
patch

[Timothy Hatcher]

We really need a way to change the context menu from JS for the Inspector so we can add items like this for discoverability.

Bug 26881 and bug 22147.

[Timothy Hatcher]

Comment on attachment 33177 [details]
patch

> +        var shortcut = WebInspector.KeyboardShortcut.makeKey(
> +            69 /* 'E' */,
> +            platformSpecificModifier | WebInspector.KeyboardShortcut.Modifiers.Shift);

I think that would be fine and more readable on one line.

Talk to Sam Weinig, he has a patch (maybe landed) that will let us add "evaluate on hover".

[Pavel Feldman]

Sending        WebCore/ChangeLog
Sending        WebCore/inspector/front-end/SourceFrame.js
Transmitting file data ..
Committed revision 46176.