Move ash specific cursor code to CursorManager.

ui/base/cursor/cursor_loader.h

+// Copyright (c) 2012 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 UI_BASE_CURSOR_CURSOR_LOADER_H_
+#define UI_BASE_CURSOR_CURSOR_LOADER_H_
+
+#include "base/logging.h"
+#include "ui/base/ui_export.h"
+#include "ui/gfx/native_widget_types.h"
+
+namespace ui {
+
+class UI_EXPORT CursorLoader {
+ public:
+  CursorLoader() : device_scale_factor_(0.0f) {}
+  virtual ~CursorLoader() {};
+
+  // Returns the device scale factor used by the loader.
+  float device_scale_factor() const {
+    return device_scale_factor_;
+  }
+
+  // Sets the device scale factor used by the loader.
+  void set_device_scale_factor(float device_scale_factor) {
+    device_scale_factor_ = device_scale_factor;
+  }
+
+  // Creates a cursor from an image resource and puts it in the cursor map.
+  virtual void LoadImageCursor(
+      int id, int resource_id, int hot_x, int hot_y) = 0;
+
+  // Creates an animated cursor from an image resource and puts it in the
+  // cursor map. The image is assumed to be a concatenation of animation frames
+  // from left to right. Also, each frame is assumed to be square
+  // (width == height).
+  // |frame_delay_ms| is the delay between frames in millisecond.
+  virtual void LoadAnimatedCursor(
+      int id, int resource_id, int hot_x, int hot_y, int frame_delay_ms) = 0;
+
+  // Unloads all the cursors.
+  virtual void UnloadAll() = 0;
+
+  // Sets the platform cursor based on the native type of |cursor|.
+  virtual void SetPlatformCursor(gfx::NativeCursor* cursor) = 0;
+
+  // Hides the host cursor.
+  virtual void HideHostCursor() = 0;

[Daniel Erat, 2012/09/07 04:55:39]

Seeing this method in this class is unexpected -- it doesn't seem to have
anything to do with loading cursors.  Is there another platform-specific cursor
utility class where this could go instead?

[mazda, 2012/09/07 19:42:24]

Thank you for the suggestion. I moved the function to ui/base/x/x11_util.{cc,h}.
What do you think?

[Daniel Erat, 2012/09/07 20:26:07]

Thanks, this seems like a better place to me.

+
+  // Creates a CursorLoader.
+  static CursorLoader* Create();
+
+ private:
+  // The device scale factor used by the loader.
+  float device_scale_factor_;
+};
+
+
+}  // namespace ui
+
+#endif  // UI_BASE_CURSOR_CURSOR_LOADER_H_