Reland of Add deduplication logic to .pak files

Reland of Add deduplication logic to .pak files

Revert was here:
https://codereview.chromium.org/2989443002/

Reason for reland:
No changes from origina. Bot issue fixed by:
https://chromium-review.googlesource.com/c/589888/

Now, when multiple entries contain the same content, multiple
table-of-contents entries will be created, but only one data region.

As of now, en-US.pak has ~3200 entries, and 350 of them are duplicates.

For MonochromePublic.apk, shrinks uncompressed .pak size by ~130kb, and
compressed .pak size by 32kb.

TBR=flackr@chromium.org,sadrul@chromium.org,achuith@chromium.org
BUG=738566
Review-Url: https://codereview.chromium.org/2984383002
Cr-Commit-Position: refs/heads/master@{#490501}
Committed: https://chromium.googlesource.com/chromium/src/+/4735fae6caad85ab08433367108d3b96c0f7d646

[agrieve, 2017-07-28 17:54:36]

Created Reland of Add deduplication logic to .pak files

[agrieve, 2017-07-28 17:55:44]

Description was changed from

==========
Reland of Add deduplication logic to .pak files (patchset #1 id:1 of
https://codereview.chromium.org/2989443002/ )

Reason for revert:
sys.path fixed up in:
https://chromium-review.googlesource.com/c/589888/

Original issue's description:
> Revert of Add deduplication logic to .pak files (patchset #10 id:180001 of
https://codereview.chromium.org/2969123002/ )
> 
> Reason for revert:
> https://bugs.chromium.org/p/chromium/issues/detail?id=747171
> 
> Original issue's description:
> > Add deduplication logic to .pak files
> > 
> > Now, when multiple entries contain the same content, multiple
> > table-of-contents entries will be created, but only one data region.
> > 
> > As of now, en-US.pak has ~3200 entries, and 350 of them are duplicates.
> > 
> > For MonochromePublic.apk, shrinks uncompressed .pak size by ~130kb, and
> > compressed .pak size by 32kb.
> > 
> > BUG=738566
> > 
> > Review-Url: https://codereview.chromium.org/2969123002
> > Cr-Commit-Position: refs/heads/master@{#488215}
> > Committed:
https://chromium.googlesource.com/chromium/src/+/ade347f539a378fb37500f6d17e8...
> 
> TBR=flackr@chromium.org,sadrul@chromium.org,agrieve@chromium.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=738566
> 
> Review-Url: https://codereview.chromium.org/2989443002
> Cr-Commit-Position: refs/heads/master@{#488554}
> Committed:
https://chromium.googlesource.com/chromium/src/+/59f00a7d06b43ae4127637baaf7d...

TBR=flackr@chromium.org,sadrul@chromium.org,achuith@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=738566
==========

to

==========
Reland of Add deduplication logic to .pak files

Reverted:
(patchset #1 id:1 of https://codereview.chromium.org/2989443002/ )

Reason for reland:
No changes from origina. Bot issue fixed by:
https://chromium-review.googlesource.com/c/589888/

Now, when multiple entries contain the same content, multiple
table-of-contents entries will be created, but only one data region.

As of now, en-US.pak has ~3200 entries, and 350 of them are duplicates.

For MonochromePublic.apk, shrinks uncompressed .pak size by ~130kb, and
compressed .pak size by 32kb.

TBR=flackr@chromium.org,sadrul@chromium.org,achuith@chromium.org
BUG=738566
==========

[agrieve, 2017-07-28 17:56:14]

Description was changed from

==========
Reland of Add deduplication logic to .pak files

Reverted:
(patchset #1 id:1 of https://codereview.chromium.org/2989443002/ )

Reason for reland:
No changes from origina. Bot issue fixed by:
https://chromium-review.googlesource.com/c/589888/

Now, when multiple entries contain the same content, multiple
table-of-contents entries will be created, but only one data region.

As of now, en-US.pak has ~3200 entries, and 350 of them are duplicates.

For MonochromePublic.apk, shrinks uncompressed .pak size by ~130kb, and
compressed .pak size by 32kb.

TBR=flackr@chromium.org,sadrul@chromium.org,achuith@chromium.org
BUG=738566
==========

to

==========
Reland of Add deduplication logic to .pak files

Revert was here:
https://codereview.chromium.org/2989443002/

Reason for reland:
No changes from origina. Bot issue fixed by:
https://chromium-review.googlesource.com/c/589888/

Now, when multiple entries contain the same content, multiple
table-of-contents entries will be created, but only one data region.

As of now, en-US.pak has ~3200 entries, and 350 of them are duplicates.

For MonochromePublic.apk, shrinks uncompressed .pak size by ~130kb, and
compressed .pak size by 32kb.

TBR=flackr@chromium.org,sadrul@chromium.org,achuith@chromium.org
BUG=738566
==========

[agrieve, 2017-07-28 17:56:22]

The CQ bit was checked by agrieve@chromium.org

[commit-bot: I haz the power, 2017-07-28 17:56:49]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-07-28 20:39:13]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1501264582039130, "parent_rev":
"3c3b814be7218c3985d1eb2b36cc6ca75fb33d86", "commit_rev":
"4735fae6caad85ab08433367108d3b96c0f7d646"}

[commit-bot: I haz the power, 2017-07-28 20:39:39]

Description was changed from

==========
Reland of Add deduplication logic to .pak files

Revert was here:
https://codereview.chromium.org/2989443002/

Reason for reland:
No changes from origina. Bot issue fixed by:
https://chromium-review.googlesource.com/c/589888/

Now, when multiple entries contain the same content, multiple
table-of-contents entries will be created, but only one data region.

As of now, en-US.pak has ~3200 entries, and 350 of them are duplicates.

For MonochromePublic.apk, shrinks uncompressed .pak size by ~130kb, and
compressed .pak size by 32kb.

TBR=flackr@chromium.org,sadrul@chromium.org,achuith@chromium.org
BUG=738566
==========

to

==========
Reland of Add deduplication logic to .pak files

Revert was here:
https://codereview.chromium.org/2989443002/

Reason for reland:
No changes from origina. Bot issue fixed by:
https://chromium-review.googlesource.com/c/589888/

Now, when multiple entries contain the same content, multiple
table-of-contents entries will be created, but only one data region.

As of now, en-US.pak has ~3200 entries, and 350 of them are duplicates.

For MonochromePublic.apk, shrinks uncompressed .pak size by ~130kb, and
compressed .pak size by 32kb.

TBR=flackr@chromium.org,sadrul@chromium.org,achuith@chromium.org
BUG=738566

Review-Url: https://codereview.chromium.org/2984383002
Cr-Commit-Position: refs/heads/master@{#490501}
Committed:
https://chromium.googlesource.com/chromium/src/+/4735fae6caad85ab08433367108d...
==========

[commit-bot: I haz the power, 2017-07-28 20:39:41]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/4735fae6caad85ab08433367108d...