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

Unified Diff: chrome/browser/BUILD.gn

Issue 1825603003: Reland of move configuration policy from chrome build files. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@policy_chrome_defines
Patch Set: Created 4 years, 9 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/app/BUILD.gn ('k') | chrome/browser/extensions/BUILD.gn » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/BUILD.gn
diff --git a/chrome/browser/BUILD.gn b/chrome/browser/BUILD.gn
index 149156da66ae62f1123d49f48a1cc49906a4dc73..0bc2ce4bcea8e110f02f551e7d9a4f1056f08235 100644
--- a/chrome/browser/BUILD.gn
+++ b/chrome/browser/BUILD.gn
@@ -150,7 +150,9 @@
"//components/packed_ct_ev_whitelist",
"//components/password_manager/core/browser",
"//components/password_manager/core/common",
+ "//components/policy",
"//components/policy:policy_component",
+ "//components/policy/proto",
"//components/proxy_config",
"//components/query_parser",
"//components/rappor",
@@ -420,16 +422,20 @@
if (!is_win && !is_mac && !is_ios) {
sources += [ "net/net_error_diagnostics_dialog_generic.cc" ]
}
- if (!is_android && !is_ios && !is_chromeos && enable_configuration_policy) {
+ if (!is_android && !is_ios && !is_chromeos) {
sources += [
"net/disk_cache_dir_policy_handler.cc",
"net/disk_cache_dir_policy_handler.h",
]
}
- if (!is_android && !is_ios && enable_configuration_policy) {
+ if (!is_android && !is_ios) {
sources += [
"download/download_dir_policy_handler.cc",
"download/download_dir_policy_handler.h",
+ ]
+ deps += [
+ "//chrome/browser/policy:path_parser",
+ "//net:net_browser_services",
]
}
if (is_mac) {
@@ -496,48 +502,6 @@
sources +=
rebase_path(gypi_values.chrome_browser_nacl_sources, ".", "//chrome")
deps += [ "//components/nacl/browser" ]
- }
-
- if (enable_configuration_policy) {
- sources +=
- rebase_path(gypi_values.chrome_browser_policy_shared_with_ios_sources,
- ".",
- "//chrome")
- deps += [
- "//components/policy",
- "//components/policy/proto",
- ]
- if (!is_ios) {
- sources += rebase_path(gypi_values.chrome_browser_policy_non_ios_sources,
- ".",
- "//chrome")
- }
- if (!is_chromeos) {
- sources += rebase_path(
- gypi_values.chrome_browser_policy_shared_with_ios_not_chromeos_sources,
- ".",
- "//chrome")
- }
- if (is_win || is_mac || is_desktop_linux) {
- sources += rebase_path(gypi_values.chrome_browser_policy_desktop_sources,
- ".",
- "//chrome")
- }
- if (is_android || is_ios) {
- sources += rebase_path(gypi_values.chrome_browser_policy_mobile_sources,
- ".",
- "//chrome")
- } else {
- deps += [
- "//chrome/browser/policy:path_parser",
- "//net:net_browser_services",
- ]
- }
- } else {
- # Configuration policy disabled.
- sources += rebase_path(gypi_values.chrome_browser_policy_disabled_sources,
- ".",
- "//chrome")
}
if (enable_plugins) {
@@ -598,6 +562,10 @@
".",
"//chrome")
deps += [ "//chrome/browser/chromeos" ]
+ } else { # Non-ChromeOS.
+ sources += rebase_path(gypi_values.chrome_browser_non_chromeos_sources,
+ ".",
+ "//chrome")
}
if (is_chromeos || is_ios) {
« no previous file with comments | « chrome/app/BUILD.gn ('k') | chrome/browser/extensions/BUILD.gn » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698