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

athena/wm/split_view_controller.cc

 // Copyright 2014 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 "athena/wm/split_view_controller.h"
 
 #include <cmath>
 
 #include "athena/screen/public/screen_manager.h"
 #include "athena/wm/public/window_list_provider.h"
@@ skipping 77 matching lines @@
   if (!right && iter != windows.rend()) {
     right = *iter;
     iter++;
     if (right == left && iter != windows.rend()) {
       right = *iter;
       iter++;
     }
   }
 
   SetState(ACTIVE);
-  if (right_window_ != right) {
-    right_window_ = right;
-    // Since the |window_list_provider_| controls directly the order of windows,
-    // it needs to change the window order accordingly.
-    window_list_provider_->MoveToFront(right_window_);
-  }
-  if (left_window_ != left) {
-    left_window_ = left;
-    // Since the |window_list_provider_| controls directly the order of windows,
-    // it needs to change the window order accordingly.
-    window_list_provider_->MoveToFront(left_window_);
-  }
+  right_window_ = right;
+  left_window_ = left;
   UpdateLayout(true);
 }
 
 void SplitViewController::ReplaceWindow(aura::Window* window,
                                         aura::Window* replace_with) {
   CHECK(IsSplitViewModeActive());
   CHECK(replace_with);
   CHECK(window == left_window_ || window == right_window_);
   CHECK(replace_with != left_window_ && replace_with != right_window_);
 #if !defined(NDEBUG)
@@ skipping 52 matching lines @@
     if (!wm::IsActiveWindow(left_window_))
       left_window_->Hide();
     if (!wm::IsActiveWindow(right_window_))
       right_window_->Hide();
     SetWindowTransforms(gfx::Transform(), gfx::Transform(), false);
     return;
   }
 
   left_window_->Show();
   right_window_->Show();
+  window_list_provider_->MoveToFront(right_window_);
+  window_list_provider_->MoveToFront(left_window_);
+
   if (state_ == ACTIVE) {
     if (animate) {
       gfx::Transform left_transform = GetTargetTransformForBoundsAnimation(
           left_window_->bounds(), GetLeftTargetBounds());
       gfx::Transform right_transform = GetTargetTransformForBoundsAnimation(
           right_window_->bounds(), GetRightTargetBounds());
       SetWindowTransforms(left_transform, right_transform, true);
     } else {
       left_window_->SetBounds(GetLeftTargetBounds());
       right_window_->SetBounds(GetRightTargetBounds());
@@ skipping 124 matching lines @@
 bool SplitViewController::CanScroll() {
   // TODO(mfomitchev): return false in full screen.
   bool result = (!IsSplitViewModeActive() &&
                  window_list_provider_->GetWindowList().size() >= 2 &&
                  IsLandscapeOrientation(gfx::Screen::GetNativeScreen()->
                      GetDisplayNearestWindow(container_).rotation()));
   return result;
 }
 
 }  // namespace athena