WIP: Added support for giflib, updated jpeg and png

platform_tools/android/gyp/dependencies.gypi

Index: platform_tools/android/gyp/dependencies.gypi
diff --git a/platform_tools/android/gyp/dependencies.gypi b/platform_tools/android/gyp/dependencies.gypi
index 6507ee45f742f6dbd6ebce160217eaddb8c76df0..ee9e095a4303502673b68d8fd014e0a82afb1feb 100644
--- a/platform_tools/android/gyp/dependencies.gypi
+++ b/platform_tools/android/gyp/dependencies.gypi
@@ -1,3 +1,8 @@
+# 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.
@@ -55,33 +60,11 @@
       }
     },
     {
-      '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',
       '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',
@@ -91,12 +74,25 @@
         '../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',
       ],
+      'conditions': [
+        [ 'arm_neon == 1',
+          {
+            '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',
+            ],
+            'flags' : [

[djsollen, 2015/03/27 14:51:22]

if you use defines you can instead of flags you can drop the -D off the front.

[scroggo, 2015/03/27 14:55:46]

I think this should be

'defines': [
  'PNG_ARM_NEON_OPT=2',
],

Also, I think we generally follow everything with a comma for future proofing.

Not sure whether 'flags' is supported, but you can use 'cflags' or 'cppflags'
(in this case you do need the "-D").

For things that will get used by skia_for_android_framework, I prefer 'defines',
since those are treated differently in that build, but this is not used for the
android framework, so I don't have a strong preference.

[msarett, 2015/03/27 15:06:55]

Done.

+              '-DPNG_ARM_NEON_OPT=2'
+            ]
+          },
+        ],
+      ],
       'include_dirs': [
         '../third_party/externals/png',
       ],
@@ -160,11 +156,46 @@
         '../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
+      ],
+      'conditions': [
+        [ 'arm_neon == 1',
+          {
+            'sources' : [
+              '../third_party/externals/jpeg/armv6_idct.S',
+              '../third_party/externals/jpeg/jsimd_arm_neon.S',
+              '../third_party/externals/jpeg/jsimd_neon.c',
+            ],
+            'flags' : [
+              '-DNV_ARM_NEON'
+            ]
+          },
+        ],
+        [ 'skia_arch_type == "mips" and mips_dsp == 2',
+          {
+            'sources' : [
+              '../third_party/externals/jpeg/mips_jidctfst.c',
+              '../third_party/externals/jpeg/mips_idct_le.S',
+            ],
+            'flags' : [
+              '-DANDROID_MIPS_IDCT'
+            ]
+          },
+        ],
+        [ '"x86" in skia_arch_type',
+          {
+            'sources' : [
+              '../third_party/externals/jpeg/jidctintelsse.c',
+            ],
+            'flags' : [
+              '-DANDROID_INTELSSE2_IDCT'
+            ]
+          },
+        ],
       ],
       'include_dirs': [
         '../third_party/externals/jpeg',