Support creation of toplevel mus::Windows on separate displays

ash/mus/window_manager_unittest.cc

 // Copyright 2015 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 <memory>
 
+#include "ash/mus/test/wm_test_base.h"
 #include "base/bind.h"
 #include "base/macros.h"
 #include "services/service_manager/public/cpp/service_test.h"
 #include "services/ui/public/cpp/window.h"
 #include "services/ui/public/cpp/window_tree_client.h"
 #include "services/ui/public/cpp/window_tree_client_delegate.h"
 #include "services/ui/public/interfaces/window_tree.mojom.h"
 
 namespace ash {
 namespace mus {
@@ skipping 56 matching lines @@
   // |child_window|. This blocks until it succeeds.
   ui::mojom::WindowTreeClientPtr tree_client;
   auto tree_client_request = GetProxy(&tree_client);
   child_window->Embed(std::move(tree_client), base::Bind(&OnEmbed));
   std::unique_ptr<ui::WindowTreeClient> child_client(new ui::WindowTreeClient(
       &window_tree_delegate, nullptr, std::move(tree_client_request)));
   child_client->WaitForEmbed();
   ASSERT_TRUE(!child_client->GetRoots().empty());
 }
 
+using DisplayWindowInSpecifiedDisplayTest = mus::WmTestBase;

[mfomitchev, 2016/11/23 17:58:00]

I think this is still too specific. This name corresponds to a set of test
cases, and you are trying to make it specific to one test case.
"IsWindowShownInCorrectDisplay" (the name in TEST_F below) is the name specific
to the test case. This one on the other hand should be more generic - something
that should be able to accommodate more tests in the future. E.g.
WindowManagerlDisplayTest.

[thanhph, 2016/11/23 18:42:19]

I misinterpreted your last comment. Done, thanks!

+
+TEST_F(DisplayWindowInSpecifiedDisplayTest, IsWindowShownInCorrectDisplay) {
+  if (!SupportsMultipleDisplays())
+    return;
+
+  UpdateDisplay("400x400,400x400");
+  EXPECT_NE(GetPrimaryDisplay().id(), GetSecondaryDisplay().id());
+
+  ui::Window* window_primary_display =
+      CreateFullscreenTestWindow(GetPrimaryDisplay().id());
+  ui::Window* window_secondary_display =
+      CreateFullscreenTestWindow(GetSecondaryDisplay().id());
+
+  DCHECK(window_primary_display);
+  DCHECK(window_secondary_display);
+
+  EXPECT_EQ(window_primary_display->display_id(), GetPrimaryDisplay().id());
+  EXPECT_EQ(window_secondary_display->display_id(), GetSecondaryDisplay().id());
+}
+
 }  // namespace mus
 }  // namespace ash