Chromium Code Reviews| Index: content/browser/android/content_view_core_impl.cc |
| diff --git a/content/browser/android/content_view_core_impl.cc b/content/browser/android/content_view_core_impl.cc |
| index c423adb26a525acca2f5cb89f0401acbcca22de2..b487f7e3ea9d37cf22e8db03aa2c015e458fdc41 100644 |
| --- a/content/browser/android/content_view_core_impl.cc |
| +++ b/content/browser/android/content_view_core_impl.cc |
| @@ -11,6 +11,7 @@ |
| #include "base/command_line.h" |
| #include "base/json/json_writer.h" |
| #include "base/logging.h" |
| +#include "base/metrics/histogram.h" |
| #include "base/strings/utf_string_conversions.h" |
| #include "base/values.h" |
| #include "cc/layers/layer.h" |
| @@ -1620,6 +1621,32 @@ void ContentViewCoreImpl::SetAccessibilityEnabled(JNIEnv* env, jobject obj, |
| } |
| } |
| +void ContentViewCoreImpl::SendDelayedTapUma( |
| + JNIEnv* env, |
| + jobject obj) { |
| + content::RecordAction(content::UserMetricsAction("DelayedGestureTap")); |
|
Rick Byers
2013/11/01 14:45:09
This is OK (I think it gives us the main measure w
bokan
2013/11/01 18:52:44
Yup, makes sense and shouldn't be hard.
|
| +} |
| + |
| +void ContentViewCoreImpl::SendActionAfterDoubleTapUma( |
| + JNIEnv* env, |
| + jobject obj, |
| + int type, |
| + bool hasDelay, |
| + int count) { |
| + // This UMA stat tracks a user's action after a double tap within |
| + // k seconds (where k == 5 currently). This UMA will tell us if |
| + // removing the tap gesture delay will lead to significantly more |
| + // accidental navigations after a double tap. |
| + if(hasDelay) { |
| + UMA_HISTOGRAM_ENUMERATION("UserInput.ActionAfterDoubleTapWithDelay", |
| + type, count); |
| + } else { |
| + UMA_HISTOGRAM_ENUMERATION("UserInput.ActionAfterDoubleTapNoDelay", |
| + type, count); |
| + } |
| + |
| +} |
| + |
| void ContentViewCoreImpl::SendOrientationChangeEventInternal() { |
| RenderWidgetHostViewAndroid* rwhv = GetRenderWidgetHostViewAndroid(); |
| if (rwhv) |