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

Side by Side Diff: BUILD.gn

Issue 2356783002: skia_enable_android_framework_defines in all code, public and private, optional and required. (Closed)
Patch Set: Created 4 years, 3 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 | « no previous file | 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 2016 Google Inc. 1 # Copyright 2016 Google Inc.
2 # 2 #
3 # Use of this source code is governed by a BSD-style license that can be 3 # Use of this source code is governed by a BSD-style license that can be
4 # found in the LICENSE file. 4 # found in the LICENSE file.
5 5
6 import("gn/shared_sources.gni") 6 import("gn/shared_sources.gni")
7 7
8 declare_args() { 8 declare_args() {
9 skia_enable_tools = !is_fuchsia && !is_component_build 9 skia_enable_tools = !is_fuchsia && !is_component_build
10 10
(...skipping 31 matching lines...) Expand 10 before | Expand all | Expand 10 after
42 "include/xml", 42 "include/xml",
43 ] 43 ]
44 44
45 # Skia public API, generally provided by :skia. 45 # Skia public API, generally provided by :skia.
46 config("skia_public") { 46 config("skia_public") {
47 include_dirs = skia_public_includes 47 include_dirs = skia_public_includes
48 defines = [ "SKIA_DLL" ] 48 defines = [ "SKIA_DLL" ]
49 if (is_linux) { 49 if (is_linux) {
50 defines += [ "SK_SAMPLES_FOR_X" ] 50 defines += [ "SK_SAMPLES_FOR_X" ]
51 } 51 }
52 if (skia_enable_android_framework_defines) {
53 defines += skia_android_framework_defines
54 }
52 } 55 }
53 56
54 # Skia internal APIs, used by Skia itself and a few test tools. 57 # Skia internal APIs, used by Skia itself and a few test tools.
55 config("skia_private") { 58 config("skia_private") {
56 visibility = [ ":*" ] 59 visibility = [ ":*" ]
57 60
58 include_dirs = [ 61 include_dirs = [
59 "include/private", 62 "include/private",
60 "src/c", 63 "src/c",
61 "src/codec", 64 "src/codec",
(...skipping 155 matching lines...) Expand 10 before | Expand all | Expand 10 after
217 "sources_when_disabled", 220 "sources_when_disabled",
218 ]) 221 ])
219 if (defined(invoker.sources_when_disabled)) { 222 if (defined(invoker.sources_when_disabled)) {
220 sources = invoker.sources_when_disabled 223 sources = invoker.sources_when_disabled
221 } 224 }
222 configs += skia_library_configs 225 configs += skia_library_configs
223 } 226 }
224 } 227 }
225 } 228 }
226 229
227 optional("android_framework_defines") {
228 enabled = skia_enable_android_framework_defines
229 public_defines = skia_android_framework_defines
230 }
231
232 optional("fontmgr_android") { 230 optional("fontmgr_android") {
233 enabled = fontmgr_android_enabled 231 enabled = fontmgr_android_enabled
234 232
235 deps = [ 233 deps = [
236 "//third_party/expat", 234 "//third_party/expat",
237 "//third_party/freetype2", 235 "//third_party/freetype2",
238 ] 236 ]
239 sources = [ 237 sources = [
240 "src/ports/SkFontMgr_android.cpp", 238 "src/ports/SkFontMgr_android.cpp",
241 "src/ports/SkFontMgr_android_factory.cpp", 239 "src/ports/SkFontMgr_android_factory.cpp",
(...skipping 143 matching lines...) Expand 10 before | Expand all | Expand 10 after
385 "src/xml/SkXMLParser.cpp", 383 "src/xml/SkXMLParser.cpp",
386 "src/xml/SkXMLWriter.cpp", 384 "src/xml/SkXMLWriter.cpp",
387 ] 385 ]
388 } 386 }
389 387
390 component("skia") { 388 component("skia") {
391 public_configs = [ ":skia_public" ] 389 public_configs = [ ":skia_public" ]
392 configs += skia_library_configs 390 configs += skia_library_configs
393 391
394 deps = [ 392 deps = [
395 ":android_framework_defines",
396 ":arm64", 393 ":arm64",
397 ":armv7", 394 ":armv7",
398 ":avx", 395 ":avx",
399 ":crc32", 396 ":crc32",
400 ":dsp", 397 ":dsp",
401 ":fontmgr_android", 398 ":fontmgr_android",
402 ":fontmgr_custom", 399 ":fontmgr_custom",
403 ":fontmgr_fontconfig", 400 ":fontmgr_fontconfig",
404 ":fontmgr_fuchsia", 401 ":fontmgr_fuchsia",
405 ":gif", 402 ":gif",
(...skipping 477 matching lines...) Expand 10 before | Expand all | Expand 10 after
883 "tools/get_images_from_skps.cpp", 880 "tools/get_images_from_skps.cpp",
884 ] 881 ]
885 deps = [ 882 deps = [
886 ":flags", 883 ":flags",
887 ":skia", 884 ":skia",
888 "//third_party/jsoncpp", 885 "//third_party/jsoncpp",
889 ] 886 ]
890 testonly = true 887 testonly = true
891 } 888 }
892 } 889 }
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698