[Devtools] Fixed contextmenu double showing for network requests

[Devtools] Fixed contextmenu double showing for network requests

This patch fixes a regression caused from
https://codereview.chromium.org/2447933002/ where it would try and
show the menu twice.

R=dgozman
BUG=660998
Committed: https://crrev.com/364ca9775ffda6909ea0126343d3c0a7458321f7
Cr-Commit-Position: refs/heads/master@{#428919}

[allada, 2016-11-01 00:00:49]

PTL

[allada, 2016-11-01 00:03:12]

The CQ bit was checked by allada@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-11-01 00:03:39]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[dgozman, 2016-11-01 00:07:47]

lgtm

[dgozman, 2016-11-01 00:08:16]

On 2016/11/01 00:07:47, dgozman wrote:
> lgtm

Did the same happen to other clients of data grid context menu?

[allada, 2016-11-01 00:25:36]

On 2016/11/01 00:08:16, dgozman wrote:
> On 2016/11/01 00:07:47, dgozman wrote:
> > lgtm
> 
> Did the same happen to other clients of data grid context menu?

No, because network is the only one that was trying to use it's own contextmenu.
Which is why I forgot to remove the contextMenu.show() (because before it was
needed to show the new contextmenu, but now it lets the datagrid show it)

[commit-bot: I haz the power, 2016-11-01 01:22:54]

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

[commit-bot: I haz the power, 2016-11-01 01:22:55]

Dry run: Try jobs failed on following builders:
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[allada, 2016-11-01 01:31:38]

The CQ bit was checked by allada@chromium.org

[commit-bot: I haz the power, 2016-11-01 01:32:05]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-01 02:51:05]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-01 02:53:34]

Description was changed from

==========
[Devtools] Fixed contextmenu double showing for network requests

This patch fixes a regression caused from
https://codereview.chromium.org/2447933002/ where it would try and
show the menu twice.

R=dgozman
BUG=660998
==========

to

==========
[Devtools] Fixed contextmenu double showing for network requests

This patch fixes a regression caused from
https://codereview.chromium.org/2447933002/ where it would try and
show the menu twice.

R=dgozman
BUG=660998

Committed: https://crrev.com/364ca9775ffda6909ea0126343d3c0a7458321f7
Cr-Commit-Position: refs/heads/master@{#428919}
==========

[commit-bot: I haz the power, 2016-11-01 02:53:35]

Patchset 1 (id:??) landed as
https://crrev.com/364ca9775ffda6909ea0126343d3c0a7458321f7
Cr-Commit-Position: refs/heads/master@{#428919}