SourceEventType added to LatencyInfo

ui/events/blink/web_input_event_traits.cc

Index: ui/events/blink/web_input_event_traits.cc
diff --git a/ui/events/blink/web_input_event_traits.cc b/ui/events/blink/web_input_event_traits.cc
index 8bfe69cec873c2152b0975558a78380b9e8d76f1..272342661943c6376f2096bbfa5867461c83a108 100644
--- a/ui/events/blink/web_input_event_traits.cc
+++ b/ui/events/blink/web_input_event_traits.cc
@@ -252,4 +252,17 @@ uint32_t WebInputEventTraits::GetUniqueTouchEventId(
   return 0U;
 }
 
+// static
+LatencyInfo WebInputEventTraits::CreateLatencyInfoForWebGestureEvent(
+    WebGestureEvent event) {
+  SourceEventType source_event_type = SourceEventType::UNKOWN;
+  if (event.sourceDevice == blink::WebGestureDevice::WebGestureDeviceTouchpad)
+    source_event_type = SourceEventType::WHEEL;
+  else if (event.sourceDevice ==
+           blink::WebGestureDevice::WebGestureDeviceTouchscreen)

[tdresser, 2016/09/12 14:23:08]

I prefer using {} when the condition is multiline (though this isn't technically
required by the style guide, so your call.

[sahel, 2016/09/12 17:02:26]

Done.

[tdresser, 2016/09/12 18:03:44]

Sorry, this means you need {} on the previous clause as well.

"However, if one part of an if-else statement uses curly braces, the other part
must too:"

https://google.github.io/styleguide/cppguide.html#Conditionals

+    source_event_type = SourceEventType::TOUCH;
+  LatencyInfo latency_info(source_event_type);
+  return latency_info;
+}
+
 }  // namespace ui