Add libpoppler PDF rasterizer

gyp/freetype.gyp

 {
   'targets': [
     {
       'target_name': 'freetype',
       'type': 'none',
       'conditions': [
         [ 'skia_os in ["linux", "freebsd", "openbsd", "solaris", "chromeos"]', {
           'direct_dependent_settings': {
             'include_dirs' : [
               '/usr/include/freetype2',
@@ skipping 28 matching lines @@
       ],
     },
     {
       'target_name': 'freetype_static',
       'type': 'static_library',
       'standalone_static_library': 1,
       'dependencies': [
         # we are dependent upon PNG for color emoji glyphs
         'images.gyp:images'
       ],
-      'sources': [
-        # base components (required)
-        '../third_party/externals/freetype/src/base/ftsystem.c',
-        '../third_party/externals/freetype/src/base/ftinit.c',
-        '../third_party/externals/freetype/src/base/ftdebug.c',
-        '../third_party/externals/freetype/src/base/ftbase.c',
-
-        '../third_party/externals/freetype/src/base/ftbbox.c',       # recommended, see <freetype/ftbbox.h>
-        '../third_party/externals/freetype/src/base/ftglyph.c',      # recommended, see <freetype/ftglyph.h>
-
-        '../third_party/externals/freetype/src/base/ftbitmap.c',     # optional, see <freetype/ftbitmap.h>
-        '../third_party/externals/freetype/src/base/ftfstype.c',     # optional
-        '../third_party/externals/freetype/src/base/ftgasp.c',       # optional, see <freetype/ftgasp.h>
-        '../third_party/externals/freetype/src/base/ftlcdfil.c',     # optional, see <freetype/ftlcdfil.h>
-        '../third_party/externals/freetype/src/base/ftmm.c',         # optional, see <freetype/ftmm.h>
-        '../third_party/externals/freetype/src/base/ftpatent.c',     # optional
-        '../third_party/externals/freetype/src/base/ftstroke.c',     # optional, see <freetype/ftstroke.h>
-        '../third_party/externals/freetype/src/base/ftsynth.c',      # optional, see <freetype/ftsynth.h>
-        '../third_party/externals/freetype/src/base/fttype1.c',      # optional, see <freetype/t1tables.h>
-        '../third_party/externals/freetype/src/base/ftwinfnt.c',     # optional, see <freetype/ftwinfnt.h>
-        '../third_party/externals/freetype/src/base/ftxf86.c',       # optional, see <freetype/ftxf86.h>
-
-        # font drivers (optional; at least one is needed)
-        '../third_party/externals/freetype/src/cff/cff.c',           # CFF/OpenType font driver
-        '../third_party/externals/freetype/src/sfnt/sfnt.c',         # SFNT files support (TrueType & OpenType)
-        '../third_party/externals/freetype/src/truetype/truetype.c', # TrueType font driver
-
-        # rasterizers (optional; at least one is needed for vector formats)
-        '../third_party/externals/freetype/src/raster/raster.c',     # monochrome rasterizer
-        '../third_party/externals/freetype/src/smooth/smooth.c',     # anti-aliasing rasterizer
-
-        # auxiliary modules (optional)
-        '../third_party/externals/freetype/src/autofit/autofit.c',   # auto hinting module
-        '../third_party/externals/freetype/src/pshinter/pshinter.c', # PS hinting module
-        '../third_party/externals/freetype/src/psnames/psnames.c',   # PostScript glyph names support
+      'includes': [
+        # common freetype sources needed for both the base Skia build and the
+        # libpoppler build for testing only
+        'freetype.gypi',
       ],
       'include_dirs': [
-        '../third_party/freetype/include_overrides',
         '../third_party/externals/freetype/internal',
         '../third_party/externals/freetype/builds',
         '../third_party/externals/freetype/include',
         '../third_party/externals/freetype',
       ],
-      'cflags': [
-        '-DFT2_BUILD_LIBRARY',
+      'defines': [
+        'FT2_BUILD_LIBRARY',
       ],
       'direct_dependent_settings': {
         'include_dirs': [
           '../third_party/freetype/include_overrides',
           '../third_party/externals/freetype/include',
         ],
       },
       'conditions': [
         [ 'skia_warnings_as_errors', {
           'cflags!': [
@@ skipping 13 matching lines @@
             '-Wall',
             '-fPIC',
             '-DPIC',
             '-DDARWIN_NO_CARBON',
             '-DFT2_BUILD_LIBRARY',
             '-O2',
           ],
         }],
       ],
     },
+    {
+      'target_name': 'freetype_poppler',
+      'type': 'static_library',
+      'standalone_static_library': 1,
+      'includes': [
+        'freetype.gypi',
+      ],
+      'sources': [
+        # additional components used by poppler
+        '../third_party/externals/freetype/src/base/ftbdf.c',
+        '../third_party/externals/freetype/src/base/ftpfr.c',
+
+        '../third_party/externals/freetype/src/bdf/bdf.c',
+        '../third_party/externals/freetype/src/cid/type1cid.c',
+        '../third_party/externals/freetype/src/pcf/pcf.c',
+        '../third_party/externals/freetype/src/pfr/pfr.c',
+        '../third_party/externals/freetype/src/psaux/psaux.c',
+        '../third_party/externals/freetype/src/type1/type1.c',
+        '../third_party/externals/freetype/src/type42/type42.c',
+        '../third_party/externals/freetype/src/winfonts/winfnt.c',
+
+        '../third_party/externals/freetype/src/gzip/ftgzip.c',
+        '../third_party/externals/freetype/src/lzw/ftlzw.c',
+      ],
+      'include_dirs': [

[vandebo (ex-Chrome), 2013/08/15 23:17:14]

Pull the other common things into the gypi:

include_dirs, defines, direct_dependent_settings, conditions

[ducky, 2013/08/16 02:18:52]

Done.

+        '../third_party/externals/freetype/internal',
+        '../third_party/externals/freetype/builds',
+        '../third_party/externals/freetype/include',
+        '../third_party/externals/freetype/include/freetype/config',
+        '../third_party/externals/freetype',
+      ],
+      'defines': [
+        'FT2_BUILD_LIBRARY',
+        'FT_CONFIG_CONFIG_H=<ftconfig.h>',

[vandebo (ex-Chrome), 2013/08/15 23:17:14]

Why does the poppler version need these and the old version doesn't?

[ducky, 2013/08/16 02:18:52]

Good question. Apparently it doesn't.

+        'FT_CONFIG_MODULES_H=<ftmodule.h>',
+      ],
+      'direct_dependent_settings': {
+        'defines': [
+          'FT_CONFIG_CONFIG_H=<ftconfig.h>',
+          'FT_CONFIG_MODULES_H=<ftmodule.h>',
+        ],
+        'include_dirs': [
+          '../third_party/externals/freetype/include',
+          '../third_party/externals/freetype/include/freetype/config',
+        ],
+      },
+      'conditions': [
+        [ 'skia_warnings_as_errors', {
+          'cflags!': [
+            '-Werror',
+          ],
+        }],
+        [ 'skia_os == "mac"', {
+          'sources': [
+            '../third_party/externals/freetype/src/base/ftmac.c',        # only on the Macintosh
+          ],
+        }],
+      ],
+    },
   ],
 }
 
 # Local Variables:
 # tab-width:2
 # indent-tabs-mode:nil
 # End:
 # vim: set expandtab tabstop=2 shiftwidth=2: