Use ui::Event instead of mojom::EventPtr in mus::InputEventHandler.

components/mus/public/cpp/input_event_handler.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.
 
 #ifndef COMPONENTS_MUS_PUBLIC_CPP_INPUT_EVENT_HANDLER_H_
 #define COMPONENTS_MUS_PUBLIC_CPP_INPUT_EVENT_HANDLER_H_
 
 #include "base/callback_forward.h"
-#include "components/mus/public/interfaces/input_events.mojom.h"
+#include "ui/events/event.h"

[sky, 2016/03/16 18:12:34]

You should be able to forward declare the event.

[Hadi, 2016/03/16 18:46:44]

Done.

 
 namespace mus {
 
 class Window;
 
 // Responsible for processing input events for mus::Window.
 class InputEventHandler {
  public:
   // The event handler can asynchronously ack the event by taking ownership of
   // the |ack_callback|. The callback takes a bool representing whether the
   // handler has consumed the event. If the handler does not take ownership of
   // the callback, then WindowTreeClientImpl will ack the event as not consumed.
   virtual void OnWindowInputEvent(
       Window* target,
-      mojom::EventPtr event,
+      ui::Event* event,

[sky, 2016/03/16 18:12:34]

const ui::Event&?

[Hadi, 2016/03/16 18:46:43]

Done. This was infected from PlatformWindowDelegate::DispatchEvent(ui::Event*)
which we call in PlatformWindowMus::OnWindowInputEvent(). I updated
OnWindowInputEvent to accept const ui::Event&. I'll update
PlatformWindowDelegate::DispatchEvent to accept const pointers (or maybe just
const objects) in another CL.

       scoped_ptr<base::Callback<void(bool)>>* ack_callback) = 0;
 
  protected:
   virtual ~InputEventHandler() {}
 };
 
 }  // namespace mus
 
 #endif  // COMPONENTS_MUS_PUBLIC_CPP_INPUT_EVENT_HANDLER_H_