Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(43)

Side by Side Diff: chrome/browser/ui/app_list/app_list_controller_browsertest.cc

Issue 11784034: Skeleton for app_list on OSX, and refactoring for enable_app_list=1 on OS=="mac". (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: disable AppListControllerBrowserTest.ShowAndShutdown on Windows for now Created 7 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
(Empty)
1 // Copyright 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 #include "chrome/browser/ui/app_list/app_list_controller.h"
6 #include "chrome/test/base/in_process_browser_test.h"
7
8 // Browser Test for AppListController that runs on all platforms supporting
9 // app_list.
10 class AppListControllerBrowserTest : public InProcessBrowserTest {
11 public:
12 AppListControllerBrowserTest() {}
13
14 private:
15 DISALLOW_COPY_AND_ASSIGN(AppListControllerBrowserTest);
16 };
17
18 // Disabled on Windows. Investigating in http://crbug.com/168955 .
19 #if defined(OS_WIN)
20 #define MAYBE_ShowAndShutdown DISABLED_ShowAndShutdown
21 #else
22 #define MAYBE_ShowAndShutdown ShowAndShutdown
23 #endif
24
25 // Test showing the app list, followed by browser close.
26 IN_PROC_BROWSER_TEST_F(AppListControllerBrowserTest, MAYBE_ShowAndShutdown) {
27 app_list_controller::ShowAppList();
sail 2013/01/09 22:09:53 I think this should be a platform specific test. T
benwells 2013/01/09 23:35:46 It would be good to test that the window is visibl
28 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698