Creating a new wrapper for gif decoder

gyp/giflib.gyp

Index: gyp/giflib.gyp
diff --git a/gyp/giflib.gyp b/gyp/giflib.gyp
index 57b3c276546e0890403be73b957304e91fdde2c5..e03e559f8c05c4272fb1cd0213f1f982ce35a6b4 100644
--- a/gyp/giflib.gyp
+++ b/gyp/giflib.gyp
@@ -14,11 +14,10 @@
         [ 'skia_android_framework == 0',
           {
             'type': 'static_library',
-            'defines': [
-              'HAVE_CONFIG_H',
-            ],
             'include_dirs': [
               '../third_party/externals/giflib',
+              # Used to include a dummy unistd.h file for windows
+                '../third_party/giflib',
             ],
             'dependencies': [
             ],
@@ -35,6 +34,17 @@
                 '-w'
               ],
             },
+            'msvs_settings': {
+              'VCCLCompilerTool': {
+                'AdditionalOptions': [
+                  '/w',
+                ],
+                'AdditionalOptions!': [
+                  # Remove the default warning flag
+                  '/W3',

[scroggo, 2015/03/31 18:39:56]

I guess this did not work?

[msarett, 2015/03/31 20:11:40]

Unfortunately not.  I did a grep for "W3" and it doesn't appear that we suggest
that flag in any of our gyp files.  This makes me think that somehow we
automatically use this flag on windows bots?  Which maybe explains why I was
unable to turn it off?  Do you think this is worth exploring further?

[scroggo, 2015/03/31 20:39:11]

I would say no, until someone complains about it.

+                ],
+              },
+            },
             'sources': [
               '../third_party/externals/giflib/dgif_lib.c',
               '../third_party/externals/giflib/gifalloc.c',