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 681 matching lines @@
   EXPECT_EQ(shelf_layout_manager->GetShelfBackgroundType(),
             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());
+  // TODO(muyuanli): This should be SHELF_BACKGROUND_OVERLAP but the test
+  // construction code is not quite correct. See crbug.com/742461.
   EXPECT_EQ(GetPrimaryShelf()->shelf_layout_manager()->GetShelfBackgroundType(),
             SHELF_BACKGROUND_DEFAULT);
 }
 
 // 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();
@@ skipping 32 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