Clean up remoting project

remoting/host/event_executor_mac.cc

 // Copyright (c) 2010 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 "remoting/host/event_executor_mac.h"
+#include "remoting/host/event_executor.h"
 
 #include <ApplicationServices/ApplicationServices.h>
 #include <Carbon/Carbon.h>
 
+#include "base/compiler_specific.h"
 #include "base/message_loop.h"
 #include "base/task.h"
 #include "base/mac/scoped_cftyperef.h"

[Lambros, 2011/04/01 15:54:00]

Nit: alphabetize

[dmac, 2011/04/01 21:15:07]

Done.

 #include "remoting/host/capturer.h"
 #include "remoting/protocol/message_decoder.h"
 #include "remoting/proto/internal.pb.h"

[Lambros, 2011/04/01 15:54:00]

Nit: proto before protocol?

[dmac, 2011/04/01 21:15:07]

Done.

 
 namespace remoting {
 
+namespace {
+
 using protocol::MouseEvent;
 using protocol::KeyEvent;
 
+// A class to generate events on Mac.
+class EventExecutorMac : public EventExecutor {
+ public:
+  EventExecutorMac(MessageLoopForUI* message_loop, Capturer* capturer);
+  virtual ~EventExecutorMac() {}
+
+  virtual void InjectKeyEvent(const KeyEvent* event, Task* done) OVERRIDE;
+  virtual void InjectMouseEvent(const MouseEvent* event, Task* done) OVERRIDE;
+
+ private:
+  MessageLoopForUI* message_loop_;
+  Capturer* capturer_;
+  int last_x_, last_y_;
+  int modifiers_, mouse_buttons_;
+
+  DISALLOW_COPY_AND_ASSIGN(EventExecutorMac);
+};
+
 EventExecutorMac::EventExecutorMac(
     MessageLoopForUI* message_loop, Capturer* capturer)
     : message_loop_(message_loop),
       capturer_(capturer), last_x_(0), last_y_(0), modifiers_(0),
       mouse_buttons_(0) {
 }
 
-EventExecutorMac::~EventExecutorMac() {
-}
-
 // Hard-coded mapping from Virtual Key codes to Mac KeySyms.
 // This mapping is only valid if both client and host are using a
 // US English keyboard layout.
 // Because we're passing VK codes on the wire, with no Scancode,
 // "extended" flag, etc, things like distinguishing left & right
 // Shift keys doesn't work.
 //
 // TODO(wez): Replace this with something more closely tied to what
 // WebInputEventFactory does on Linux/GTK, and which respects the
 // host's keyboard layout.
@@ skipping 238 matching lines @@
   if (event->has_wheel_offset_x() && event->has_wheel_offset_y()) {
     // TODO(jamiewalch): Use either CGPostScrollWheelEvent() or
     // CGEventCreateScrollWheelEvent() to inject scroll events.
     NOTIMPLEMENTED() << "No scroll wheel support yet.";
   }
 
   done->Run();
   delete done;
 }
 
-protocol::InputStub* CreateEventExecutor(MessageLoopForUI* message_loop,
-                                         Capturer* capturer) {
+}  // namespace
+
+EventExecutor* EventExecutor::Create(MessageLoopForUI* message_loop,
+                                     Capturer* capturer) {
   return new EventExecutorMac(message_loop, capturer);
 }
 
 }  // namespace remoting