Find display root ServerWindow for located events in mus.

services/ui/ws/window_manager_state.cc

 // Copyright 2016 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.
 
 #include "services/ui/ws/window_manager_state.h"
 
+#include <utility>
+
 #include "base/memory/weak_ptr.h"
 #include "services/shell/public/interfaces/connector.mojom.h"
 #include "services/ui/common/event_matcher_util.h"
 #include "services/ui/ws/accelerator.h"
 #include "services/ui/ws/display_manager.h"
 #include "services/ui/ws/platform_display.h"
 #include "services/ui/ws/server_window.h"
 #include "services/ui/ws/user_display_manager.h"
 #include "services/ui/ws/user_id_tracker.h"
 #include "services/ui/ws/window_manager_display_root.h"
@@ skipping 498 matching lines @@
   }
 
   if (!tree) {
     DCHECK(in_nonclient_area);
     tree = window_tree_;
   }
   return tree->id();
 }
 
 ServerWindow* WindowManagerState::GetRootWindowContaining(
-    const gfx::Point& location) {
-  if (display_manager()->displays().empty())
+    gfx::Point* location) {
+  Display* target_display = nullptr;
+  for (Display* display : display_manager()->displays()) {
+    if (display->platform_display()->GetBounds().Contains(*location)) {
+      target_display = display;
+      break;
+    }
+  }
+
+  // TODO(kylechar): Better handle locations outside the window. Overlapping X11
+  // windows, dragging and touch sensors need to be handled properly.
+  if (!target_display) {
+    DVLOG(1) << "Invalid event location " << location->ToString();
+    target_display = *(display_manager()->displays().begin());
+  }
+
+  WindowManagerDisplayRoot* display_root =
+      target_display->GetWindowManagerDisplayRootForUser(user_id());
+
+  if (!display_root)
     return nullptr;
 
-  // TODO(sky): this isn't right. To correctly implement need bounds of
-  // Display, which we aren't tracking yet. For now, use the first display.
-  Display* display = *(display_manager()->displays().begin());
-  WindowManagerDisplayRoot* display_root =
-      display->GetWindowManagerDisplayRootForUser(user_id());
-  return display_root ? display_root->root() : nullptr;
+  // Translate the location to be relative to the display instead of relative
+  // to the screen space.
+  gfx::Point origin = target_display->platform_display()->GetBounds().origin();
+  location->Offset(-origin.x(), -origin.y());

[sky, 2016/08/12 16:47:46]

Typically we do something like:
*location -= origin.OffsetFromOrigin().

[kylechar, 2016/08/12 17:25:12]

Done.

+  return display_root->root();
 }
 
 void WindowManagerState::OnEventTargetNotFound(const ui::Event& event) {
   window_server()->SendToPointerWatchers(event, user_id(),
                                          nullptr /* ignore_tree */);
 }
 
 }  // namespace ws
 }  // namespace ui