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

Unified Diff: gyp/ports.gyp

Issue 1228833003: Allow Android font manager on *nix. (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: Remove unintended whitespace. Created 5 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | gyp/tests.gypi » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: gyp/ports.gyp
diff --git a/gyp/ports.gyp b/gyp/ports.gyp
index a0cfbd64da4df1c7bc3bf10d6fb4fe5e11f62e0b..9d63d5503a9ed536870300925675f88a4e09ca1f 100644
--- a/gyp/ports.gyp
+++ b/gyp/ports.gyp
@@ -77,29 +77,33 @@
'sources': [
'../src/ports/SkFontHost_FreeType.cpp',
'../src/ports/SkFontHost_FreeType_common.cpp',
+ '../src/ports/SkFontMgr_android.cpp',
+ '../src/ports/SkFontMgr_android_parser.cpp',
+ '../src/ports/SkFontMgr_custom.cpp',
],
'dependencies': [
'freetype.gyp:freetype',
],
- }],
- [ 'skia_os in ["linux", "freebsd", "openbsd", "solaris", "chromeos"]', {
'conditions': [
- [ 'skia_embedded_fonts', {
+ [ 'skia_os == "android"', {
+ 'dependencies': [
+ 'android_deps.gyp:expat',
+ ],
+ }, {
'link_settings': {
'libraries': [
'-ldl',
+ '-lexpat',
],
},
+ }],
+ [ 'skia_embedded_fonts', {
'variables': {
'embedded_font_data_identifier': 'sk_fonts',
'fonts_to_include': [
'../resources/fonts/Funkster.ttf',
],
},
- 'sources': [
- '../include/ports/SkFontMgr_custom.h',
- '../src/ports/SkFontMgr_custom.cpp',
- ],
'sources/': [['include', '../src/ports/SkFontMgr_custom_embedded_factory.cpp']],
'actions': [{
'action_name': 'generate_embedded_font_data',
@@ -123,21 +127,13 @@
'SK_EMBEDDED_FONTS=<(embedded_font_data_identifier)',
],
}, 'skia_no_fontconfig', {
- 'link_settings': {
- 'libraries': [
- '-ldl',
- ],
- },
- 'sources': [
- '../include/ports/SkFontMgr_custom.h',
- '../src/ports/SkFontMgr_custom.cpp',
- ],
'sources/': [['include', '../src/ports/SkFontMgr_custom_directory_factory.cpp']],
+ }, 'skia_os == "android"', {
+ 'sources/': [['include', '../src/ports/SkFontMgr_android_factory.cpp']],
}, {
'link_settings': {
'libraries': [
'-lfontconfig',
- '-ldl',
],
},
'sources': [
@@ -157,9 +153,6 @@
'../src/ports/SkFontHost_mac.cpp',
'../src/utils/mac/SkStream_mac.cpp',
],
- 'sources!': [
- '../src/ports/SkFontHost_tables.cpp',
- ],
}],
[ 'skia_os == "ios"', {
'include_dirs': [
@@ -170,9 +163,6 @@
'../src/ports/SkFontHost_mac.cpp',
'../src/utils/mac/SkStream_mac.cpp',
],
- 'sources!': [
- '../src/ports/SkFontHost_tables.cpp',
- ],
}],
[ 'skia_os == "win"', {
'include_dirs': [
@@ -214,12 +204,6 @@
],
'sources': [
'../src/ports/SkDebug_android.cpp',
- '../src/ports/SkFontMgr_android.cpp',
- '../src/ports/SkFontMgr_android_parser.cpp',
- ],
- 'sources/': [['include', '../src/ports/SkFontMgr_android_factory.cpp']],
- 'dependencies': [
- 'android_deps.gyp:expat',
],
}],
],
« no previous file with comments | « no previous file | gyp/tests.gypi » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698