Remove common EventConverterOzone class & move to EventConverterEvdev

ui/events/ozone/event_factory_ozone.cc

 // Copyright (c) 2013 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 "ui/events/ozone/event_factory_ozone.h"
 
 #include "base/command_line.h"
+#include "base/debug/trace_event.h"
 #include "base/message_loop/message_pump_ozone.h"
 #include "base/strings/stringprintf.h"
+#include "ui/events/event.h"
 #include "ui/events/event_switches.h"
 
 namespace ui {
 
+namespace {
+
+void DispatchEventHelper(scoped_ptr<ui::Event> key) {
+  TRACE_EVENT1("ozone", "DispatchEventHelper", "type", key->type());

[rjkroege, 2014/01/29 17:01:43]

it is worth perhaps expanding the text in this trace to make it easier to
understand that this function is the highest stack frame of one particular
event's handling

nit: and perhaps you could add a TRACE_EVENT after the Dispatch itself?

[spang, 2014/01/29 17:58:55]

Agree the name could be clearer. Any ideas?

DispatchEventTask?
The event lasts until the scope closes. So it covers the dispatch already (and
logs two messages).

+  base::MessagePumpOzone::Current()->Dispatch(key.get());
+}
+
+}  // namespace
+
 // static
 EventFactoryOzone* EventFactoryOzone::impl_ = NULL;
 
 EventFactoryOzone::EventFactoryOzone() {}
 
 EventFactoryOzone::~EventFactoryOzone() {}
 
 EventFactoryOzone* EventFactoryOzone::GetInstance() {
   CHECK(impl_) << "No EventFactoryOzone implementation set.";
   return impl_;
 }
 
 void EventFactoryOzone::SetInstance(EventFactoryOzone* impl) { impl_ = impl; }
 
 void EventFactoryOzone::StartProcessingEvents() {}
 
+// static
+void EventFactoryOzone::DispatchEvent(scoped_ptr<ui::Event> event) {
+  base::MessageLoop::current()->PostTask(
+      FROM_HERE, base::Bind(&DispatchEventHelper, base::Passed(&event)));
+}
+
 }  // namespace ui