Index: ui/events/blink/input_handler_proxy.cc |
diff --git a/ui/events/blink/input_handler_proxy.cc b/ui/events/blink/input_handler_proxy.cc |
index 20b84d1c4936211dbfaafe0c9ffb4c2fbecbbdee..5f17029cb98046378feaba49cf27ce670e216dd1 100644 |
--- a/ui/events/blink/input_handler_proxy.cc |
+++ b/ui/events/blink/input_handler_proxy.cc |
@@ -556,19 +556,20 @@ void InputHandlerProxy::RecordMainThreadScrollingReasons( |
// greater than the sample value. kMainThreadScrollingReasonCount doesn't |
// include the NotScrollingOnMain enum but the histograms do so adding |
// the +1 is necessary. |
- uint32_t mainThreadScrollingReasonEnumMax = |
+ // TODO(dcheng): Fix https://crbug.com/705169 so this isn't needed. |
+ constexpr uint32_t kMainThreadScrollingReasonEnumMax = |
cc::MainThreadScrollingReason::kMainThreadScrollingReasonCount + 1; |
if (reasons == cc::MainThreadScrollingReason::kNotScrollingOnMain) { |
if (device == blink::WebGestureDeviceTouchscreen) { |
UMA_HISTOGRAM_ENUMERATION( |
kGestureHistogramName, |
cc::MainThreadScrollingReason::kNotScrollingOnMain, |
- mainThreadScrollingReasonEnumMax); |
+ kMainThreadScrollingReasonEnumMax); |
} else { |
UMA_HISTOGRAM_ENUMERATION( |
kWheelHistogramName, |
cc::MainThreadScrollingReason::kNotScrollingOnMain, |
- mainThreadScrollingReasonEnumMax); |
+ kMainThreadScrollingReasonEnumMax); |
} |
} |
@@ -587,10 +588,10 @@ void InputHandlerProxy::RecordMainThreadScrollingReasons( |
} |
if (device == blink::WebGestureDeviceTouchscreen) { |
UMA_HISTOGRAM_ENUMERATION(kGestureHistogramName, i + 1, |
- mainThreadScrollingReasonEnumMax); |
+ kMainThreadScrollingReasonEnumMax); |
} else { |
UMA_HISTOGRAM_ENUMERATION(kWheelHistogramName, i + 1, |
- mainThreadScrollingReasonEnumMax); |
+ kMainThreadScrollingReasonEnumMax); |
} |
} |
} |