evdev: Move EventConverterOzone ownership from EventFactoryOzone to subclass

ui/events/ozone/event_converter_ozone.h

 // 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.
 
 #ifndef UI_EVENTS_OZONE_EVENT_CONVERTER_OZONE_H_
 #define UI_EVENTS_OZONE_EVENT_CONVERTER_OZONE_H_
 
 #include "base/memory/scoped_ptr.h"
-#include "base/message_loop/message_pump_libevent.h"
 #include "ui/events/events_export.h"
 
 namespace ui {
 class Event;
 
-// In ozone, Chrome reads events from file descriptors created from Linux device
-// drivers. The |MessagePumpLibevent::Watcher| parent class provides the
-// functionality to watch a file descriptor for the arrival of new data and
-// notify its subclasses. Device-specific event converters turn bytes read from
-// the file descriptor into |ui::Event| instances. This class provides the
-// functionality needed in common across all converters: dispatching the
-// |ui::Event| to aura.
-class EVENTS_EXPORT EventConverterOzone
-    : public base::MessagePumpLibevent::Watcher {
+// Ozone implementations can generate input events in whatever way is
+// appropriate for the platform.
+// This class provides the functionality needed in common across all
+// converters: dispatching the |ui::Event| to aura.
+class EVENTS_EXPORT EventConverterOzone {
  public:
   EventConverterOzone();
   virtual ~EventConverterOzone();
 
  protected:
   // Subclasses should use this method to post a task that will dispatch
   // |event| from the UI message loop. This method takes ownership of
   // |event|. |event| will be deleted at the end of the posted task.
-  virtual void DispatchEvent(scoped_ptr<ui::Event> event);
+  static void DispatchEvent(scoped_ptr<ui::Event> event);

[spang, 2014/01/28 22:01:32]

This is looking pretty lonely. All that's left here is DispatchEvent, which is
static. Furthermore, we never reference these objects in common code.

I think this class can go..

[rjkroege, 2014/01/28 23:27:55]

sgtm. Added to EventFactory perhaps?

 
  private:
   DISALLOW_COPY_AND_ASSIGN(EventConverterOzone);
 };
 
 }  // namespace ui
 
 #endif  // UI_EVENTS_OZONE_EVENT_CONVERTER_OZONE_H_