Hide the cursor on the client when it is hidden on the host

remoting/host/chromeos/mouse_cursor_monitor_aura.cc

Index: remoting/host/chromeos/mouse_cursor_monitor_aura.cc
diff --git a/remoting/host/chromeos/mouse_cursor_monitor_aura.cc b/remoting/host/chromeos/mouse_cursor_monitor_aura.cc
index ad48f141bc0cc040193096016d546607ba7618c3..03d71ea6d5a5d988792ac8f87ef484dc37017032 100644
--- a/remoting/host/chromeos/mouse_cursor_monitor_aura.cc
+++ b/remoting/host/chromeos/mouse_cursor_monitor_aura.cc
@@ -53,6 +53,11 @@ void MouseCursorMonitorAura::Capture() {
 void MouseCursorMonitorAura::NotifyCursorChanged(const ui::Cursor& cursor) {
   scoped_ptr<SkBitmap> cursor_bitmap(new SkBitmap());
   gfx::Point cursor_hotspot;
+
+  if (cursor.native_type() == ui::kCursorNone) {
+    return;

[Sergey Ulanov, 2015/01/05 18:00:25]

Should we notify the client that the cursor is hidden in this case (i.e. send an
empty cursor message)?

[Wez, 2015/01/05 20:02:46]

Yes, please do. There is even an EmptyCursorShape() helper function in the
EmptyCursorFilter header. :)

[kelvinp, 2015/01/06 02:22:18]

Whoa, excellent point.
I have taken the step a little further to make sure that the client
always hide the cursor when the host send an empty cursor.
Please take another look. 

+  }
+
   if (!ui::GetCursorBitmap(cursor, cursor_bitmap.get(), &cursor_hotspot)) {
     LOG(ERROR) << "Failed to load bitmap for cursor type:"
                << cursor.native_type();