WIP: Added support for giflib, updated jpeg and png

platform_tools/android/gyp/dependencies.gypi

+# Copyright 2015 Google Inc.
+#
+# Use of this source code is governed by a BSD-style license that can be
+# found in the LICENSE file.
+
 # This GYP file stores the dependencies necessary to build Skia on the Android
 # platform. The OS doesn't provide many stable libraries as part of the
 # distribution so we have to build a few of them ourselves.
 #
 # NOTE: We tried adding the gyp file to the android/ directory at the root of
 # the Skia repo, but that resulted in the generated makefiles being created
 # outside of the out directory. We may be able to move the bulk of this gyp
 # to the /android directory and put a simple shim here, but that has yet to be
 # tested.
 
@@ skipping 37 matching lines @@
       'defines': [
         'HAVE_EXPAT_CONFIG_H',
       ],
       'direct_dependent_settings': {
         'include_dirs': [
           '../third_party/externals/expat/lib',  # For expat.h
         ],
       }
     },
     {
-      'target_name': 'gif',
-      'type': 'static_library',
-      'sources': [
-        '../third_party/externals/gif/dgif_lib.c',
-        '../third_party/externals/gif/gifalloc.c',
-        '../third_party/externals/gif/gif_err.c',
-      ],
-      'include_dirs': [
-        '../third_party/externals/gif',
-      ],
-      'cflags': [
-        '-w',
-        '-DHAVE_CONFIG_H',
-      ],
-      'direct_dependent_settings': {
-        'include_dirs': [
-          '../third_party/externals/gif',
-        ],
-      }
-    },
-    {
       'target_name': 'png',
       'type': 'static_library',
+      'conditions': [
+        [ 'arm_neon',
+          {
+            'sources' : [
+              '../third_party/externals/png/arm/arm_init.c',
+              '../third_party/externals/png/arm/filter_neon.S',
+              '../third_party/externals/png/arm/filter_neon_intrinsics.c',
+            ]
+          },
+        ],
+      ],
       'sources': [
         '../third_party/externals/png/png.c',
         '../third_party/externals/png/pngerror.c',
-        '../third_party/externals/png/pnggccrd.c',
         '../third_party/externals/png/pngget.c',
         '../third_party/externals/png/pngmem.c',
         '../third_party/externals/png/pngpread.c',
         '../third_party/externals/png/pngread.c',
         '../third_party/externals/png/pngrio.c',
         '../third_party/externals/png/pngrtran.c',
         '../third_party/externals/png/pngrutil.c',
         '../third_party/externals/png/pngset.c',
         '../third_party/externals/png/pngtrans.c',
-        '../third_party/externals/png/pngvcrd.c',
         '../third_party/externals/png/pngwio.c',
         '../third_party/externals/png/pngwrite.c',
         '../third_party/externals/png/pngwtran.c',
         '../third_party/externals/png/pngwutil.c',
       ],
       'include_dirs': [
         '../third_party/externals/png',
       ],
       'cflags': [
         '-w',
         '-fvisibility=hidden',
       ],
       'link_settings': {
         'libraries': [
           '-lz',
         ],
       },
       'direct_dependent_settings': {
         'include_dirs': [
           '../third_party/externals/png',
         ],
       }
     },
     {
       'target_name': 'jpeg',
       'type': 'static_library',
+      'conditions': [
+        [ 'arm_neon',
+          {
+            'sources' : [
+              '../third_party/externals/jpeg/jsimd_arm_neon.S',
+              '../third_party/externals/jpeg/jsimd_neon.c',
+            ]
+          },
+        ],
+        [ 'skia_arch_type == "mips"',
+          {
+            'sources' : [
+              '../third_party/externals/jpeg/mips_jidctfst.c',
+              '../third_party/externals/jpeg/mips_idct_le.S',
+            ]
+          },
+        ],
+        [ 'skia_arch_type in [ "x86", "x86_64" ]',

[scroggo, 2015/03/26 21:15:02]

This could also be:

'"x86" in skia_arch_type'

Not sure if that's better.

[msarett, 2015/03/27 14:01:23]

Done.

+          {
+            'sources' : [
+              '../third_party/externals/jpeg/jidctintelsse.c',
+            ]
+          },
+        ],
+      ],
       'sources': [
+        '../third_party/externals/jpeg/armv6_idct.S',
         '../third_party/externals/jpeg/jcapimin.c',
         '../third_party/externals/jpeg/jcapistd.c',
         '../third_party/externals/jpeg/jccoefct.c',
         '../third_party/externals/jpeg/jccolor.c',
         '../third_party/externals/jpeg/jcdctmgr.c',
         '../third_party/externals/jpeg/jchuff.c',
         '../third_party/externals/jpeg/jcinit.c',
         '../third_party/externals/jpeg/jcmainct.c',
         '../third_party/externals/jpeg/jcmarker.c',
         '../third_party/externals/jpeg/jcmaster.c',
@@ skipping 21 matching lines @@
         '../third_party/externals/jpeg/jdsample.c',
         '../third_party/externals/jpeg/jdtrans.c',
         '../third_party/externals/jpeg/jerror.c',
         '../third_party/externals/jpeg/jfdctflt.c',
         '../third_party/externals/jpeg/jfdctfst.c',
         '../third_party/externals/jpeg/jfdctint.c',
         '../third_party/externals/jpeg/jidctflt.c',
         '../third_party/externals/jpeg/jidctfst.c',
         '../third_party/externals/jpeg/jidctint.c',
         '../third_party/externals/jpeg/jidctred.c',
+        '../third_party/externals/jpeg/jmem-android.c',
+        '../third_party/externals/jpeg/jmemmgr.c',
         '../third_party/externals/jpeg/jquant1.c',
         '../third_party/externals/jpeg/jquant2.c',
         '../third_party/externals/jpeg/jutils.c',
-        '../third_party/externals/jpeg/jmemmgr.c',
-        '../third_party/externals/jpeg/jmem-android.c', # ashmem is also available
       ],
       'include_dirs': [
         '../third_party/externals/jpeg',
       ],
       'cflags': [
         '-w',
         '-fvisibility=hidden',
         '-DAVOID_TABLES',
         '-O3',
         '-fstrict-aliasing',
@@ skipping 57 matching lines @@
           }],
         ],
         'sources': [
           '../app/jni/com_skia_SkiaSampleRenderer.cpp',
         ],
       },
 
     },
   ]
 }