Move ash specific cursor code to CursorManager.

ash/shell.cc

Index: ash/shell.cc
diff --git a/ash/shell.cc b/ash/shell.cc
index f43a1e35d5505160205a72f4878a2466f5d49e9b..1a9c7a935047a5a14de870d326d2c9aa0b395848 100644
--- a/ash/shell.cc
+++ b/ash/shell.cc
@@ -100,6 +100,7 @@
 
 #if defined(OS_CHROMEOS)
 #include "ash/display/output_configurator_animation.h"
+#include "base/chromeos/chromeos_version.h"
 #include "base/message_pump_aurax11.h"
 #include "chromeos/display/output_configurator.h"
 #endif  // defined(OS_CHROMEOS)
@@ -361,6 +362,10 @@ void Shell::Init() {
   aura::Env::GetInstance()->SetEventFilter(env_filter_);
 
   cursor_manager_.set_delegate(this);
+#if defined(OS_CHROMEOS)
+  if (base::chromeos::IsRunningOnChromeOS())
+    cursor_manager_.HideHostCursor();
+#endif
 
   focus_manager_.reset(new aura::FocusManager);
   activation_controller_.reset(
@@ -372,6 +377,9 @@ void Shell::Init() {
   aura::RootWindow* root_window = display_controller_->GetPrimaryRootWindow();
   active_root_window_ = root_window;
 
+  cursor_manager_.SetDeviceScaleFactor(
+      root_window->AsRootWindowHostDelegate()->GetDeviceScaleFactor());
+
 #if !defined(OS_MACOSX)
   nested_dispatcher_controller_.reset(new NestedDispatcherController);
   accelerator_controller_.reset(new AcceleratorController);
@@ -701,7 +709,6 @@ void Shell::InitRootWindowController(
   if (user_action_client_.get())
     aura::client::SetUserActionClient(root_window, user_action_client_.get());
 
-  root_window->SetCursor(ui::kCursorPointer);

[Daniel Erat, 2012/09/06 23:47:15]

Why's it okay to remove this?  CursorManager looks like it starts out with
kCursorNone.

[mazda, 2012/09/07 02:20:57]

I restored the line.
CompoundEventFilter sets the cursor immediately on mouse move, so I thought it's
unnecessary. But when initially_hide_cursor_ is false, removing this line may
show wrong cursor.

   controller->InitLayoutManagers();
 
   // TODO(oshima): Move the instance to RootWindowController when