Merge the mojo blink type converter into ui/events/blink

content/renderer/mus/compositor_mus_connection.cc

Index: content/renderer/mus/compositor_mus_connection.cc
diff --git a/content/renderer/mus/compositor_mus_connection.cc b/content/renderer/mus/compositor_mus_connection.cc
index 6c3d2783abee1bdb204bcc1711be6b002c5bab08..e96ce9ba60e92a5f25f0593305d0d60e3a71d92c 100644
--- a/content/renderer/mus/compositor_mus_connection.cc
+++ b/content/renderer/mus/compositor_mus_connection.cc
@@ -7,8 +7,8 @@
 #include "base/single_thread_task_runner.h"
 #include "content/renderer/input/input_handler_manager.h"
 #include "content/renderer/mus/render_widget_mus_connection.h"
-#include "mojo/converters/blink/blink_input_events_type_converters.h"
 #include "ui/events/blink/did_overscroll_params.h"
+#include "ui/events/blink/web_input_event.h"
 #include "ui/events/blink/web_input_event_traits.h"
 #include "ui/events/latency_info.h"
 #include "ui/events/mojo/event.mojom.h"
@@ -19,6 +19,33 @@ namespace {
 
 void DoNothingWithEventResult(EventResult result) {}
 
+gfx::Point GetScreenLocationFromEvent(const ui::LocatedEvent& event) {
+  return event.root_location();
+}
+
+std::unique_ptr<blink::WebInputEvent> Convert(const ui::Event& event) {
+  if (event.IsMousePointerEvent()) {
+    const ui::MouseEvent mouse_event(*event.AsPointerEvent());
+    blink::WebMouseEvent blink_event = ui::MakeWebMouseEvent(
+        mouse_event, base::Bind(&GetScreenLocationFromEvent));
+    return base::WrapUnique(new blink::WebMouseEvent(blink_event));
+  } else if (event.IsTouchPointerEvent()) {
+    blink::WebTouchEvent blink_event = ui::MakeWebTouchEvent(
+        *event.AsPointerEvent(), base::Bind(&GetScreenLocationFromEvent));
+    return base::WrapUnique(new blink::WebTouchEvent(blink_event));
+  } else if (event.IsMouseWheelEvent()) {
+    blink::WebMouseWheelEvent blink_event = ui::MakeWebMouseWheelEvent(
+        *event.AsMouseWheelEvent(), base::Bind(&GetScreenLocationFromEvent));
+    return base::WrapUnique(new blink::WebMouseWheelEvent(blink_event));
+  } else if (event.IsKeyEvent()) {
+    blink::WebKeyboardEvent blink_event =
+        ui::MakeWebKeyboardEvent(*event.AsKeyEvent());
+    return base::WrapUnique(new blink::WebKeyboardEvent(blink_event));
+  } else {
+    return nullptr;
+  }
+}
+
 }  // namespace
 
 namespace content {
@@ -125,9 +152,7 @@ void CompositorMusConnection::OnWindowInputEvent(
     const ui::Event& event,
     std::unique_ptr<base::Callback<void(EventResult)>>* ack_callback) {
   DCHECK(compositor_task_runner_->BelongsToCurrentThread());
-  std::unique_ptr<blink::WebInputEvent> web_event(
-      mojo::TypeConverter<std::unique_ptr<blink::WebInputEvent>,
-                          ui::Event>::Convert(event));
+  std::unique_ptr<blink::WebInputEvent> web_event(Convert(event));

[sadrul, 2016/08/24 18:05:36]

Can the WebInputEvent be created on the stack instead?  (if a valid event can't
be created, the event-type should be Undefined)

[jonross, 2016/08/24 21:42:04]

The unique_ptr is used here to preserve and pass the event around through
delayed tasks.
  - line 181
  - later on render_widget_mus_connection

If we are fine copying the WebInputEvent we could declare it on the stack.

Thoughts?

[sadrul, 2016/08/24 21:47:14]

Copying should be OK, yeah.

[jonross, 2016/08/25 15:11:59]

As discussed offline, differing object sizes led to incorrect copying. Staying
with unique_ptr

   // TODO(sad): We probably need to plumb LatencyInfo through Mus.
   ui::LatencyInfo info;
   InputEventAckState ack_state = input_handler_manager_->HandleInputEvent(