Update WindowTree::OnWindowInputEventAck to include handled

To support post-target accelerators we plan to have the client application report whether an input event was handled or not, when ack-ing the event.

This change updates WindowTree::OnWindowInputEventAck to include a boolean parameter |handled|.

PlatformWindowMus will always mark events as handled. CompositorMusConnection will base the handled state on the ack state of the renderer.

TEST=WindowTreeClientImplTest, WindowTreeApptest, WindowTreeTest, CompositorMusConnectionTest
BUG=560478

Review URL: https://codereview.chromium.org/1749323002

Cr-Commit-Position: refs/heads/master@{#379821}

1 files changed, 9 insertions, 5 deletions

diff --git a/components/mus/public/cpp/lib/window_tree_client_impl.cc b/components/mus/public/cpp/lib/window_tree_client_impl.cc
index cdcd7b6..7c0a477 100644
--- a/components/mus/public/cpp/lib/window_tree_client_impl.cc
+++ b/components/mus/public/cpp/lib/window_tree_client_impl.cc
@@ -820,17 +820,21 @@ void WindowTreeClientImpl::OnWindowInputEvent(uint32_t event_id,
                                               mojom::EventPtr event) {
   Window* window = GetWindowById(window_id);
   if (!window || !window->input_event_handler_) {
-    tree_->OnWindowInputEventAck(event_id);
+    tree_->OnWindowInputEventAck(event_id, false);
     return;
   }
 
-  scoped_ptr<base::Closure> ack_callback(
-      new base::Closure(base::Bind(&mojom::WindowTree::OnWindowInputEventAck,
-                                   base::Unretained(tree_), event_id)));
+  scoped_ptr<base::Callback<void(bool)>> ack_callback(
+      new base::Callback<void(bool)>(
+          base::Bind(&mojom::WindowTree::OnWindowInputEventAck,
+                     base::Unretained(tree_), event_id)));
   window->input_event_handler_->OnWindowInputEvent(window, std::move(event),
                                                    &ack_callback);
+
+  // The handler did not take ownership of the callback, so we send the ack,
+  // marking the event as not consumed.
   if (ack_callback)
-    ack_callback->Run();
+    ack_callback->Run(false);
 }
 
 void WindowTreeClientImpl::OnWindowFocused(Id focused_window_id) {