summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorBruce Dawson <brucedawson@chromium.org>2017-08-10 16:07:07 -0700
committerChromium commit bot <commit-bot@chromium.org>2017-08-10 23:43:09 +0000
commitd27998f6526272a5b8732106aa9b75f724434aca (patch)
tree2caa5881be9a41fd9d87e2bbfa105aa0c58ecb6c
parentfdc3acb42a7983053c53d1445c62654d7ef6b3b2 (diff)
downloadpdfium-chromium/3184.tar.xz
Fix component builds for FreeType changeschromium/3184chromium/3183
FreeType has changed so it doesn't define ft_adobe_glyph_list once for every translation unit, which avoids wasting space in the data segment with duplicates. However Chromium component builds were depending on this behavior. This fixes component builds while avoiding duplicate arrays in non-component builds. This change makes no difference to pdfium at the moment but will probably be needed when it gains support for component builds. Change-Id: Ieed7c711bdbd3fa6eef19da60e906667dab8fb1b Reviewed-on: https://pdfium-review.googlesource.com/10670 Commit-Queue: Bruce Dawson <brucedawson@chromium.org> Reviewed-by: Lei Zhang <thestig@chromium.org>
-rw-r--r--BUILD.gn5
1 files changed, 5 insertions, 0 deletions
diff --git a/BUILD.gn b/BUILD.gn
index 9dc2362aa6..cd2b0b05db 100644
--- a/BUILD.gn
+++ b/BUILD.gn
@@ -1070,6 +1070,11 @@ static_library("fxge") {
]
defines = [ "DEFINE_PS_TABLES" ]
+ if (is_component_build) {
+ # ft_adobe_glyph_list is not exported from the Freetype shared library so we
+ # need it defined in component builds.
+ defines += [ "DEFINE_PS_TABLES_DATA" ]
+ }
if (pdf_enable_xfa) {
sources += [