From b3a788e5e37955620b26be5b6e3048d37b605e00 Mon Sep 17 00:00:00 2001
From: John Abd-El-Malek <jam@chromium.org>
Date: Sun, 1 Feb 2015 16:19:29 -0800
Subject: Fix GN PDFium build when building all.

The problem was FT_INTERNAL_DEBUG_H wasn't being defined because FT2_BUILD_LIBRARY wasn't set. So even if PDFium isn't using bundled FreeType, the FreeType target needs FT2_BUILD_LIBRARY defined. The GYP build was already correct.

BUG=453844
TBR=brettw

Review URL: https://codereview.chromium.org/891993003
---
 BUILD.gn             | 4 ----
 pdfium.gyp           | 5 -----
 third_party/BUILD.gn | 1 +
 3 files changed, 1 insertion(+), 9 deletions(-)

diff --git a/BUILD.gn b/BUILD.gn
index b52913b977..988ca7ee6f 100644
--- a/BUILD.gn
+++ b/BUILD.gn
@@ -43,10 +43,6 @@ config("pdfium_config") {
       "/wd4345",
     ]
   }
-
-  if (bundle_freetype) {
-    defines += [ "FT2_BUILD_LIBRARY" ]
-  }
 }
 
 static_library("pdfium") {
diff --git a/pdfium.gyp b/pdfium.gyp
index 2b21e9d9aa..7b6ac0c094 100644
--- a/pdfium.gyp
+++ b/pdfium.gyp
@@ -23,11 +23,6 @@
       ['pdf_use_skia==1', {
         'defines': ['_SKIA_SUPPORT_'],
       }],
-      ['bundle_freetype==1', {
-        'defines' : [
-          'FT2_BUILD_LIBRARY',
-        ],
-      }],
       ['OS=="linux"', {
         'conditions': [
           ['target_arch=="x64"', {
diff --git a/third_party/BUILD.gn b/third_party/BUILD.gn
index e6340c6401..df02938a93 100644
--- a/third_party/BUILD.gn
+++ b/third_party/BUILD.gn
@@ -36,6 +36,7 @@ static_library("freetype") {
       "//third_party/pdfium:pdfium_config",
       "//build/config/compiler:no_chromium_code",
   ]
+  defines = [ "FT2_BUILD_LIBRARY" ]
   sources = [
     "freetype/include/freetype.h",
     "freetype/include/ft2build.h",
-- 
cgit v1.2.3