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

Unified Diff: content/public/app/BUILD.gn

Issue 704363002: XXX test gn format everything (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: . Created 6 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
« no previous file with comments | « content/public/android/BUILD.gn ('k') | content/public/browser/BUILD.gn » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/public/app/BUILD.gn
diff --git a/content/public/app/BUILD.gn b/content/public/app/BUILD.gn
index ccb31698574d5a33f850e85d5488c8803cc27704..2b1ebf480a8f78cd97fe677eacefa98206301393 100644
--- a/content/public/app/BUILD.gn
+++ b/content/public/app/BUILD.gn
@@ -37,7 +37,6 @@ public_app_shared_deps = [
]
if (is_component_build) {
-
source_set("both_sources") {
# Only the main content shared library can pull this in.
visibility = [ "//content:content" ]
@@ -47,24 +46,28 @@ if (is_component_build) {
configs += [ "//content:content_implementation" ]
deps = public_app_shared_deps + [
- "//content/app:both",
- "//content/public/browser:browser_sources",
- ]
+ "//content/app:both",
+ "//content/public/browser:browser_sources",
+ ]
}
# These all just forward to content, which in turn depends on "both_sources".
group("browser") {
- deps = [ "//content" ]
+ deps = [
+ "//content",
+ ]
}
group("child") {
- deps = [ "//content" ]
+ deps = [
+ "//content",
+ ]
}
group("both") {
- deps = [ "//content" ]
+ deps = [
+ "//content",
+ ]
}
-
} else {
-
# content_main_delegate.cc conditionally includes content_browser_client.h
# from //content/public/browser when it's not the child build. However,
# the header checker doesn't know this doesn't apply and throws an error.
@@ -82,10 +85,10 @@ if (is_component_build) {
sources = public_app_shared_sources
configs += [ "//content:content_implementation" ]
deps = public_app_shared_deps + [
- "//content/app:both",
- "//content/public/browser",
- "//content/public/common",
- ]
+ "//content/app:both",
+ "//content/public/browser",
+ "//content/public/common",
+ ]
}
# TODO(GYP) enable chrome_multiple_dll support
@@ -101,10 +104,10 @@ if (is_component_build) {
configs += [ "//content:content_implementation" ]
deps = public_app_shared_deps + [
- "//content/app:browser",
- "//content/public/browser",
- "//content/public/common",
- ]
+ "//content/app:browser",
+ "//content/public/browser",
+ "//content/public/common",
+ ]
}
source_set("child") {
@@ -116,19 +119,22 @@ if (is_component_build) {
configs += [ "//content:content_implementation" ]
deps = public_app_shared_deps + [
- "//content/app:child",
- "//content/public/common",
- ]
+ "//content/app:child",
+ "//content/public/common",
+ ]
}
} else {
# When the multi-DLL build is disabled, there is only one type of the
# "app" target, and "browser" and "child" are the same as "both".
group("browser") {
- deps = [ ":both" ]
+ deps = [
+ ":both",
+ ]
}
group("child") {
- deps = [ ":both" ]
+ deps = [
+ ":both",
+ ]
}
}
-
}
« no previous file with comments | « content/public/android/BUILD.gn ('k') | content/public/browser/BUILD.gn » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698