Palette tool laser prototype.

ash/common/system/chromeos/palette/tools/laser_pointer_mode.h

Index: ash/common/system/chromeos/palette/tools/laser_pointer_mode.h
diff --git a/ash/common/system/chromeos/palette/tools/laser_pointer_mode.h b/ash/common/system/chromeos/palette/tools/laser_pointer_mode.h
new file mode 100644
index 0000000000000000000000000000000000000000..d3a9f6f8c0ef53b997c2124fee1964ef58b19fdb
--- /dev/null
+++ b/ash/common/system/chromeos/palette/tools/laser_pointer_mode.h
@@ -0,0 +1,63 @@
+// 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_COMMON_SYSTEM_CHROMEOS_PALETTE_TOOLS_LASER_POINTER_MODE_H_
+#define ASH_COMMON_SYSTEM_CHROMEOS_PALETTE_TOOLS_LASER_POINTER_MODE_H_
+
+#include "ash/common/system/chromeos/palette/common_palette_tool.h"
+#include "ash/common/system/chromeos/palette/tools/laser_pointer_view.h"
+#include "base/threading/thread_checker.h"
+#include "ui/views/pointer_watcher.h"
+
+namespace ash {
+
+class LaserPointerModeTestApi;

[jdufault, 2016/08/16 19:33:55]

Newline below this. Please also verify this is needed.

[sammiequon, 2016/08/16 23:18:54]

Done.

+// Controller for the laser pointer functionality. Enables/disables laser
+// pointer as well as receives points and passes them off to be rendered.
+class LaserPointerMode : public CommonPaletteTool,
+                         public views::PointerWatcher {
+  friend class LaserPointerModeTestApi;

[jdufault, 2016/08/16 19:33:55]

Move friend class decl to right below private:

[sammiequon, 2016/08/16 23:18:54]

Done.

+
+ public:
+  explicit LaserPointerMode(Delegate* delegate);
+  ~LaserPointerMode() override;
+
+ private:
+  // PaletteTool:
+  PaletteGroup GetGroup() const override;
+  PaletteToolId GetToolId() const override;
+  void OnEnable() override;
+  void OnDisable() override;
+  gfx::VectorIconId GetActiveTrayIcon() override;
+  views::View* CreateView() override;
+
+  // CommonPaletteTool:
+  gfx::VectorIconId GetPaletteIconId() override;
+
+  // views::PointerWatcher:
+  void OnPointerEventObserved(const ui::PointerEvent& event,
+                              const gfx::Point& location_in_screen,
+                              views::Widget* target) override;
+
+  void StopTimer();
+
+  // Timer callback which adds a point where the mouse was last seen. This
+  // allows the trail to fade away when the mouse is stationary.
+  void AddStationaryPoint();
+
+  // Timer which will add a new stationary point when the mouse stops moving.
+  // This will remove points past a certain threshold.
+  std::unique_ptr<base::Timer> timer_;
+  base::ThreadChecker thread_checker_;

[jdufault, 2016/08/16 19:33:55]

I'd move the thread_checker_ decl somewhere else so it doesn't separate timer_
and timer_repeat_count_

[sammiequon, 2016/08/16 23:18:54]

Done.

+  mutable int timer_repeat_count_ = 0;

[jdufault, 2016/08/16 19:33:55]

Remove mutable since this should only be accessed on one thread

[sammiequon, 2016/08/16 23:18:54]

Done.

+
+  gfx::Point current_mouse_location_;
+  std::unique_ptr<LaserPointerView> laser_pointer_view_;
+
+  DISALLOW_COPY_AND_ASSIGN(LaserPointerMode);
+};
+
+}  // namespace ash
+
+#endif  // ASH_COMMON_SYSTEM_CHROMEOS_PALETTE_TOOLS_LASER_POINTER_MODE_H_