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

Unified Diff: services/ui/ws/window_finder.cc

Issue 2884463002: Make event-targeting asynchronous in window server. (Closed)
Patch Set: rebase and comments 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
Index: services/ui/ws/window_finder.cc
diff --git a/services/ui/ws/window_finder.cc b/services/ui/ws/window_finder.cc
index 6aa0657581c900c74c51a3c35cb437b0b0d515ab..7583c7ee492a8b483a93e59f6d1120658303d3d9 100644
--- a/services/ui/ws/window_finder.cc
+++ b/services/ui/ws/window_finder.cc
@@ -52,9 +52,9 @@ gfx::Point ConvertPointFromParentToChild(const ServerWindow* child,
return gfx::ToFlooredPoint(location_in_child3.AsPointF());
}
-bool FindDeepestVisibleWindowForEventsImpl(ServerWindow* window,
- const gfx::Point& location,
- DeepestWindow* deepest_window) {
+bool FindDeepestVisibleWindowForLocationImpl(ServerWindow* window,
+ const gfx::Point& location,
+ DeepestWindow* deepest_window) {
// The non-client area takes precedence over descendants, as otherwise the
// user would likely not be able to hit the non-client area as it's common
// for descendants to go into the non-client area.
@@ -90,8 +90,8 @@ bool FindDeepestVisibleWindowForEventsImpl(ServerWindow* window,
continue;
}
- if (FindDeepestVisibleWindowForEventsImpl(child, location_in_child,
- deepest_window)) {
+ if (FindDeepestVisibleWindowForLocationImpl(child, location_in_child,
+ deepest_window)) {
return true;
}
}
@@ -107,10 +107,10 @@ bool FindDeepestVisibleWindowForEventsImpl(ServerWindow* window,
} // namespace
-DeepestWindow FindDeepestVisibleWindowForEvents(ServerWindow* root_window,
- const gfx::Point& location) {
+DeepestWindow FindDeepestVisibleWindowForLocation(ServerWindow* root_window,
+ const gfx::Point& location) {
DeepestWindow result;
- FindDeepestVisibleWindowForEventsImpl(root_window, location, &result);
+ FindDeepestVisibleWindowForLocationImpl(root_window, location, &result);
return result;
}

Powered by Google App Engine
This is Rietveld 408576698