Remove redundant UMA metrics for context menus.

Remove redundant UMA metrics for context menus.

These have been superseded by the ContextMenu.SelectedOption
histograms added in https://codereview.chromium.org/1267213002

BUG=483685
Committed: https://crrev.com/1b516c688d740ecdb913408e1d186932cabaa6fd
Cr-Commit-Position: refs/heads/master@{#345924}

[newt (away), 2015-08-25 00:16:38]

newt@chromium.org changed reviewers:
+ dtrainor@chromium.org

[newt (away), 2015-08-25 00:16:39]

PTAL

[newt (away), 2015-08-26 21:20:17]

friendly ping

[David Trainor- moved to gerrit, 2015-08-26 21:42:43]

lgtm

https://codereview.chromium.org/1305893003/diff/1/chrome/android/java/src/org...
File chrome/android/java/src/org/chromium/chrome/browser/tab/ChromeTab.java
(right):

https://codereview.chromium.org/1305893003/diff/1/chrome/android/java/src/org...
chrome/android/java/src/org/chromium/chrome/browser/tab/ChromeTab.java:592: if
(isLink && shouldInterceptContextMenuDownload(url)) {
Could this just be return !isLink || !shouldInterceptContextMenuDownload(url);?

[newt (away), 2015-08-26 21:48:54]

Thanks for the review, neighbor :)

https://codereview.chromium.org/1305893003/diff/1/chrome/android/java/src/org...
File chrome/android/java/src/org/chromium/chrome/browser/tab/ChromeTab.java
(right):

https://codereview.chromium.org/1305893003/diff/1/chrome/android/java/src/org...
chrome/android/java/src/org/chromium/chrome/browser/tab/ChromeTab.java:592: if
(isLink && shouldInterceptContextMenuDownload(url)) {
On 2015/08/26 21:42:43, David Trainor wrote:
> Could this just be return !isLink ||
!shouldInterceptContextMenuDownload(url);?

Yeah, but that's less readable IMO and it's harder to later extend.

[newt (away), 2015-08-26 21:49:56]

newt@chromium.org changed reviewers:
+ asvitkine@chromium.org

[newt (away), 2015-08-26 21:49:56]

+asvitkine for actions.xml

[Alexei Svitkine (slow), 2015-08-27 16:46:30]

lgtm

[newt (away), 2015-08-27 16:56:04]

The CQ bit was checked by newt@chromium.org

[commit-bot: I haz the power, 2015-08-27 16:56:45]

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

[commit-bot: I haz the power, 2015-08-27 16:58:41]

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

[commit-bot: I haz the power, 2015-08-27 16:58:42]

Try jobs failed on following builders:
  ios_dbg_simulator_ninja on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_dbg_simulator...)
  ios_rel_device_ninja on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_rel_device_ni...)
  mac_chromium_compile_dbg_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)
  mac_chromium_rel_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[newt (away), 2015-08-27 17:15:33]

The CQ bit was checked by newt@chromium.org

[newt (away), 2015-08-27 17:15:34]

The patchset sent to the CQ was uploaded after l-g-t-m from
dtrainor@chromium.org, asvitkine@chromium.org
Link to the patchset: https://codereview.chromium.org/1305893003/#ps20001
(title: " ")

[commit-bot: I haz the power, 2015-08-27 17:16:00]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1305893003/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1305893003/20001

[commit-bot: I haz the power, 2015-08-27 18:16:06]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-08-27 18:17:25]

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