Added libjpeg and zlib to SDK

Added libjpeg and zlib to SDK

These libraries are already in Chrome's third party, so we just add
a library.dsc to build them.  They will eventually need to be converted
to GYP file along with the rest of the NaCl SDK gyp work.

R=binji@chromium.org
BUG=162141
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=171031

[noelallen1, 2012-11-26 23:13:22]

ptal

[binji, 2012-11-26 23:23:20]

https://codereview.chromium.org/11417098/diff/1/native_client_sdk/src/build_t...
File native_client_sdk/src/build_tools/build_sdk.py (right):

https://codereview.chromium.org/11417098/diff/1/native_client_sdk/src/build_t...
native_client_sdk/src/build_tools/build_sdk.py:577: #  'libpng',
remove

https://codereview.chromium.org/11417098/diff/1/native_client_sdk/src/librari...
File native_client_sdk/src/libraries/libjpeg/library.dsc (right):

https://codereview.chromium.org/11417098/diff/1/native_client_sdk/src/librari...
native_client_sdk/src/libraries/libjpeg/library.dsc:34: 'jdct.h',
put headers in 'HEADERS': {...}

https://codereview.chromium.org/11417098/diff/1/native_client_sdk/src/librari...
File native_client_sdk/src/libraries/zlib/library.dsc (right):

https://codereview.chromium.org/11417098/diff/1/native_client_sdk/src/librari...
native_client_sdk/src/libraries/zlib/library.dsc:24: 'crc32.h',
put headers in 'HEADERS': {...}

https://codereview.chromium.org/11417098/diff/1/native_client_sdk/src/librari...
native_client_sdk/src/libraries/zlib/library.dsc:41: 'mixed-source.patch',
I took a look at this patch -- there are more changes here than I expected. It's
hard to tell if it is compatible with standard zlib. Maybe we'd want to ship a
cleaner zlib than this...?

[binji, 2012-11-26 23:23:58]

We also might want them to be experimental for now, until we have some tests.

[noelallen1, 2012-11-27 23:52:24]

Added experimental

https://codereview.chromium.org/11417098/diff/1/native_client_sdk/src/build_t...
File native_client_sdk/src/build_tools/build_sdk.py (right):

https://codereview.chromium.org/11417098/diff/1/native_client_sdk/src/build_t...
native_client_sdk/src/build_tools/build_sdk.py:577: #  'libpng',
On 2012/11/26 23:23:20, binji wrote:
> remove

Done.

https://codereview.chromium.org/11417098/diff/1/native_client_sdk/src/librari...
File native_client_sdk/src/libraries/libjpeg/library.dsc (right):

https://codereview.chromium.org/11417098/diff/1/native_client_sdk/src/librari...
native_client_sdk/src/libraries/libjpeg/library.dsc:34: 'jdct.h',
On 2012/11/26 23:23:20, binji wrote:
> put headers in 'HEADERS': {...}

Done.

https://codereview.chromium.org/11417098/diff/1/native_client_sdk/src/librari...
File native_client_sdk/src/libraries/zlib/library.dsc (right):

https://codereview.chromium.org/11417098/diff/1/native_client_sdk/src/librari...
native_client_sdk/src/libraries/zlib/library.dsc:24: 'crc32.h',
On 2012/11/26 23:23:20, binji wrote:
> put headers in 'HEADERS': {...}

Done.

https://codereview.chromium.org/11417098/diff/1/native_client_sdk/src/librari...
native_client_sdk/src/libraries/zlib/library.dsc:41: 'mixed-source.patch',
Hmm...  I don't think we want to check out another copy of zlib, that would just
confuse things...  I think our other option is to not ship.

On 2012/11/26 23:23:20, binji wrote:
> I took a look at this patch -- there are more changes here than I expected.
It's
> hard to tell if it is compatible with standard zlib. Maybe we'd want to ship a
> cleaner zlib than this...?

[binji, 2012-11-28 22:03:03]

lgtm

[commit-bot: I haz the power, 2012-12-03 21:09:00]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/noelallen@chromium.org/11417098/10002

[commit-bot: I haz the power, 2012-12-03 22:48:22]

Retried try job too often on win_rel for step(s) browser_tests

[commit-bot: I haz the power, 2012-12-04 18:04:24]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/noelallen@chromium.org/11417098/10002

[commit-bot: I haz the power, 2012-12-04 20:10:14]

Change committed as 171031