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

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

Issue 2778943005: Keep root_location to be in pixels and display coordinates in WS. (Closed)
Patch Set: TODO Created 3 years, 7 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
« no previous file with comments | « no previous file | services/ui/ws/display_manager.cc » ('j') | services/ui/ws/event_dispatcher.h » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: services/ui/ws/cursor_location_manager.h
diff --git a/services/ui/ws/cursor_location_manager.h b/services/ui/ws/cursor_location_manager.h
index 6096cdc5e6446f6ac78e51213b25c601694ac528..e4ef049744362a0e0cdc0ec0801f85c1f71acf76 100644
--- a/services/ui/ws/cursor_location_manager.h
+++ b/services/ui/ws/cursor_location_manager.h
@@ -23,7 +23,7 @@ class CursorLocationManager {
~CursorLocationManager();
// Sets the current cursor location to |point|. Atomically writes the location
- // to shared memory.
+ // to shared memory. |point| should be in screen-coord and DIP.
void OnMouseCursorLocationChanged(const gfx::Point& point);
sky 2017/05/25 22:10:19 Please rename to point_in_dips to make that clear.
riajiang 2017/05/25 22:59:23 Renamed to point_in_dip?
// Returns a read-only handle to the shared memory which contains the global
« no previous file with comments | « no previous file | services/ui/ws/display_manager.cc » ('j') | services/ui/ws/event_dispatcher.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698