Fix Mac fontconfig build

gyp/gm.gyp

 # GYP file to build the "gm" (golden master) executable.
 {
   'includes': [
     'apptype_console.gypi',
   ],
   'targets': [
     {
       'target_name': 'gm_expectations',
       'type': 'static_library',
       'include_dirs' : [
@@ skipping 55 matching lines @@
           'defines': [
             'SK_BUILD_NATIVE_PDF_RENDERER',
           ],
           'include_dirs' : [
             '../experimental/PdfViewer/inc',
           ],
           'dependencies': [
             'pdfviewer_lib.gyp:pdfviewer_lib',
           ],
         }],
-        ['skia_os in ["linux", "mac", "win"]', {
+        ['skia_os in ["linux", "win"] or (skia_os == "mac" and skia_mac_poppler)', {

[epoger, 2014/01/08 19:40:22]

I think it would be clearer to use a single flag (e.g. skia_enable_poppler) for
the conditional, and (elsewhere) set its default value based on skia_os.

Like how skia_arch_width is automatically set (in common_variables.gypi)
depending on skia_os, but can be overridden from the command line...

This may be what Ben is referring to in
https://codereview.chromium.org/113203003/diff/60001/gyp/common_variables.gyp...
, but I'm not sure.

[vandebo (ex-Chrome), 2014/01/08 21:24:15]

Changed to skia_enable_popper and made that work across all platforms.

[epoger, 2014/01/08 21:38:12]

Why do we still need the list of operating systems here?  Can't you just check
for skia_poppler_enabled , and set the default for that variable based on the
OS?

[vandebo (ex-Chrome), 2014/01/08 22:08:18]

Done.

           'dependencies': [
             'poppler.gyp:libpoppler-cpp-gpl',
           ],
           'sources': [
             '../src/utils/SkPDFRasterizer.cpp',
           ],
           'defines': [
             'SK_BUILD_POPPLER',
           ],
         }],
@@ skipping 21 matching lines @@
             '../src/gpu',
           ],
           'dependencies': [
             'gputest.gyp:skgputest',
           ],
         }],
       ],
     },
   ],
 }