Add MobileUsingMenuBy* metric actions owner and remove an unused one.

chrome/browser/android/uma_bridge.cc

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/android/uma_bridge.h"
 
 #include <jni.h>
 
 #include "content/public/browser/user_metrics.h"
 #include "jni/UmaBridge_jni.h"
 
 using base::UserMetricsAction;
 using content::RecordAction;
 using content::RecordComputedAction;
 
 static void RecordMenuShow(JNIEnv*, jclass) {
   RecordAction(UserMetricsAction("MobileMenuShow"));
 }
 
 static void RecordUsingMenu(JNIEnv*,
                             jclass,
                             jboolean is_by_hw_button,
                             jboolean is_dragging) {
   if (is_by_hw_button) {
     if (is_dragging) {
-      RecordAction(UserMetricsAction("MobileUsingMenuByHwButtonDragging"));

[Alexei Svitkine (slow), 2014/08/08 15:20:49]

According to the counts dashboard, we have seen samples for this action:

https://uma.googleplex.com/counts

So the new NOTREACHED() statement will be reached. (Unless some other code
changes since then.)

[Kibeom Kim (inactive), 2014/08/08 18:29:15]

I removed HW dragging code recently.

+      NOTREACHED() << "We do not support dragging for hardware menu button.";
     } else {
       RecordAction(UserMetricsAction("MobileUsingMenuByHwButtonTap"));
     }
   } else {
     if (is_dragging) {
       RecordAction(UserMetricsAction("MobileUsingMenuBySwButtonDragging"));
     } else {
       RecordAction(UserMetricsAction("MobileUsingMenuBySwButtonTap"));
     }
   }
 }
 
 namespace chrome {
 namespace android {
 
 // Register native methods
 bool RegisterUmaBridge(JNIEnv* env) {
   return RegisterNativesImpl(env);
 }
 
 }  // namespace android
 }  // namespace chrome