Refactors away method implementations in ui::EventTargeter

ui/events/null_event_targeter.h

Index: ui/events/null_event_targeter.h
diff --git a/ui/events/null_event_targeter.h b/ui/events/null_event_targeter.h
index 010c06045ba00232f41868461607244a9d98bb9e..fdd87ae84f195a3f47b5901fdee581008fd35efa 100644
--- a/ui/events/null_event_targeter.h
+++ b/ui/events/null_event_targeter.h
@@ -3,6 +3,7 @@
 // found in the LICENSE file.
 
 #include "base/compiler_specific.h"
+#include "base/macros.h"

[tdanderson, 2015/05/22 18:29:56]

Not sure why you're including macros.h in here, view_targeter.h, and
window_targeter.h.

[varkha, 2015/05/22 22:18:34]

Needed to define DISALLOW_COPY_AND_ASSIGN. It is needed because I have removed
#include "ui/events/event.h" in event_targeter and it was bringing in the
base/macros.h. This is in general spirit of putting includes where they are
necessary in implementation files rather than in headers.

[tdanderson, 2015/05/25 15:00:07]

Acknowledged.

 #include "ui/events/event_targeter.h"
 #include "ui/events/events_export.h"
 
@@ -13,7 +14,10 @@ class EVENTS_EXPORT NullEventTargeter : public EventTargeter {
   NullEventTargeter();
   ~NullEventTargeter() override;
 
+  // EventTargeter:
   EventTarget* FindTargetForEvent(EventTarget* root, Event* event) override;
+  EventTarget* FindNextBestTarget(EventTarget* previous_target,
+                                  Event* event) override;
 
  private:
   DISALLOW_COPY_AND_ASSIGN(NullEventTargeter);