Laser tool blocks events from propagating.

ash/laser/laser_pointer_controller_test_api.h

Index: ash/laser/laser_pointer_controller_test_api.h
diff --git a/ash/laser/laser_pointer_controller_test_api.h b/ash/laser/laser_pointer_controller_test_api.h
new file mode 100644
index 0000000000000000000000000000000000000000..92ef371af7b9d1d801059f1d1f1f073730548914
--- /dev/null
+++ b/ash/laser/laser_pointer_controller_test_api.h
@@ -0,0 +1,38 @@
+// Copyright 2016 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 ASH_LASER_LASER_POINTER_CONTROLLER_TESTAPI_H_
+#define ASH_LASER_LASER_POINTER_CONTROLLER_TESTAPI_H_
+
+#include <memory>
+
+#include "base/macros.h"
+
+namespace ash {
+
+class LaserPointerController;
+class LaserPointerPoints;
+class LaserPointerView;
+
+// An api for testing the LaserPointerController class.
+class LaserPointerControllerTestApi {
+ public:
+  explicit LaserPointerControllerTestApi(
+      std::unique_ptr<LaserPointerController> instance);

[James Cook, 2016/09/14 22:53:38]

Hmm. It's unusual for a test API to own an instance. I would change this to take
a LaserPointerController* and make the test itself own the instance. Sorry I
didn't notice that before.

[sammiequon, 2016/09/16 17:35:26]

Done.

+  ~LaserPointerControllerTestApi();
+
+  void SetEnabled(bool enabled);
+  bool IsShowingLaserPointer();
+  const LaserPointerPoints& laser_points();
+  LaserPointerView* laser_pointer_view();
+
+ private:
+  std::unique_ptr<LaserPointerController> instance_;

[James Cook, 2016/09/14 22:53:38]

and LaserPointerController* instance_ here.

[sammiequon, 2016/09/16 17:35:26]

Done.

+
+  DISALLOW_COPY_AND_ASSIGN(LaserPointerControllerTestApi);
+};
+
+}  // namespace ash
+
+#endif  // ASH_LASER_LASER_POINTER_CONTROLLER_TESTAPI_H_