gyp: clean up warnings with clang-3.8

gyp: clean up warnings with clang-3.8

Also: update zlib DEPS.

GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2002233002

Committed: https://skia.googlesource.com/skia/+/3d5942e8a4fe8845c49a7afaf8248c4e9b5b7520

[hal.canary, 2016-05-23 19:52:13]

Description was changed from

==========
gyp: clean up warnings with clang-3.8

.
==========

to

==========
gyp: clean up warnings with clang-3.8

.
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2002233002
==========

[hal.canary, 2016-05-23 19:52:43]

The CQ bit was checked by halcanary@google.com to run a CQ dry run

[commit-bot: I haz the power, 2016-05-23 19:52:54]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2002233002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/2002233002/1

[hal.canary, 2016-05-23 20:00:53]

halcanary@google.com changed reviewers:
+ bungeman@google.com

[hal.canary, 2016-05-23 20:00:53]

ptal

[commit-bot: I haz the power, 2016-05-23 20:07:11]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-05-23 20:07:12]

Dry run: This issue passed the CQ dry run.

[bungeman-skia, 2016-05-23 20:26:33]

Description was changed from

==========
gyp: clean up warnings with clang-3.8

.
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2002233002
==========

to

==========
gyp: clean up warnings with clang-3.8


GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2002233002
==========

[hal.canary, 2016-05-23 20:44:43]

The CQ bit was checked by halcanary@google.com to run a CQ dry run

[commit-bot: I haz the power, 2016-05-23 20:44:50]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2002233002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/2002233002/20001

[hal.canary, 2016-05-23 20:46:21]

Description was changed from

==========
gyp: clean up warnings with clang-3.8


GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2002233002
==========

to

==========
gyp: clean up warnings with clang-3.8

Also: update zlib DEPS.

GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2002233002
==========

[bungeman-skia, 2016-05-23 20:53:10]

Why not? lgtm

https://codereview.chromium.org/2002233002/diff/20001/gyp/zlib.gyp
File gyp/zlib.gyp (right):

https://codereview.chromium.org/2002233002/diff/20001/gyp/zlib.gyp#newcode25
gyp/zlib.gyp:25: '-Wno-shift-negative-value',
It would be nice to comment here that this was already fixed in the zlib
development branch by "Avoid shifts of negative values inflateMark().". It would
also be nice if Chromium hadn't foobared their zlib fork (it isn't really a
mirror). Alas, probably not even worth the comment.

[hal.canary, 2016-05-23 20:59:43]

The CQ bit was unchecked by halcanary@google.com

[hal.canary, 2016-05-23 21:01:45]

The CQ bit was checked by halcanary@google.com

[commit-bot: I haz the power, 2016-05-23 21:01:57]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2002233002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/2002233002/20001

[commit-bot: I haz the power, 2016-05-23 21:02:57]

Description was changed from

==========
gyp: clean up warnings with clang-3.8

Also: update zlib DEPS.

GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2002233002
==========

to

==========
gyp: clean up warnings with clang-3.8

Also: update zlib DEPS.

GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2002233002

Committed:
https://skia.googlesource.com/skia/+/3d5942e8a4fe8845c49a7afaf8248c4e9b5b7520
==========

[commit-bot: I haz the power, 2016-05-23 21:02:58]

Committed patchset #2 (id:20001) as
https://skia.googlesource.com/skia/+/3d5942e8a4fe8845c49a7afaf8248c4e9b5b7520