mash: Convert OnWindowInputEventAck to use an enum for handled status

components/mus/public/cpp/tests/window_tree_client_impl_unittest.cc

 // 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 "components/mus/public/cpp/lib/window_tree_client_impl.h"
 
 #include <stdint.h>
 
 #include "base/logging.h"
 #include "base/macros.h"
@@ skipping 13 matching lines @@
 #include "mojo/converters/input_events/input_events_type_converters.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "ui/events/event.h"
 #include "ui/events/event_utils.h"
 #include "ui/gfx/geometry/rect.h"
 
 namespace mus {
 
 namespace {
 
-void DoNothingBool(bool result) {}
+void DoNothingWithEventResult(mojom::EventResult result) {}
 
 Id server_id(mus::Window* window) {
   return WindowPrivate(window).server_id();
 }
 
 }  // namespace
 
 mojo::Array<uint8_t> Int32ToPropertyTransportValue(int32_t value) {
   const std::vector<uint8_t> bytes =
       mojo::ConvertTo<std::vector<uint8_t>>(value);
@@ skipping 78 matching lines @@
  public:
   TestInputEventHandler()
       : received_event_(false), should_manually_ack_(false) {}
   ~TestInputEventHandler() override {}
 
   void set_should_manually_ack() { should_manually_ack_ = true; }
 
   void AckEvent() {
     DCHECK(should_manually_ack_);
     DCHECK(!ack_callback_.is_null());
-    ack_callback_.Run(true);
-    ack_callback_ = base::Bind(&DoNothingBool);
+    ack_callback_.Run(mojom::EventResult::HANDLED);
+    ack_callback_ = base::Bind(&DoNothingWithEventResult);
   }
 
   void Reset() {
     received_event_ = false;
-    ack_callback_ = base::Bind(&DoNothingBool);
+    ack_callback_ = base::Bind(&DoNothingWithEventResult);
   }
   bool received_event() const { return received_event_; }
 
  private:
   // InputEventHandler:
-  void OnWindowInputEvent(
-      Window* target,
-      const ui::Event& event,
-      scoped_ptr<base::Callback<void(bool)>>* ack_callback) override {
+  void OnWindowInputEvent(Window* target,
+                          const ui::Event& event,
+                          scoped_ptr<base::Callback<void(mojom::EventResult)>>*
+                              ack_callback) override {
     EXPECT_FALSE(received_event_)
         << "Observer was not reset after receiving event.";
     received_event_ = true;
     if (should_manually_ack_) {
       ack_callback_ = *ack_callback->get();
       ack_callback->reset();
     }
   }
 
   bool received_event_;
   bool should_manually_ack_;
-  base::Callback<void(bool)> ack_callback_;
+  base::Callback<void(mojom::EventResult)> ack_callback_;
 
   DISALLOW_COPY_AND_ASSIGN(TestInputEventHandler);
 };
 
 using WindowTreeClientImplTest = testing::Test;
 
 // Verifies bounds are reverted if the server replied that the change failed.
 TEST_F(WindowTreeClientImplTest, SetBoundsFailed) {
   WindowTreeSetup setup;
   Window* root = setup.GetFirstRoot();
@@ skipping 749 matching lines @@
 
   setup.window_tree_client()->OnChangeCompleted(change_id2, false);
   EXPECT_FALSE(child->HasCapture());
   EXPECT_TRUE(root->HasCapture());
 
   setup.window_tree_client()->OnLostCapture(server_id(root));
   EXPECT_FALSE(root->HasCapture());
 }
 
 }  // namespace mus