Make the cursor have separate mode for disabled mouse events and invisible.

ui/aura/client/cursor_client.h

Index: ui/aura/client/cursor_client.h
diff --git a/ui/aura/client/cursor_client.h b/ui/aura/client/cursor_client.h
index 3f970b0b5c3939eac14a2040a5f49af6d58d5efc..2ea5a727e7320edbc647293cbc3bfbfa65986d34 100644
--- a/ui/aura/client/cursor_client.h
+++ b/ui/aura/client/cursor_client.h
@@ -18,12 +18,20 @@ class AURA_EXPORT CursorClient {
   // Notes that |window| has requested the change to |cursor|.
   virtual void SetCursor(gfx::NativeCursor cursor) = 0;
 
-  // Changes the visibility of the cursor.
+  // Changes the visibility of the cursor. Mouse events keep being sent even
+  // when the cursor is invisible,
   virtual void ShowCursor(bool show) = 0;
 
   // Gets whether the cursor is visible.
   virtual bool IsCursorVisible() const = 0;
 
+  // Enables or disables the cursor. When cursor is disabled, no mouse events
+  // are sent.
+  virtual void EnableCursor(bool enabled) = 0;

[sky, 2012/12/17 16:35:38]

This class has inconsistently named methods and the names you're adding are
mildly confusing as well. How about one of:

ShowCursor();
HideCursor();
EnableMouseEvents();
DisableMouseEvents();

or:

SetCursorVisible(bool);
SetMouseEventsEnabled(bool);
SetCursorLocked(bool);

Is the plan that disable mouse events implicitly hides?

[mazda, 2012/12/18 02:49:32]

I change the interface to the former ones.
I don't think of use case where mouse events are disabled and cursor is visible.
But separating the mouse events state and visibility makes the implementation
simpler, so I changed that way.

+
+  // Returns true if the cursor is enabled.
+  virtual bool IsCursorEnabled() const = 0;
+
   // Sets the device scale factor of the cursor.
   virtual void SetDeviceScaleFactor(float device_scale_factor) = 0;