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

Unified Diff: chrome/test/BUILD.gn

Issue 936193004: add gn_all, gyp_all targets to build files, remove GN's //:root (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: temporarily disable gn_all and gyp_all on Android Created 5 years, 10 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
Index: chrome/test/BUILD.gn
diff --git a/chrome/test/BUILD.gn b/chrome/test/BUILD.gn
index 06b9aff07268eba9e9663bd55f3aedacfcc49555..e905563cf05ce7938f1b2ee737e193682df254c9 100644
--- a/chrome/test/BUILD.gn
+++ b/chrome/test/BUILD.gn
@@ -380,9 +380,7 @@ if (!is_android && (!is_win || link_chrome_on_windows)) {
}
if (is_mac) {
- datadeps = [
- "//chrome",
- ]
+ datadeps += [ "//chrome" ]
# TODO(mark): We really want this for all non-static library targets, but
# when we tried to pull it up to the common.gypi level, it broke other
@@ -879,7 +877,6 @@ if (!is_android && (!is_win || link_chrome_on_windows)) {
]
sources += [
"../browser/media_galleries/fileapi/iphoto_data_provider_browsertest.cc",
- "../browser/renderer_host/chrome_render_widget_host_view_mac_delegate_browsertest.cc",
"../browser/renderer_host/chrome_render_widget_host_view_mac_history_swiper_browsertest.mm",
"../browser/spellchecker/spellcheck_message_filter_mac_browsertest.cc",
]
@@ -890,11 +887,13 @@ if (!is_android && (!is_win || link_chrome_on_windows)) {
# TODO(rouslan): This test depends on the custom dictionary UI,
# which is disabled on Mac.
- "../browser/ui/webui/options/edit_dictionary_browsertest.js",
+ # TODO(GYP): This should be added somewhere first ...
+ # "../browser/ui/webui/options/edit_dictionary_browsertest.js",
# TODO(rouslan): This test depends on hunspell and we cannot run it
# on Mac, which does use hunspell by default.
- "../browser/ui/webui/options/language_options_dictionary_download_browsertest.js",
+ # TODO(GYP): This should be added somewhere first ...
+ # "../browser/ui/webui/options/language_options_dictionary_download_browsertest.js",
# ProcessSingletonMac doesn"t do anything.
"../browser/process_singleton_browsertest.cc",
@@ -1551,7 +1550,7 @@ if (!is_android && (!is_win || link_chrome_on_windows)) {
# on disk so it can pull from it.
deps += [
"//third_party/google_toolbox_for_mac",
- "//third_party/ocmock",
+ # "//third_party/ocmock", # TODO(GYP)
]
# TODO(mark): We really want this for all non-static library targets,

Powered by Google App Engine
This is Rietveld 408576698