summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorBruce Dawson <brucedawson@chromium.org>2017-08-12 21:45:12 -0700
committerChromium commit bot <commit-bot@chromium.org>2017-08-13 05:52:17 +0000
commitaeac95192c106b41c058c8372aa2ec8d0092f60b (patch)
treec717a0babd7be74f2b8bd27a3fd0edc357bdea73
parentd3002340caad8d688d0e4928a36e90d354a797bd (diff)
downloadpdfium-aeac95192c106b41c058c8372aa2ec8d0092f60b.tar.xz
Fix use_system_freetype builds for FreeType changeschromium/3185
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 builds with use_system_freetype true were depending on this behavior. This fixes these builds while avoiding duplicate arrays in other builds. This change makes no difference to pdfium itself, but is necessary when it is pulled in to Chromium. This is a follow-on to a previous change that fixed this issue for component builds. Change-Id: I3e68d7b778475e4a2781d77cd413e1bc22f82830 Reviewed-on: https://pdfium-review.googlesource.com/10830 Commit-Queue: Bruce Dawson <brucedawson@chromium.org> Reviewed-by: Lei Zhang <thestig@chromium.org>
-rw-r--r--BUILD.gn5
1 files changed, 3 insertions, 2 deletions
diff --git a/BUILD.gn b/BUILD.gn
index cd2b0b05db..846e8a5ec8 100644
--- a/BUILD.gn
+++ b/BUILD.gn
@@ -2,6 +2,7 @@
# Use of this source code is governed by a BSD-style license that can be
# found in the LICENSE file.
+import("//build/config/freetype/freetype.gni")
import("//testing/test.gni")
import("pdfium.gni")
@@ -1070,9 +1071,9 @@ static_library("fxge") {
]
defines = [ "DEFINE_PS_TABLES" ]
- if (is_component_build) {
+ if (is_component_build || use_system_freetype) {
# ft_adobe_glyph_list is not exported from the Freetype shared library so we
- # need it defined in component builds.
+ # need it defined in component builds and builds using system freetype.
defines += [ "DEFINE_PS_TABLES_DATA" ]
}