ozone: evdev: Fix cursorless constructor

ui/events/ozone/evdev/event_factory_evdev.cc

 // Copyright 2014 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/evdev/event_factory_evdev.h"
 
 #include <fcntl.h>
 #include <linux/input.h>
 
 #include "base/debug/trace_event.h"
@@ skipping 112 matching lines @@
 // therefore should be run on a thread where latency is not critical. We
 // run it on the FILE thread.
 void CloseInputDevice(const base::FilePath& path,
                       scoped_ptr<EventConverterEvdev> converter) {
   TRACE_EVENT1("ozone", "CloseInputDevice", "path", path.value());
   converter.reset();
 }
 
 }  // namespace
 
-EventFactoryEvdev::EventFactoryEvdev()
-    : device_manager_(NULL),
-      has_started_processing_events_(false),
-      ui_task_runner_(base::MessageLoopProxy::current()),
-      cursor_(NULL),
-      dispatch_callback_(
-          base::Bind(base::IgnoreResult(&EventFactoryEvdev::DispatchUiEvent),
-                     base::Unretained(this))),
-      weak_ptr_factory_(this) {}
-
 EventFactoryEvdev::EventFactoryEvdev(
     CursorDelegateEvdev* cursor,
     DeviceManager* device_manager)
     : device_manager_(device_manager),
       has_started_processing_events_(false),
       ui_task_runner_(base::MessageLoopProxy::current()),
       cursor_(cursor),
       dispatch_callback_(
           base::Bind(base::IgnoreResult(&EventFactoryEvdev::DispatchUiEvent),
                      base::Unretained(this))),
@@ skipping 70 matching lines @@
     base::WorkerPool::PostTask(
         FROM_HERE,
         base::Bind(&CloseInputDevice, path, base::Passed(&converter)),
         true);
   }
 }
 
 void EventFactoryEvdev::StartProcessingEvents() {
   CHECK(ui_task_runner_->RunsTasksOnCurrentThread());
 
   if (device_manager_ && !has_started_processing_events_) {

[dnicoara, 2014/05/15 20:31:35]

nit: I suppose device_manager_ should be removed from this if statement then.

[sadrul, 2014/05/16 03:11:51]

I am including this in https://codereview.chromium.org/285303004/

     has_started_processing_events_ = true;
     // Scan & monitor devices.
     device_manager_->AddObserver(this);
     device_manager_->ScanDevices(this);
   }
 }
 
 void EventFactoryEvdev::WarpCursorTo(gfx::AcceleratedWidget widget,
                                      const gfx::PointF& location) {
   if (cursor_) {
     cursor_->MoveCursorTo(widget, location);
     MouseEvent mouse_event(ET_MOUSE_MOVED,
                            cursor_->location(),
                            cursor_->location(),
                            modifiers_.GetModifierFlags(),
                            /* changed_button_flags */ 0);
     DispatchEvent(&mouse_event);
   }
 }
 
 }  // namespace ui