Refactors away method implementations in ui::EventTargeter

ui/events/null_event_targeter.h

 // Copyright 2015 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 "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"
 
 namespace ui {
 
 class EVENTS_EXPORT NullEventTargeter : public EventTargeter {

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

I know you didn't add this class, but could you include a brief description of
the class above line 12? It wasn't obvious to me how/when this would ever need
to be used without using codesearch.

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

Done.

  public:
   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);
 };
 
 }  // namespace ui