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

Unified Diff: skia/BUILD.gn

Issue 2599933002: [Android] Stop using skia globals for font overriding in layout tests. (Closed)
Patch Set: Created 4 years 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
Index: skia/BUILD.gn
diff --git a/skia/BUILD.gn b/skia/BUILD.gn
index da4eb2e7a95f7048164ef60afadf261bebf6f5c9..c6c1ab4afa6fae9f2bddd2d2cd9e1ebb104d2e7a 100644
--- a/skia/BUILD.gn
+++ b/skia/BUILD.gn
@@ -197,6 +197,7 @@ component("skia") {
"ext/benchmarking_canvas.cc",
"ext/convolver.cc",
"ext/event_tracer_impl.cc",
+ "ext/fontmgr_default_android.cc",
"ext/fontmgr_default_linux.cc",
"ext/fontmgr_default_win.cc",
"ext/google_logging.cc",
@@ -243,7 +244,6 @@ component("skia") {
"//third_party/skia/src/ports/SkFontHost_win.cpp",
"//third_party/skia/src/ports/SkFontMgr_FontConfigInterface.cpp",
"//third_party/skia/src/ports/SkFontMgr_android.cpp",
- "//third_party/skia/src/ports/SkFontMgr_android_factory.cpp",
"//third_party/skia/src/ports/SkFontMgr_android_parser.cpp",
"//third_party/skia/src/ports/SkFontMgr_win_dw.cpp",
"//third_party/skia/src/ports/SkGlobalInitialization_default.cpp",
@@ -338,13 +338,6 @@ component("skia") {
"//third_party/skia/src/ports/SkTypeface_win_dw.cpp",
]
}
- if (!is_android) {
- sources -= [
- "//third_party/skia/src/ports/SkFontMgr_android.cpp",
- "//third_party/skia/src/ports/SkFontMgr_android_factory.cpp",
- "//third_party/skia/src/ports/SkFontMgr_android_parser.cpp",
- ]
- }
if (!is_mac) {
sources -= [ "//third_party/skia/src/ports/SkFontHost_mac.cpp" ]
}
@@ -358,19 +351,15 @@ component("skia") {
]
}
- # Add the files for the SkFontMgr_Android. This is used to emulate android
- # fonts on linux. See content/zygote/zygote_main_linux.cc
- if (is_linux) {
- sources += [
- "//third_party/skia/src/ports/SkFontMgr_android.cpp",
- "//third_party/skia/src/ports/SkFontMgr_android_parser.cpp",
- ]
- }
-
if (!is_linux && !is_android) {
sources -= [
"//third_party/skia/src/ports/SkFontHost_FreeType.cpp",
"//third_party/skia/src/ports/SkFontHost_FreeType_common.cpp",
+
+ # Retain the files for the SkFontMgr_Android on linux to emulate android
+ # fonts. See content/zygote/zygote_main_linux.cc
+ "//third_party/skia/src/ports/SkFontMgr_android.cpp",
+ "//third_party/skia/src/ports/SkFontMgr_android_parser.cpp",
]
}

Powered by Google App Engine
This is Rietveld 408576698