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

Unified Diff: chrome/test/BUILD.gn

Issue 1861233003: Prepare for building with enable_app_list=0 on Desktop (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase for r392580 - app_list_shower_views_unittest deleted by that rev Created 4 years, 7 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/common/extensions/api/schemas.gypi ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/test/BUILD.gn
diff --git a/chrome/test/BUILD.gn b/chrome/test/BUILD.gn
index 7266e2698174996ae66d49ec155a2619303dabcb..93046f8cfb3504ed33f21eedcace6723013853ca 100644
--- a/chrome/test/BUILD.gn
+++ b/chrome/test/BUILD.gn
@@ -661,7 +661,6 @@ if (!is_android) {
"//sync",
"//sync:test_support_sync_fake_server",
"//sync:test_support_sync_testserver",
- "//ui/app_list:test_support",
]
if (is_mac) {
@@ -672,7 +671,9 @@ if (!is_android) {
"../browser/sync/test/integration/dictionary_load_observer.h",
]
}
- if (!enable_app_list) {
+ if (enable_app_list) {
+ deps += [ "//ui/app_list:test_support" ]
+ } else {
sources -= [
"../browser/sync/test/integration/sync_app_list_helper.cc",
"../browser/sync/test/integration/sync_app_list_helper.h",
@@ -1316,6 +1317,12 @@ if (!is_android) {
chrome_tests_gypi_values.chrome_browser_tests_app_list_sources,
".",
"//chrome")
+ if (is_mac) {
+ # This assumes the AppList is views-based, but Mac only links
+ # browser parts for the Cocoa implementation.
+ sources -=
+ [ "../browser/ui/app_list/app_list_service_views_browsertest.cc" ]
+ }
}
if (enable_service_discovery) {
sources += rebase_path(
@@ -2206,6 +2213,8 @@ test("unit_tests") {
chrome_tests_unit_gypi_values.chrome_unit_tests_app_list_chromeos_sources,
".",
"//chrome")
+ sources -=
+ [ "../browser/ui/views/app_list/linux/app_list_linux_unittest.cc" ]
deps += [ "//components/arc:arc_test_support" ]
}
if (enable_plugins && !enable_plugin_installation) {
« no previous file with comments | « chrome/common/extensions/api/schemas.gypi ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698