Update the .pak file format to version 2: Make resource ids 16bit

Update the .pak file format to version 2: Make resource ids 16bit
values instead of 32bit values.

On gzip compressed locales files, this saves 17k total.

Bump the cached theme pak version so the caches get rebuilt.

BUG=76285
TEST=Covered by unit tests and manually tested installing themes.

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=95487

[tony, 2011-08-03 18:36:04]

The gfx_unittest failures are a lie.  The changes to sample.pak didn't get
applied because it's a binary file.

[Evan Martin, 2011-08-03 18:43:09]

LGTM

http://codereview.chromium.org/7555003/diff/1/chrome/browser/themes/browser_t...
File chrome/browser/themes/browser_theme_pack.cc (right):

http://codereview.chromium.org/7555003/diff/1/chrome/browser/themes/browser_t...
chrome/browser/themes/browser_theme_pack.cc:35: // int32_t. kHeaderID should
always have the maximum value because we want the
uint16

http://codereview.chromium.org/7555003/diff/1/tools/data_pack/data_pack.py
File tools/data_pack/data_pack.py (right):

http://codereview.chromium.org/7555003/diff/1/tools/data_pack/data_pack.py#ne...
tools/data_pack/data_pack.py:26: print input_file
Probably didn't mean to leave this in

http://codereview.chromium.org/7555003/diff/1/tools/data_pack/data_pack.py#ne...
tools/data_pack/data_pack.py:49: index_length = len(ids) * (2 + 2 * 4)
Too bad this is duplicated with the above code.  Not your fault though.

[tony, 2011-08-04 17:42:53]

Elliot, did you want to take a quick look?

http://codereview.chromium.org/7555003/diff/1/chrome/browser/themes/browser_t...
File chrome/browser/themes/browser_theme_pack.cc (right):

http://codereview.chromium.org/7555003/diff/1/chrome/browser/themes/browser_t...
chrome/browser/themes/browser_theme_pack.cc:35: // int32_t. kHeaderID should
always have the maximum value because we want the
On 2011/08/03 18:43:09, Evan Martin wrote:
> uint16

Fixed

http://codereview.chromium.org/7555003/diff/1/tools/data_pack/data_pack.py
File tools/data_pack/data_pack.py (right):

http://codereview.chromium.org/7555003/diff/1/tools/data_pack/data_pack.py#ne...
tools/data_pack/data_pack.py:26: print input_file
On 2011/08/03 18:43:09, Evan Martin wrote:
> Probably didn't mean to leave this in

Oops, removed.

[Elliot Glaysher, 2011-08-04 17:52:50]

LGTM (just did a quick pass; nothing in depth. thank you for remembering to
update theme pack version number.)