Collapse the API surface on WebInputEvent via accessor functions.

content/renderer/input/render_widget_input_handler.cc

 // Copyright 2015 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 "content/renderer/input/render_widget_input_handler.h"
 
 #include <stddef.h>
 #include <stdint.h>
 #include <utility>
 
@@ skipping 39 matching lines @@
 namespace content {
 
 namespace {
 
 int64_t GetEventLatencyMicros(double event_timestamp, base::TimeTicks now) {
   return (now - base::TimeDelta::FromSecondsD(event_timestamp))
       .ToInternalValue();
 }
 
 void LogInputEventLatencyUma(const WebInputEvent& event, base::TimeTicks now) {
-  WebInputEvent::Type event_type = event.type;
+  WebInputEvent::Type event_type = event.type();
   UMA_HISTOGRAM_CUSTOM_COUNTS(
       "Event.AggregatedLatency.Renderer2",
-      GetEventLatencyMicros(event.timeStampSeconds, now), 1, 10000000, 100);
+      GetEventLatencyMicros(event.timeStampSeconds(), now), 1, 10000000, 100);
 
-#define CASE_TYPE(t)                                                           \
-  case WebInputEvent::t:                                                       \
-    UMA_HISTOGRAM_CUSTOM_COUNTS(                                               \
-        "Event.Latency.Renderer2." #t,                                         \
-        GetEventLatencyMicros(event.timeStampSeconds, now), 1, 10000000, 100); \
+#define CASE_TYPE(t)                                                       \
+  case WebInputEvent::t:                                                   \
+    UMA_HISTOGRAM_CUSTOM_COUNTS(                                           \
+        "Event.Latency.Renderer2." #t,                                     \
+        GetEventLatencyMicros(event.timeStampSeconds(), now), 1, 10000000, \
+        100);                                                              \
     break;
 
   switch (event_type) {
     CASE_TYPE(Undefined);
     CASE_TYPE(MouseDown);
     CASE_TYPE(MouseUp);
     CASE_TYPE(MouseMove);
     CASE_TYPE(MouseEnter);
     CASE_TYPE(MouseLeave);
     CASE_TYPE(ContextMenu);
@@ skipping 123 matching lines @@
 
 RenderWidgetInputHandler::~RenderWidgetInputHandler() {}
 
 void RenderWidgetInputHandler::HandleInputEvent(
     const WebInputEvent& input_event,
     const ui::LatencyInfo& latency_info,
     InputEventDispatchType dispatch_type) {
   base::AutoReset<bool> handling_input_event_resetter(&handling_input_event_,
                                                       true);
   base::AutoReset<WebInputEvent::Type> handling_event_type_resetter(
-      &handling_event_type_, input_event.type);
+      &handling_event_type_, input_event.type());
 
   // Calls into |didOverscroll()| while handling this event will populate
   // |event_overscroll|, which in turn will be bundled with the event ack.
   std::unique_ptr<DidOverscrollParams> event_overscroll;
   base::AutoReset<std::unique_ptr<DidOverscrollParams>*>
       handling_event_overscroll_resetter(&handling_event_overscroll_,
                                          &event_overscroll);
 
 #if defined(OS_ANDROID)
   ImeEventGuard guard(widget_);
 #endif
 
   base::TimeTicks start_time;
   if (base::TimeTicks::IsHighResolution())
     start_time = base::TimeTicks::Now();
 
   TRACE_EVENT1("renderer,benchmark,rail",
                "RenderWidgetInputHandler::OnHandleInputEvent", "event",
-               WebInputEvent::GetName(input_event.type));
+               WebInputEvent::GetName(input_event.type()));
   TRACE_EVENT_SYNTHETIC_DELAY_BEGIN("blink.HandleInputEvent");
   TRACE_EVENT_WITH_FLOW1("input,benchmark", "LatencyInfo.Flow",
                          TRACE_ID_DONT_MANGLE(latency_info.trace_id()),
                          TRACE_EVENT_FLAG_FLOW_IN | TRACE_EVENT_FLAG_FLOW_OUT,
                          "step", "HandleInputEventMain");
 
   // If we don't have a high res timer, these metrics won't be accurate enough
   // to be worth collecting. Note that this does introduce some sampling bias.
   if (!start_time.is_null())
     LogInputEventLatencyUma(input_event, start_time);
 
   std::unique_ptr<cc::SwapPromiseMonitor> latency_info_swap_promise_monitor;
   ui::LatencyInfo swap_latency_info(latency_info);
   swap_latency_info.AddLatencyNumber(
       ui::LatencyComponentType::INPUT_EVENT_LATENCY_RENDERER_MAIN_COMPONENT, 0,
       0);
   if (widget_->compositor()) {
     latency_info_swap_promise_monitor =
         widget_->compositor()->CreateLatencyInfoSwapPromiseMonitor(
             &swap_latency_info);
   }
 
   bool prevent_default = false;
-  if (WebInputEvent::isMouseEventType(input_event.type)) {
+  if (WebInputEvent::isMouseEventType(input_event.type())) {
     const WebMouseEvent& mouse_event =
         static_cast<const WebMouseEvent&>(input_event);
     TRACE_EVENT2("renderer", "HandleMouseMove", "x", mouse_event.x, "y",
                  mouse_event.y);
     context_menu_source_type_ = ui::MENU_SOURCE_MOUSE;
     prevent_default = delegate_->WillHandleMouseEvent(mouse_event);
   }
 
-  if (WebInputEvent::isKeyboardEventType(input_event.type)) {
+  if (WebInputEvent::isKeyboardEventType(input_event.type())) {
     context_menu_source_type_ = ui::MENU_SOURCE_KEYBOARD;
 #if defined(OS_ANDROID)
     // The DPAD_CENTER key on Android has a dual semantic: (1) in the general
     // case it should behave like a select key (i.e. causing a click if a button
     // is focused). However, if a text field is focused (2), its intended
     // behavior is to just show the IME and don't propagate the key.
     // A typical use case is a web form: the DPAD_CENTER should bring up the IME
     // when clicked on an input text field and cause the form submit if clicked
     // when the submit button is focused, but not vice-versa.
     // The UI layer takes care of translating DPAD_CENTER into a RETURN key,
     // but at this point we have to swallow the event for the scenario (2).
     const WebKeyboardEvent& key_event =
         static_cast<const WebKeyboardEvent&>(input_event);
     if (key_event.nativeKeyCode == AKEYCODE_DPAD_CENTER &&
         widget_->GetTextInputType() != ui::TEXT_INPUT_TYPE_NONE) {
       widget_->showImeIfNeeded();
       prevent_default = true;
     }
 #endif
   }
 
-  if (WebInputEvent::isGestureEventType(input_event.type)) {
+  if (WebInputEvent::isGestureEventType(input_event.type())) {
     const WebGestureEvent& gesture_event =
         static_cast<const WebGestureEvent&>(input_event);
-    if (input_event.type == WebInputEvent::GestureLongPress) {
+    if (input_event.type() == WebInputEvent::GestureLongPress) {
       context_menu_source_type_ = ui::MENU_SOURCE_LONG_PRESS;
-    } else if (input_event.type == WebInputEvent::GestureLongTap) {
+    } else if (input_event.type() == WebInputEvent::GestureLongTap) {
       context_menu_source_type_ = ui::MENU_SOURCE_LONG_TAP;
     } else {
       context_menu_source_type_ = ui::MENU_SOURCE_TOUCH;
     }
     prevent_default =
         prevent_default || delegate_->WillHandleGestureEvent(gesture_event);
   }
 
   WebInputEventResult processed = prevent_default
                                       ? WebInputEventResult::HandledSuppressed
                                       : WebInputEventResult::NotHandled;
-  if (input_event.type != WebInputEvent::Char || !suppress_next_char_events_) {
+  if (input_event.type() != WebInputEvent::Char ||
+      !suppress_next_char_events_) {
     suppress_next_char_events_ = false;
     if (processed == WebInputEventResult::NotHandled && widget_->GetWebWidget())
       processed = widget_->GetWebWidget()->handleInputEvent(input_event);
   }
 
   // TODO(dtapuska): Use the input_event.timeStampSeconds as the start
   // ideally this should be when the event was sent by the compositor to the
   // renderer. crbug.com/565348
-  if (input_event.type == WebInputEvent::TouchStart ||
-      input_event.type == WebInputEvent::TouchMove ||
-      input_event.type == WebInputEvent::TouchEnd) {
+  if (input_event.type() == WebInputEvent::TouchStart ||
+      input_event.type() == WebInputEvent::TouchMove ||
+      input_event.type() == WebInputEvent::TouchEnd) {
     const WebTouchEvent& touch = static_cast<const WebTouchEvent&>(input_event);
 
     LogPassiveEventListenersUma(processed, touch.dispatchType,
-                                input_event.timeStampSeconds, latency_info);
+                                input_event.timeStampSeconds(), latency_info);
 
     // TODO(lanwei): Remove this metric for event latency outside fling in M56,
     // once we've gathered enough data to decide if we want to ship the passive
     // event listener for fling, see https://crbug.com/638661.
     if (touch.dispatchType == WebInputEvent::Blocking &&
         touch.touchStartOrFirstTouchMove &&
         base::TimeTicks::IsHighResolution()) {
       base::TimeTicks now = base::TimeTicks::Now();
       UMA_HISTOGRAM_CUSTOM_COUNTS(
           "Event.Touch.TouchLatencyOutsideFling",
-          GetEventLatencyMicros(input_event.timeStampSeconds, now), 1,
+          GetEventLatencyMicros(input_event.timeStampSeconds(), now), 1,
           100000000, 50);
     }
-  } else if (input_event.type == WebInputEvent::MouseWheel) {
+  } else if (input_event.type() == WebInputEvent::MouseWheel) {
     LogPassiveEventListenersUma(
         processed,
         static_cast<const WebMouseWheelEvent&>(input_event).dispatchType,
-        input_event.timeStampSeconds, latency_info);
+        input_event.timeStampSeconds(), latency_info);
   }
 
   // If this RawKeyDown event corresponds to a browser keyboard shortcut and
   // it's not processed by webkit, then we need to suppress the upcoming Char
   // events.
   bool is_keyboard_shortcut =
-      input_event.type == WebInputEvent::RawKeyDown &&
+      input_event.type() == WebInputEvent::RawKeyDown &&
       static_cast<const WebKeyboardEvent&>(input_event).isBrowserShortcut;
   if (processed == WebInputEventResult::NotHandled && is_keyboard_shortcut)
     suppress_next_char_events_ = true;
 
   InputEventAckState ack_result = processed == WebInputEventResult::NotHandled
                                       ? INPUT_EVENT_ACK_STATE_NOT_CONSUMED
                                       : INPUT_EVENT_ACK_STATE_CONSUMED;
   if (processed == WebInputEventResult::NotHandled &&
-      input_event.type == WebInputEvent::TouchStart) {
+      input_event.type() == WebInputEvent::TouchStart) {
     const WebTouchEvent& touch_event =
         static_cast<const WebTouchEvent&>(input_event);
     // Hit-test for all the pressed touch points. If there is a touch-handler
     // for any of the touch points, then the renderer should continue to receive
     // touch events.
     ack_result = INPUT_EVENT_ACK_STATE_NO_CONSUMER_EXISTS;
     for (size_t i = 0; i < touch_event.touchesLength; ++i) {
       if (touch_event.touches[i].state == WebTouchPoint::StatePressed &&
           delegate_->HasTouchEventHandlersAt(
               gfx::ToFlooredPoint(touch_event.touches[i].position))) {
         ack_result = INPUT_EVENT_ACK_STATE_NOT_CONSUMED;
         break;
       }
     }
   }
 
   // Send gesture scroll events and their dispositions to the compositor thread,
   // so that they can be used to produce the elastic overscroll effect on Mac.
-  if (input_event.type == WebInputEvent::GestureScrollBegin ||
-      input_event.type == WebInputEvent::GestureScrollEnd ||
-      input_event.type == WebInputEvent::GestureScrollUpdate) {
+  if (input_event.type() == WebInputEvent::GestureScrollBegin ||
+      input_event.type() == WebInputEvent::GestureScrollEnd ||
+      input_event.type() == WebInputEvent::GestureScrollUpdate) {
     const WebGestureEvent& gesture_event =
         static_cast<const WebGestureEvent&>(input_event);
     if (gesture_event.sourceDevice == blink::WebGestureDeviceTouchpad) {
       delegate_->ObserveGestureEventAndResult(
           gesture_event,
           event_overscroll ? event_overscroll->latest_overscroll_delta
                            : gfx::Vector2dF(),
           processed != WebInputEventResult::NotHandled);
     }
   }
 
   TRACE_EVENT_SYNTHETIC_DELAY_END("blink.HandleInputEvent");
 
   if (dispatch_type == DISPATCH_TYPE_BLOCKING_NOTIFY_MAIN ||
       dispatch_type == DISPATCH_TYPE_NON_BLOCKING_NOTIFY_MAIN) {
     // |non_blocking| means it was ack'd already by the InputHandlerProxy
     // so let the delegate know the event has been handled.
-    delegate_->NotifyInputEventHandled(input_event.type, ack_result);
+    delegate_->NotifyInputEventHandled(input_event.type(), ack_result);
   }
 
   if ((dispatch_type == DISPATCH_TYPE_BLOCKING ||
        dispatch_type == DISPATCH_TYPE_BLOCKING_NOTIFY_MAIN)) {
     std::unique_ptr<InputEventAck> response(new InputEventAck(
-        InputEventAckSource::MAIN_THREAD, input_event.type, ack_result,
+        InputEventAckSource::MAIN_THREAD, input_event.type(), ack_result,
         swap_latency_info, std::move(event_overscroll),
         ui::WebInputEventTraits::GetUniqueTouchEventId(input_event)));
     delegate_->OnInputEventAck(std::move(response));
   } else {
     DCHECK(!event_overscroll) << "Unexpected overscroll for un-acked event";
   }
   if (RenderThreadImpl::current()) {
     RenderThreadImpl::current()
         ->GetRendererScheduler()
         ->DidHandleInputEventOnMainThread(input_event);
   }
 
 #if defined(OS_ANDROID)
   // Allow the IME to be shown when the focus changes as a consequence
   // of a processed touch end event.
-  if (input_event.type == WebInputEvent::TouchEnd &&
+  if (input_event.type() == WebInputEvent::TouchEnd &&
       processed != WebInputEventResult::NotHandled) {
     delegate_->UpdateTextInputState(ShowIme::IF_NEEDED,
                                     ChangeSource::FROM_NON_IME);
   }
 #elif defined(USE_AURA)
   // Show the virtual keyboard if enabled and a user gesture triggers a focus
   // change.
   if (processed != WebInputEventResult::NotHandled &&
-      (input_event.type == WebInputEvent::TouchEnd ||
-       input_event.type == WebInputEvent::MouseUp)) {
+      (input_event.type() == WebInputEvent::TouchEnd ||
+       input_event.type() == WebInputEvent::MouseUp)) {
     delegate_->UpdateTextInputState(ShowIme::IF_NEEDED, ChangeSource::FROM_IME);
   }
 #endif
 
-  if (!prevent_default && WebInputEvent::isKeyboardEventType(input_event.type))
+  if (!prevent_default &&
+      WebInputEvent::isKeyboardEventType(input_event.type()))
     delegate_->OnDidHandleKeyEvent();
 
 // TODO(rouslan): Fix ChromeOS and Windows 8 behavior of autofill popup with
 // virtual keyboard.
 #if !defined(OS_ANDROID)
   // Virtual keyboard is not supported, so react to focus change immediately.
   if (processed != WebInputEventResult::NotHandled &&
-      (input_event.type == WebInputEvent::TouchEnd ||
-       input_event.type == WebInputEvent::MouseUp)) {
+      (input_event.type() == WebInputEvent::TouchEnd ||
+       input_event.type() == WebInputEvent::MouseUp)) {
     delegate_->FocusChangeComplete();
   }
 #endif
 }
 
 void RenderWidgetInputHandler::DidOverscrollFromBlink(
     const WebFloatSize& overscrollDelta,
     const WebFloatSize& accumulatedOverscroll,
     const WebFloatPoint& position,
     const WebFloatSize& velocity) {
@@ skipping 10 matching lines @@
   // it can be bundled in the event ack.
   if (handling_event_overscroll_) {
     *handling_event_overscroll_ = std::move(params);
     return;
   }
 
   delegate_->OnDidOverscroll(*params);
 }
 
 }  // namespace content