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

Unified Diff: chrome/browser/apps/custom_launcher_page_browsertest_views.cc

Issue 2582743002: Remove obsolete app_list Linux code. (Closed)
Patch Set: Remove more things. Created 4 years 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
« no previous file with comments | « no previous file | chrome/browser/ui/BUILD.gn » ('j') | chrome/test/BUILD.gn » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/apps/custom_launcher_page_browsertest_views.cc
diff --git a/chrome/browser/apps/custom_launcher_page_browsertest_views.cc b/chrome/browser/apps/custom_launcher_page_browsertest_views.cc
index a0db749585b6752abad314a65b7ca41c1a2409e4..76b8c27a54a24b0ec50adfcee2a571d39359a446 100644
--- a/chrome/browser/apps/custom_launcher_page_browsertest_views.cc
+++ b/chrome/browser/apps/custom_launcher_page_browsertest_views.cc
@@ -10,14 +10,14 @@
#include "build/build_config.h"
#include "chrome/browser/apps/app_browsertest_util.h"
#include "chrome/browser/ui/app_list/app_list_service.h"
-#include "chrome/browser/ui/app_list/app_list_service_views.h"
-#include "chrome/browser/ui/app_list/app_list_shower_views.h"
+#include "chrome/browser/ui/ash/app_list/test/app_list_service_ash_test_api.h"
#include "content/public/browser/render_frame_host.h"
#include "content/public/browser/web_contents.h"
#include "extensions/common/extension.h"
#include "extensions/common/switches.h"
#include "extensions/test/extension_test_message_listener.h"
#include "ui/app_list/app_list_switches.h"
+#include "ui/app_list/presenter/app_list_presenter_impl.h"
#include "ui/app_list/views/app_list_main_view.h"
#include "ui/app_list/views/app_list_view.h"
#include "ui/app_list/views/contents_view.h"
@@ -29,11 +29,6 @@
#include "ui/views/controls/webview/webview.h"
#include "ui/views/focus/focus_manager.h"
-#if defined(OS_CHROMEOS)
-#include "chrome/browser/ui/ash/app_list/test/app_list_service_ash_test_api.h"
-#include "ui/app_list/presenter/app_list_presenter_impl.h"
-#endif
-
namespace {
// The path of the test application within the "platform_apps" directory.
@@ -75,18 +70,9 @@ class CustomLauncherPageBrowserTest
app_list::AppListView* GetAppListView() {
app_list::AppListView* app_list_view = nullptr;
-#if defined(OS_CHROMEOS)
AppListServiceAshTestApi service_test;
app_list_view = service_test.GetAppListView();
EXPECT_TRUE(service_test.GetAppListPresenter()->GetTargetVisibility());
-#else
- AppListServiceViews* service =
- static_cast<AppListServiceViews*>(AppListService::Get());
- // The app list should have loaded instantly since the profile is already
- // loaded.
- EXPECT_TRUE(service->IsAppListVisible());
- app_list_view = service->shower().app_list();
-#endif
return app_list_view;
}
@@ -222,7 +208,6 @@ IN_PROC_BROWSER_TEST_F(CustomLauncherPageBrowserTest,
const int num_steps = 5;
const int num_fingers = 2;
-#if defined(OS_CHROMEOS)
// Gesture events need to be in host coordinates. On Desktop platforms, the
tapted 2016/12/16 05:11:41 nit: remove "On Desktop platforms.." sentence?
Matt Giuca 2016/12/21 06:18:03 Done.
// Widget is the host, so nothing needs to be done. On ChromeOS, the points
// need to be put into screen coordinates. This works because the root window
@@ -231,7 +216,6 @@ IN_PROC_BROWSER_TEST_F(CustomLauncherPageBrowserTest,
point_above_clickzone.SetPoint(point_in_clickzone.x(), bounds.y() - 10);
views::View::ConvertPointToScreen(contents_view, &point_above_clickzone);
views::View::ConvertPointToScreen(contents_view, &point_in_clickzone);
-#endif
// Back to the start page. And send a scroll gesture.
SetActiveStateAndVerify(app_list::AppListModel::STATE_START);
« no previous file with comments | « no previous file | chrome/browser/ui/BUILD.gn » ('j') | chrome/test/BUILD.gn » ('J')

Powered by Google App Engine
This is Rietveld 408576698