Laser tool blocks events from propagating.

ash/common/palette_delegate.h

Index: ash/common/palette_delegate.h
diff --git a/ash/common/palette_delegate.h b/ash/common/palette_delegate.h
index 8b6be72891504488cd105dfb5db06c016af2d834..1ba03f5764221d348481a1b8e0c36b306f4697dc 100644
--- a/ash/common/palette_delegate.h
+++ b/ash/common/palette_delegate.h
@@ -62,9 +62,8 @@ class PaletteDelegate {
   // Cancels any active partial screenshot session.
   virtual void CancelPartialScreenshot() = 0;
 
-  // Called when the laser pointer has been enabled or disabled.
-  virtual void OnLaserPointerEnabled() = 0;
-  virtual void OnLaserPointerDisabled() = 0;
+  // Enables or disables the laser pointer.
+  virtual void SetLaserPointerState(bool enabled) = 0;

[James Cook, 2016/09/14 17:52:25]

Instead of delegating this call via //ash/common -> //chrome -> //ash it would
be better to go direct from //ash/common -> //ash. One way to do this is to add
it to one of the Wm* classes like WmShell. Then you can change WmShellAura (or
similar) to call the code you need in //ash, and leave WmShellMus as not
implemented.

In general we want to minimize ash <-> chrome communication, as they will be in
different processes in mustash.

optional: Rename the method to SetLaserPointerEnabled() which will make call
sites easier to read.

[sammiequon, 2016/09/14 21:44:08]

Done.

 
  private:
   DISALLOW_ASSIGN(PaletteDelegate);