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

Unified Diff: chrome/browser/ui/views/tabs/window_finder_x11.cc

Issue 2449103004: Refactor WindowFinder definition (Closed)
Patch Set: addressed more of sky's feedback Created 4 years, 1 month 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: chrome/browser/ui/views/tabs/window_finder_x11.cc
diff --git a/chrome/browser/ui/views/tabs/window_finder_x11.cc b/chrome/browser/ui/views/tabs/window_finder_x11.cc
index f8418ff84060126212c8d141486ae3518a718cb9..e360045289f5b0b617fbc223b99c12dd55e7d9a5 100644
--- a/chrome/browser/ui/views/tabs/window_finder_x11.cc
+++ b/chrome/browser/ui/views/tabs/window_finder_x11.cc
@@ -4,6 +4,7 @@
#include "chrome/browser/ui/views/tabs/window_finder.h"
+#include "chrome/browser/ui/views/tabs/window_finder_mus.h"
#include "ui/display/display.h"
#include "ui/display/screen.h"
#include "ui/gfx/geometry/point_conversions.h"
@@ -26,8 +27,11 @@ gfx::Point DIPToPixelPoint(const gfx::Point& dip_point) {
gfx::NativeWindow WindowFinder::GetLocalProcessWindowAtPoint(
const gfx::Point& screen_point,
const std::set<gfx::NativeWindow>& ignore) {
- // The X11 server is the canonical state of what the window stacking order
- // is.
+ gfx::NativeWindow out;
+ if (GetLocalProcessWindowAtPointMus(screen_point, ignore, &out))
+ return out;
+
+ // The X11 server is the canonical state of what the window stacking order is.
views::X11TopmostWindowFinder finder;
return finder.FindLocalProcessWindowAt(DIPToPixelPoint(screen_point), ignore);
}

Powered by Google App Engine
This is Rietveld 408576698