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

Unified Diff: chrome/browser/BUILD.gn

Issue 1459793002: Android: Allow compiling browser without Java UI (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 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/BUILD.gn
diff --git a/chrome/browser/BUILD.gn b/chrome/browser/BUILD.gn
index 2a0fcbe1f52a5d56617977b83f465175a8afc468..be335f374f1340e42302578e69de4f1c7827fa9a 100644
--- a/chrome/browser/BUILD.gn
+++ b/chrome/browser/BUILD.gn
@@ -137,13 +137,13 @@ source_set("browser") {
"//components/metrics:net",
"//components/metrics:profiler",
"//components/metrics:profiler_content",
- "//components/metrics/proto:proto",
"//components/metrics:ui",
+ "//components/metrics/proto:proto",
"//components/metrics_services_manager",
"//components/mime_util",
"//components/navigation_metrics",
- "//components/ntp_snippets",
"//components/network_time",
+ "//components/ntp_snippets",
"//components/offline_pages",
"//components/omnibox/browser",
"//components/os_crypt",
@@ -180,9 +180,9 @@ source_set("browser") {
"//components/variations",
"//components/variations/net",
"//components/variations/service",
+ "//components/web_resource",
"//components/webdata/common",
"//components/webdata_services",
- "//components/web_resource",
"//content/public/browser",
"//content/public/common",
"//courgette:courgette_lib",
@@ -194,8 +194,8 @@ source_set("browser") {
"//sync",
"//third_party/cacheinvalidation",
"//third_party/icu",
- "//third_party/libxml",
"//third_party/libjingle",
+ "//third_party/libxml",
"//third_party/widevine/cdm:version_h",
"//third_party/zlib",
"//third_party/zlib:minizip",
@@ -205,9 +205,9 @@ source_set("browser") {
"//ui/gfx",
"//ui/gfx/geometry",
"//ui/message_center",
+ "//ui/resources",
"//ui/shell_dialogs",
"//ui/strings",
- "//ui/resources",
]
data_deps = []
@@ -276,6 +276,34 @@ source_set("browser") {
sources += rebase_path(gypi_values.chrome_browser_web_resource_sources,
".",
"//chrome")
+ if (android_java_ui) {
+ sources +=
+ rebase_path(gypi_values.chrome_browser_sync_android_java_ui_sources,
+ ".",
+ "//chrome")
+ sources +=
+ rebase_path(gypi_values.chrome_browser_ssl_android_java_ui_sources,
+ ".",
+ "//chrome")
+ sources += rebase_path(
+ gypi_values.chrome_browser_history_android_java_ui_sources,
+ ".",
+ "//chrome")
+ sources += rebase_path(
+ gypi_values.chrome_browser_permissions_android_java_ui_sources,
+ ".",
+ "//chrome")
+ sources += rebase_path(
+ gypi_values.chrome_browser_search_engines_android_java_ui_sources,
+ ".",
+ "//chrome")
+ if (enable_supervised_users) {
+ sources += rebase_path(
+ gypi_values.chrome_browser_supervised_user_android_java_ui_sources,
+ ".",
+ "//chrome")
+ }
+ }
deps += [
"//apps",
@@ -634,11 +662,11 @@ source_set("browser") {
sources += rebase_path(gypi_values.chrome_browser_notifications_sources,
".",
"//chrome")
- if (is_android) {
- sources +=
- rebase_path(gypi_values.chrome_browser_notifications_android_sources,
- ".",
- "//chrome")
+ if (android_java_ui) {
+ sources += rebase_path(
+ gypi_values.chrome_browser_notifications_android_java_ui_sources,
+ ".",
+ "//chrome")
} else {
sources += rebase_path(
gypi_values.chrome_browser_notifications_non_android_sources,
@@ -657,8 +685,8 @@ source_set("browser") {
".",
"//chrome")
deps += [
- "//printing",
"//components/printing/browser",
+ "//printing",
]
if (is_win) {
@@ -717,6 +745,12 @@ source_set("browser") {
if (is_android) {
sources +=
rebase_path(gypi_values.chrome_browser_android_sources, ".", "//chrome")
+ }
+
+ if (android_java_ui) {
+ sources += rebase_path(gypi_values.chrome_browser_android_java_ui_sources,
+ ".",
+ "//chrome")
sources += rebase_path(gypi_values.chrome_browser_bookmark_android_sources,
".",
"//chrome")
@@ -724,7 +758,6 @@ source_set("browser") {
":client_discourse_context_proto",
":delta_file_proto",
":jni_headers",
- "//components/cdm/browser",
"//components/data_usage/android",
"//components/enhanced_bookmarks",
"//components/precache/content",
@@ -733,16 +766,25 @@ source_set("browser") {
"//components/service_tab_launcher",
"//components/toolbar",
"//components/web_contents_delegate_android",
+ ]
+
+ defines += [ "ENABLE_DATA_REDUCTION_PROXY_DEBUGGING" ]
+ }
+
+ if (is_android) {
+ deps += [
+ "//components/cdm/browser",
+ "//components/resources:components_resources",
"//third_party/android_opengl/etc1",
"//third_party/android_tools:cpu_features",
"//third_party/libaddressinput:util",
]
+
deps -= [
- "//third_party/libaddressinput",
"//components/storage_monitor",
"//components/web_modal",
+ "//third_party/libaddressinput",
]
- defines += [ "ENABLE_DATA_REDUCTION_PROXY_DEBUGGING" ]
if (use_seccomp_bpf) {
defines += [ "USE_SECCOMP_BPF" ]
@@ -903,7 +945,7 @@ source_set("browser") {
}
}
-if (is_android) {
+if (android_java_ui) {
# GYP version: chrome/chrome_browser.gypi:chrome_browser_jni_headers
generate_jni("jni_headers") {
sources =
@@ -1068,6 +1110,7 @@ source_set("test_support") {
]
deps = [
"//base:prefs_test_support",
+ "//chrome/app/theme:theme_resources",
"//chrome/browser",
"//chrome/common",
"//chrome/common/safe_browsing:proto",
@@ -1078,7 +1121,6 @@ source_set("test_support") {
"//components/syncable_prefs:test_support",
"//components/user_prefs/tracked:user_prefs_tracked_test_support",
"//content/test:test_support",
- "//chrome/app/theme:theme_resources",
"//net:test_support",
"//skia",
"//testing/gmock",

Powered by Google App Engine
This is Rietveld 408576698