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

Unified Diff: chrome/browser/ui/app_list/app_list_controller_browsertest.cc

Issue 67923002: Fixes for -Wunused-function on Linux, Android and ChromeOS (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebase for realz Created 7 years, 1 month 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/app_list/app_list_controller_browsertest.cc
diff --git a/chrome/browser/ui/app_list/app_list_controller_browsertest.cc b/chrome/browser/ui/app_list/app_list_controller_browsertest.cc
index 92068035f9752609c0a6689726a2302d9f722ba8..d7c106085806cfcecaf68b5e1751be8e8616f34b 100644
--- a/chrome/browser/ui/app_list/app_list_controller_browsertest.cc
+++ b/chrome/browser/ui/app_list/app_list_controller_browsertest.cc
@@ -65,6 +65,7 @@ class AppListControllerBrowserTest : public InProcessBrowserTest {
DISALLOW_COPY_AND_ASSIGN(AppListControllerBrowserTest);
};
+#if !defined(OS_LINUX)
AppListService* GetAppListService() {
// TODO(tapted): Consider testing ash explicitly on the win-ash trybot.
return AppListService::Get(chrome::GetActiveDesktop());
@@ -72,7 +73,6 @@ AppListService* GetAppListService() {
// Test the CreateNewWindow function of the controller delegate.
// TODO(mgiuca): Enable on Linux when supported.
-#if !defined(OS_LINUX)
IN_PROC_BROWSER_TEST_F(AppListControllerBrowserTest, CreateNewWindow) {
const chrome::HostDesktopType desktop = chrome::GetActiveDesktop();
AppListService* service = GetAppListService();

Powered by Google App Engine
This is Rietveld 408576698