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

Side by Side Diff: chrome/browser/BUILD.gn

Issue 2105513002: Use split_static_library in GN build. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@split_list
Patch Set: merge Created 4 years, 5 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 unified diff | Download patch
« no previous file with comments | « build/split_static_library.gni ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # Copyright 2014 The Chromium Authors. All rights reserved. 1 # Copyright 2014 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be 2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file. 3 # found in the LICENSE file.
4 4
5 import("//build/config/chrome_build.gni") 5 import("//build/config/chrome_build.gni")
6 import("//build/config/crypto.gni") 6 import("//build/config/crypto.gni")
7 import("//build/config/features.gni") 7 import("//build/config/features.gni")
8 import("//build/config/ui.gni") 8 import("//build/config/ui.gni")
9 import("//build/split_static_library.gni")
9 import("//chrome/common/features.gni") 10 import("//chrome/common/features.gni")
10 import("//media/media_options.gni") 11 import("//media/media_options.gni")
11 import("//third_party/protobuf/proto_library.gni") 12 import("//third_party/protobuf/proto_library.gni")
12 13
13 # //build/config/android/rules.gni imports //tools/grit/grit_rule.gni, which 14 # //build/config/android/rules.gni imports //tools/grit/grit_rule.gni, which
14 # produces a conflict for the "grit" template so we have to only include one. 15 # produces a conflict for the "grit" template so we have to only include one.
15 if (is_android) { 16 if (is_android) {
16 import("//build/config/android/rules.gni") 17 import("//build/config/android/rules.gni")
17 } else { 18 } else {
18 import("//tools/grit/grit_rule.gni") 19 import("//tools/grit/grit_rule.gni")
(...skipping 52 matching lines...) Expand 10 before | Expand all | Expand 10 after
71 # ADDITIONALLY add this config on top of ":gnome_keyring". pkg-config is a 72 # ADDITIONALLY add this config on top of ":gnome_keyring". pkg-config is a
72 # bit slow, so prefer not to run it again. In practice, gnome-keyring's libs 73 # bit slow, so prefer not to run it again. In practice, gnome-keyring's libs
73 # are just itself and common gnome ones we link already, so we can get away 74 # are just itself and common gnome ones we link already, so we can get away
74 # with additionally just coding the library name here. 75 # with additionally just coding the library name here.
75 config("gnome_keyring_direct") { 76 config("gnome_keyring_direct") {
76 libs = [ "gnome-keyring" ] 77 libs = [ "gnome-keyring" ]
77 } 78 }
78 } 79 }
79 80
80 # Use a static library here because many test binaries depend on this but don't 81 # Use a static library here because many test binaries depend on this but don't
81 # require many files from it. This makes linking more efficient. However, on 82 # require many files from it. This makes linking more efficient.
82 # Windows official builds the static library will be too large (> 32-bits) so 83 split_static_library("browser") {
83 # use source sets in that case. 84 # Split into multiple static libraries on Windows official builds, where we
84 if (is_win && is_official_build) { 85 # run into a 2GB max size limit.
85 chrome_browser_target_type = "source_set" 86 if (is_win && is_official_build) {
86 } else { 87 split_count = 5
87 chrome_browser_target_type = "static_library" 88 } else {
88 } 89 split_count = 1
89 target(chrome_browser_target_type, "browser") { 90 }
91
90 configs += [ 92 configs += [
91 "//build/config/compiler:wexit_time_destructors", 93 "//build/config/compiler:wexit_time_destructors",
92 "//build/config:precompiled_headers", 94 "//build/config:precompiled_headers",
93 "//third_party/WebKit/public:debug_devtools", 95 "//third_party/WebKit/public:debug_devtools",
94 ] 96 ]
95 defines = [] 97 defines = []
96 sources = [] 98 sources = []
97 libs = [] 99 libs = []
98 ldflags = [] 100 ldflags = []
99 101
(...skipping 1261 matching lines...) Expand 10 before | Expand all | Expand 10 after
1361 # linking all of the test support. 1363 # linking all of the test support.
1362 source_set("pepper_cdm_test_constants") { 1364 source_set("pepper_cdm_test_constants") {
1363 testonly = true 1365 testonly = true
1364 visibility = [ "//chrome/*" ] 1366 visibility = [ "//chrome/*" ]
1365 sources = [ 1367 sources = [
1366 "media/pepper_cdm_test_constants.cc", 1368 "media/pepper_cdm_test_constants.cc",
1367 "media/pepper_cdm_test_constants.h", 1369 "media/pepper_cdm_test_constants.h",
1368 ] 1370 ]
1369 } 1371 }
1370 } 1372 }
OLDNEW
« no previous file with comments | « build/split_static_library.gni ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698