Support creation of toplevel mus::Windows on separate displays

ash/mus/root_window_controller.cc

Index: ash/mus/root_window_controller.cc
diff --git a/ash/mus/root_window_controller.cc b/ash/mus/root_window_controller.cc
index f5da782f9ba624961ad47415a11f907d94d743bb..394c558daeebc0b8398483fbd09b7d89773955c2 100644
--- a/ash/mus/root_window_controller.cc
+++ b/ash/mus/root_window_controller.cc
@@ -102,9 +102,21 @@ ui::Window* RootWindowController::NewTopLevelWindow(
   if (GetRequestedContainer(window, &container_id)) {
     container_window = GetWindowByShellWindowId(container_id)->mus_window();
   } else {
-    // TODO(sky): window->bounds() isn't quite right.
+    const gfx::Point top_right_in_screen =

[sky, 2016/11/23 20:48:19]

wm_window has a GetBoundsInScreen. Can't you use it?

[thanhph, 2016/11/23 21:31:42]

I can't. I tried to use it as below but got ASAN:DEADLYSIGNAL error because
window_ wasn't initialized.

Const gfx::Rect bounds_in_screen =
WmWindowMus::Get(window)->GetBoundsInScreen();

Hence, I use wm_root_window_controller_ instead.

[sky, 2016/11/23 21:45:36]

This is likely failing because window doesn't have a parent yet. I think you
want:

gfx::Point origin =
wm_root_window_controller_->ConvertPointToScreen(WmWindowMus::Get(root_),
gfx::Point());
gfx::Rect bounds_in_screen(origin, bounds.size());

[mfomitchev, 2016/11/23 22:00:33]

Scott, this will not work if there is a transform on the window or one of its
parents, right? (bounds won't be the same)

[thanhph, 2016/11/23 22:25:55]

Done, thanks! This is shorter.

+        wm_root_window_controller_->ConvertPointToScreen(
+            WmWindowMus::Get(window), window->bounds().top_right());
+
+    const gfx::Point bottom_left_in_screen =
+        wm_root_window_controller_->ConvertPointToScreen(
+            WmWindowMus::Get(window), window->bounds().bottom_left());
+
+    const gfx::Rect bounds_in_screen =
+        gfx::Rect(bottom_left_in_screen.x(), top_right_in_screen.y(),
+                  top_right_in_screen.x() - bottom_left_in_screen.x(),
+                  bottom_left_in_screen.y() - top_right_in_screen.y());
+
     container_window = WmWindowMus::GetMusWindow(wm::GetDefaultParent(
-        WmWindowMus::Get(root_), WmWindowMus::Get(window), window->bounds()));
+        WmWindowMus::Get(root_), WmWindowMus::Get(window), bounds_in_screen));
   }
   DCHECK(WmWindowMus::Get(container_window)->IsContainer());