DevTools: Pause on mouse contextmenu event.

DevTools: Pause on mouse contextmenu event.

R=pfeldman, vsevik

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=183151

[vsevik, 2014-09-30 16:09:01]

lgtm.
context menu could be triggered by special keyboard button so I don't think it
is actually mouse event. we could put in some other category if we have one that
is better. I have no idea however.

[apavlov, 2014-09-30 16:18:56]

apavlov@chromium.org changed reviewers:
+ apavlov@chromium.org

[apavlov, 2014-09-30 16:18:57]

https://codereview.chromium.org/614073002/diff/1/Source/devtools/front_end/so...
File Source/devtools/front_end/sources/BreakpointsSidebarPane.js (right):

https://codereview.chromium.org/614073002/diff/1/Source/devtools/front_end/so...
Source/devtools/front_end/sources/BreakpointsSidebarPane.js:558:
this._createCategory(WebInspector.UIString("Mouse"), ["click", "dblclick",
"mousedown", "mouseup", "mouseover", "mousemove", "mouseout", "mousewheel",
"wheel", "contextmenu"]);
My guess is that the "Control" category will work best here

[aandrey, 2014-09-30 17:22:03]

On 2014/09/30 16:09:01, vsevik wrote:
> lgtm.
> context menu could be triggered by special keyboard button so I don't think it
> is actually mouse event. we could put in some other category if we have one
that
> is better. I have no idea however.

"click" event can be triggered by Enter keyboard button.
I'm inclined to leave it as is.

[aandrey, 2014-10-02 15:30:33]

The CQ bit was checked by aandrey@chromium.org

[commit-bot: I haz the power, 2014-10-02 15:30:53]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/614073002/1

[commit-bot: I haz the power, 2014-10-02 16:56:39]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-10-02 16:56:40]

Try jobs failed on following builders:
  win_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/win_blink_rel/builds/29912)

[aandrey, 2014-10-02 17:37:31]

The CQ bit was checked by aandrey@chromium.org

[commit-bot: I haz the power, 2014-10-02 17:38:25]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/614073002/1

[commit-bot: I haz the power, 2014-10-02 18:13:14]

Committed patchset #1 (id:1) as 183151