Palette tool laser prototype.

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

+// 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_points.h"
+#include "ash/common/system/chromeos/palette/tools/laser_pointer_view.h"
+#include "ui/views/pointer_watcher.h"
+
+namespace ash {
+
+// Controller for the laser pointer functionality. Enables/disables laser
+// pointer as well as receives points and passes them off to be rendered.

[jdufault, 2016/08/12 19:57:58]

This needs to use actual strings/labels. See
https://codereview.chromium.org/2235063002/ for an example, but you need to
override CreateView.

[sammiequon, 2016/08/16 17:00:04]

Done.

+class LaserPointerMode : public CommonPaletteTool,
+                         public views::PointerWatcher {
+ public:
+  explicit LaserPointerMode(Delegate* delegate);
+  ~LaserPointerMode() override;
+
+ private:
+  // 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::Lock points_lock_;

[jdufault, 2016/08/12 19:57:58]

Move all data members to the end of the class declaration, immediately before
DISALLOW_COPY_AND_ASSIGN.

[sammiequon, 2016/08/16 17:00:04]

Done.

+  int timer_repeat_count_ = 0;
+
+  gfx::Point current_mouse_location_;
+  LaserPointerPoints laser_points_;
+  std::unique_ptr<LaserPointerView> laser_pointer_view_;
+
+  // PaletteTool:
+  PaletteGroup GetGroup() const override;
+  PaletteToolId GetToolId() const override;
+  void OnEnable() override;
+  void OnDisable() override;
+  gfx::VectorIconId GetActiveTrayIcon() override;
+
+  // CommonPaletteTool overrides.

[jdufault, 2016/08/12 19:57:58]

CommonPaletteTool:

[sammiequon, 2016/08/16 17:00:04]

Done.

+  gfx::VectorIconId GetPaletteIconId() override;
+  // views::PointerWatcher:

[jdufault, 2016/08/12 19:57:58]

Newline above

[sammiequon, 2016/08/16 17:00:04]

Done.

+  void OnPointerEventObserved(const ui::PointerEvent& event,
+                              const gfx::Point& location_in_screen,
+                              views::Widget* target) override;
+
+  void InitializeTimer();
+  void StartTimer();
+  void StopTimer();
+  // Timer callback which adds a point where the mouse was last seen. This takes

[jdufault, 2016/08/12 19:57:58]

Newline above comment.

[jdufault, 2016/08/12 19:57:58]

What about for the second sentence,

  This allows the trail to fade away when the mouse is stationary.

[sammiequon, 2016/08/16 17:00:04]

Done.

[sammiequon, 2016/08/16 17:00:04]

Done.

+  // care of fading away the tail of the laser when the mouse is stationary.
+  void AddStationaryPoint();
+
+  DISALLOW_COPY_AND_ASSIGN(LaserPointerMode);
+};
+
+}  // namespace ash
+
+#endif  // ASH_COMMON_SYSTEM_CHROMEOS_PALETTE_TOOLS_LASER_POINTER_MODE_H_