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

Side by Side Diff: extensions/shell/BUILD.gn

Issue 1969503002: Address more TOOD(GYP) in GN build. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: merge Created 4 years, 7 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 | « extensions/BUILD.gn ('k') | media/gpu/BUILD.gn » ('j') | 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("//extensions/shell/app_shell.gni") 5 import("//extensions/shell/app_shell.gni")
6 6
7 # Technically, this directory should not depend on files from src/chrome, but 7 # Technically, this directory should not depend on files from src/chrome, but
8 # that's where the VERSION file is. This should probably all be moved to 8 # that's where the VERSION file is. This should probably all be moved to
9 # src/build. 9 # src/build.
10 import("//chrome/version.gni") 10 import("//chrome/version.gni")
(...skipping 114 matching lines...) Expand 10 before | Expand all | Expand 10 after
125 "//build/win:default_exe_manifest", 125 "//build/win:default_exe_manifest",
126 "//extensions:shell_and_test_pak", 126 "//extensions:shell_and_test_pak",
127 ] 127 ]
128 128
129 if (is_win) { 129 if (is_win) {
130 configs += [ "//build/config/win:windowed" ] 130 configs += [ "//build/config/win:windowed" ]
131 configs -= [ "//build/config/win:console" ] 131 configs -= [ "//build/config/win:console" ]
132 } 132 }
133 133
134 if (is_mac) { 134 if (is_mac) {
135 # TODO(GYP) bug 546894: Fix GN and toolchains to handle spaces here. 135 output_name = "App Shell"
136 #output_name = "App Shell" 136
137 # TODO(GYP): Mac bundling. See also content_shell which this is basically 137 # TODO(GYP): Mac bundling. See also content_shell which this is basically
138 # a copy-paste of. 138 # a copy-paste of.
139 deps += [ ":app_shell_framework" ] 139 deps += [ ":app_shell_framework" ]
140
140 # TODO(GYP): Mac app_shell_helper stuff. 141 # TODO(GYP): Mac app_shell_helper stuff.
141 } 142 }
142 } 143 }
143 } 144 }
144 145
145 test("app_shell_unittests") { 146 test("app_shell_unittests") {
146 sources = rebase_path(app_shell_gypi_values.app_shell_unittests_sources, 147 sources = rebase_path(app_shell_gypi_values.app_shell_unittests_sources,
147 ".", 148 ".",
148 "//extensions/shell") 149 "//extensions/shell")
149 150
(...skipping 33 matching lines...) Expand 10 before | Expand all | Expand 10 after
183 if (is_chromeos) { 184 if (is_chromeos) {
184 deps += [ "//chromeos:test_support_without_gmock" ] 185 deps += [ "//chromeos:test_support_without_gmock" ]
185 186
186 chromeos_sources = 187 chromeos_sources =
187 rebase_path(app_shell_gypi_values.app_shell_unittests_sources_chromeos, 188 rebase_path(app_shell_gypi_values.app_shell_unittests_sources_chromeos,
188 ".", 189 ".",
189 "//extensions/shell") 190 "//extensions/shell")
190 sources += chromeos_sources 191 sources += chromeos_sources
191 } 192 }
192 193
193 # TODO(GYP): Enable this when //components/nacl GN is done. 194 if (enable_nacl) {
194 if (false) { 195 sources +=
195 if (use_nacl) { 196 rebase_path(app_shell_gypi_values.app_shell_unittests_sources_nacl,
196 nacl_sources = 197 ".",
197 rebase_path(app_shell_gypi_values.app_shell_unittests_sources_nacl, 198 "//extensions/shell")
198 ".",
199 "//extensions/shell")
200 sources += nacl_sources
201 }
202 } 199 }
203 } 200 }
204 201
205 process_version("version_header") { 202 process_version("version_header") {
206 template_file = "common/version.h.in" 203 template_file = "common/version.h.in"
207 output = "$target_gen_dir/common/version.h" 204 output = "$target_gen_dir/common/version.h"
208 } 205 }
209 206
210 if (is_mac) { 207 if (is_mac) {
211 # TODO(GYP) this should be a bundle. Lots of other stuff in this target. 208 # TODO(GYP) this should be a bundle. Lots of other stuff in this target.
212 # Should be able to copy content shell framework (this is basically a 209 # Should be able to copy content shell framework (this is basically a
213 # copy-paste of that target). 210 # copy-paste of that target).
214 shared_library("app_shell_framework") { 211 shared_library("app_shell_framework") {
215 testonly = true 212 testonly = true
216 sources = [ 213 sources = [
217 "app/shell_main_mac.cc", 214 "app/shell_main_mac.cc",
218 "app/shell_main_mac.h", 215 "app/shell_main_mac.h",
219 ] 216 ]
220 deps = [ 217 deps = [
221 ":app_shell_lib", 218 ":app_shell_lib",
222 ] 219 ]
223 } 220 }
224 } 221 }
OLDNEW
« no previous file with comments | « extensions/BUILD.gn ('k') | media/gpu/BUILD.gn » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698