Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2684)

Unified Diff: services/ui/ws/threaded_image_cursors.h

Issue 2916823002: Move Mus into chrome's process when running with --mus.
Patch Set: Undo Screen TLS change, don't use Screen::GetScreen() in Mus. Created 3 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: services/ui/ws/threaded_image_cursors.h
diff --git a/services/ui/ws/threaded_image_cursors.h b/services/ui/ws/threaded_image_cursors.h
new file mode 100644
index 0000000000000000000000000000000000000000..6f73c383904a5846ff404312ad080ac311af522b
--- /dev/null
+++ b/services/ui/ws/threaded_image_cursors.h
@@ -0,0 +1,70 @@
+// Copyright 2017 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 SERVICES_UI_WS_THREADED_IMAGE_CURSORS_H_
+#define SERVICES_UI_WS_THREADED_IMAGE_CURSORS_H_
+
+#include "base/memory/weak_ptr.h"
+#include "ui/base/cursor/image_cursors.h"
+
+namespace base {
+class SingleThreadTaskRunner;
+}
+
+namespace ui {
+class PlatformWindow;
+
+namespace ws {
+
+// Wrapper around ui::ImageCursors, capable of executing its methods
+// asynchronously via the provided task runner.
+class ThreadedImageCursors {
+ public:
+ // |resource_runner| is the task runner for the thread which can be used to
+ // load resources; |image_cursors_weak_ptr| is the object used to perform
+ // cursor operations, and should only be dereferenced on |resource_runner|.
+ ThreadedImageCursors(
+ scoped_refptr<base::SingleThreadTaskRunner>& resource_task_runner,
+ base::WeakPtr<ui::ImageCursors> image_cursors_weak_ptr);
+ ~ThreadedImageCursors();
+
+ // Executes ui::ImageCursors::SetDisplay asynchronously.
+ // Sets the display the cursors are loaded for. |scale_factor| determines the
+ // size of the image to load. Returns true if the cursor image is reloaded.
+ void SetDisplay(const display::Display& display, float scale_factor);
+
+ // Asynchronously sets the size of the mouse cursor icon.
+ void SetCursorSize(CursorSize cursor_size);
+
+ // Asynchronously sets the cursor type and then sets the corresponding
+ // PlatformCursor on the provided |platform_window|.
+ // |platform_window| pointer needs to be valid while this object is alive.
+ void SetCursor(ui::CursorType cursor_type,
+ ui::PlatformWindow* platform_window);
+
+ // Helper method. Sets |platform_cursor| on the |platform_window|.
+ void SetCursorOnPlatformWindow(ui::PlatformCursor platform_cursor,
+ ui::PlatformWindow* platform_window);
+
+ private:
+ // Task runner used for accessing |image_cursors_weak_ptr_.
+ scoped_refptr<base::SingleThreadTaskRunner> resource_task_runner_;
+
+ // Task runner of the Window Server thread (where this object is created and
sky 2017/06/27 19:58:57 In some comments you refer to the 'ui service' (th
mfomitchev 2017/07/11 21:47:00 Done.
+ // destroyed).
+ scoped_refptr<base::SingleThreadTaskRunner> ws_task_runner_;
+
+ // Weak pointer to the actual ui::ImageCursors object used to perform the
+ // cursor operations.
+ base::WeakPtr<ui::ImageCursors> image_cursors_weak_ptr_;
+
+ base::WeakPtrFactory<ThreadedImageCursors> weak_ptr_factory_;
+
+ DISALLOW_COPY_AND_ASSIGN(ThreadedImageCursors);
+};
+
+} // namespace ws
+} // namespace ui
+
+#endif // SERVICES_UI_WS_THREADED_IMAGE_CURSORS_H_

Powered by Google App Engine
This is Rietveld 408576698