collect windows from mru_list in workspace_controller.

ash/app_list/app_list_presenter_delegate_unittest.cc

 // Copyright 2013 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/ash_switches.h"
 #include "ash/public/cpp/config.h"
 #include "ash/public/cpp/shelf_types.h"
 #include "ash/public/cpp/shell_window_ids.h"
@@ skipping 682 matching lines @@
             SHELF_BACKGROUND_DEFAULT);
   app_list_presenter_impl()->Dismiss();
 
   // Set the alignment to the side and show the app list. The background
   // should show.
   GetPrimaryShelf()->SetAlignment(ShelfAlignment::SHELF_ALIGNMENT_LEFT);
   app_list_presenter_impl()->Show(GetPrimaryDisplayId());
   EXPECT_TRUE(app_list::features::IsFullscreenAppListEnabled());
   EXPECT_FALSE(GetPrimaryShelf()->IsHorizontalAlignment());
   EXPECT_EQ(GetPrimaryShelf()->shelf_layout_manager()->GetShelfBackgroundType(),
-            SHELF_BACKGROUND_DEFAULT);

[oshima, 2017/07/13 04:41:32]

This should stay right?

[Muyuan, 2017/07/13 18:46:01]

It should be OVERLAP per UX requirement. The test itself is not setup properly..
I put down a todo and created a bug:
https://bugs.chromium.org/p/chromium/issues/detail?id=742461 to track the issue
separately.

+            SHELF_BACKGROUND_OVERLAP);
 }
 
 // Tests that the half app list closes if the user taps outside its bounds.
 TEST_F(FullscreenAppListPresenterDelegateTest,
        TapAndClickOutsideClosesHalfAppList) {
   // TODO(newcomer): Investigate mash failures crbug.com/726838
   app_list_presenter_impl()->Show(GetPrimaryDisplayId());
   ui::test::EventGenerator& generator = GetEventGenerator();
 
   // Transition to half app list by entering text.
@@ skipping 30 matching lines @@
   generator.PressKey(ui::KeyboardCode::VKEY_0, 0);
   EXPECT_EQ(app_list->app_list_state(), app_list::AppListView::HALF);
 
   // Clicking outside the bounds closes the app list.
   generator.MoveMouseTo(gfx::Point(10, 10));
   generator.ClickLeftButton();
   EXPECT_FALSE(app_list_presenter_impl()->IsVisible());
 }
 
 }  // namespace ash