Keep the split view activities topmost in z-order in Athena

athena/wm/split_view_controller_unittest.cc

Index: athena/wm/split_view_controller_unittest.cc
diff --git a/athena/wm/split_view_controller_unittest.cc b/athena/wm/split_view_controller_unittest.cc
index 6bde13b5f66817c2df9e98a98cda9e3c5507bfda..4bd5ce67d4b42d44503176160c959d13d02d0376 100644
--- a/athena/wm/split_view_controller_unittest.cc
+++ b/athena/wm/split_view_controller_unittest.cc
@@ -32,6 +32,20 @@ class SplitViewControllerTest : public test::AthenaTestBase {
     test::AthenaTestBase::TearDown();
   }
 
+  // Returns whether the split view windows are topmost.
+  bool SplitViewWindowsTopmost() const {
+    SplitViewController* controller = api_->GetSplitViewController();
+    DCHECK(controller->IsSplitViewModeActive());
+    aura::Window::Windows list =
+        api_->GetWindowListProvider()->GetWindowList();
+    aura::Window* topmost = *list.rbegin();
+    aura::Window* second_topmost = *(list.rbegin() + 1);
+    return (topmost == controller->left_window() ||
+            topmost == controller->right_window()) &&
+           (second_topmost == controller->left_window() ||
+            second_topmost == controller->right_window());
+  }
+
   bool IsSplitViewAllowed() const {
     return api_->GetSplitViewController()->CanScroll();
   }
@@ -58,7 +72,6 @@ TEST_F(SplitViewControllerTest, SplitModeActivation) {
   }
 
   SplitViewController* controller = api()->GetSplitViewController();
-  WindowListProvider* list_provider = api()->GetWindowListProvider();
   ASSERT_FALSE(controller->IsSplitViewModeActive());
 
   controller->ActivateSplitMode(NULL, NULL);
@@ -66,34 +79,36 @@ TEST_F(SplitViewControllerTest, SplitModeActivation) {
   // The last two windows should be on the left and right, respectively.
   EXPECT_EQ(windows[kNumWindows - 1], controller->left_window());
   EXPECT_EQ(windows[kNumWindows - 2], controller->right_window());
+  EXPECT_TRUE(SplitViewWindowsTopmost());
 
   // Select the window that is currently on the left for the right panel. The
   // windows should switch.
   controller->ActivateSplitMode(NULL, windows[kNumWindows - 1]);
   EXPECT_EQ(windows[kNumWindows - 2], controller->left_window());
   EXPECT_EQ(windows[kNumWindows - 1], controller->right_window());
+  EXPECT_TRUE(SplitViewWindowsTopmost());
 
   controller->ActivateSplitMode(windows[kNumWindows - 1], NULL);
   EXPECT_EQ(windows[kNumWindows - 1], controller->left_window());
   EXPECT_EQ(windows[kNumWindows - 2], controller->right_window());
+  EXPECT_TRUE(SplitViewWindowsTopmost());
 
   // Select one of the windows behind the stacks for the right panel. The window
   // on the left should remain unchanged.
   controller->ActivateSplitMode(NULL, windows[0]);
   EXPECT_EQ(windows[kNumWindows - 1], controller->left_window());
   EXPECT_EQ(windows[0], controller->right_window());
-  EXPECT_EQ(windows[0], *list_provider->GetWindowList().rbegin());
+  EXPECT_TRUE(SplitViewWindowsTopmost());
 
   controller->ActivateSplitMode(windows[1], NULL);
   EXPECT_EQ(windows[1], controller->left_window());
   EXPECT_EQ(windows[0], controller->right_window());
-  EXPECT_EQ(windows[1], *list_provider->GetWindowList().rbegin());
+  EXPECT_TRUE(SplitViewWindowsTopmost());
 
   controller->ActivateSplitMode(windows[4], windows[5]);
   EXPECT_EQ(windows[4], controller->left_window());
   EXPECT_EQ(windows[5], controller->right_window());
-  EXPECT_EQ(windows[4], *list_provider->GetWindowList().rbegin());
-  EXPECT_EQ(windows[5], *(list_provider->GetWindowList().rbegin() + 1));
+  EXPECT_TRUE(SplitViewWindowsTopmost());

[pkotwicz, 2014/09/11 17:26:15]

The tests happen to pass without the changes in this CL in
split_view_controller.cc
However, SplitViewController::ActivateSplitMode() did not guarantee that the
split view windows were the topmost windows.

e.g. Calling controller->ActivateSplitMode(windows[0], NULL) after line 111
would have not resulted in the split view windows being topmost.

[sadrul, 2014/09/12 04:32:29]

Can you add that test-case here?

[oshima, 2014/09/12 18:48:41]

You probably missed this?

 }
 
 TEST_F(SplitViewControllerTest, LandscapeOnly) {