🐸 Reland of Check for 0-length files in ResourceExtractor

Reland of Check for 0-length files in ResourceExtractor

Reason for reland:
No longer using ZipFile. Hopefully this was the reason
for the dirty memory.

TBR=yfriedman@chromium.org,torne@chromium.org,wnwen@chromium.org,qyearsley@chromium.org
BUG=606413, 607598
Committed: https://crrev.com/f17984d06c7a736aa2cf7aac3d8aa6dc2628ef9a
Cr-Commit-Position: refs/heads/master@{#394866}

[agrieve, 2016-05-19 18:32:11]

Created Reland of 🐸 Check for 0-length files in ResourceExtractor

[agrieve, 2016-05-19 18:55:28]

Description was changed from

==========
Reland of 🐸 Check for 0-length files in ResourceExtractor (patchset #2 id:40001
of https://codereview.chromium.org/1948033004/ )

Reason for revert:
Will update description once changed.

Original issue's description:
> Revert of Check for 0-length files in ResourceExtractor (patchset #3 id:40001
of https://codereview.chromium.org/1920893003/ )
> 
> Reason for revert:
> Possibly regressed memory usage on android (http://crbug.com/607598)
> 
> Speculative revert to see if vm_private_dirty_final_browser values go back
down for android bots on the perf waterfall
> 
> Original issue's description:
> > Check for 0-length files in ResourceExtractor
> >
> > Also moves from an on-disk version timestamp file to SharedPreferences
> > (should be faster).
> >
> > BUG=606413
> >
> > Committed: https://crrev.com/b28607d4c72261657ea9e24ea9195bc23a472b3f
> > Cr-Commit-Position: refs/heads/master@{#390068}
> 
>
TBR=yfriedman@chromium.org,torne@chromium.org,wnwen@chromium.org,agrieve@chro...
> # Not skipping CQ checks because original CL landed more than 1 days ago.
> BUG=606413
> 
> Committed: https://crrev.com/a3f2dd5d896ca693224403ad072cfad70882174b
> Cr-Commit-Position: refs/heads/master@{#393023}

TBR=yfriedman@chromium.org,torne@chromium.org,wnwen@chromium.org,qyearsley@ch...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=606413
==========

to

==========
Reland of Check for 0-length files in ResourceExtractor

Reason for reland:
No longer using ZipFile. Hopefully this was the reason
for the dirty memory.

TBR=yfriedman@chromium.org,torne@chromium.org,wnwen@chromium.org,qyearsley@ch...
BUG=606413,607598
==========

[agrieve, 2016-05-19 18:56:46]

The CQ bit was checked by agrieve@chromium.org

[commit-bot: I haz the power, 2016-05-19 18:57:19]

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

[commit-bot: I haz the power, 2016-05-19 21:17:18]

Description was changed from

==========
Reland of Check for 0-length files in ResourceExtractor

Reason for reland:
No longer using ZipFile. Hopefully this was the reason
for the dirty memory.

TBR=yfriedman@chromium.org,torne@chromium.org,wnwen@chromium.org,qyearsley@ch...
BUG=606413,607598
==========

to

==========
Reland of Check for 0-length files in ResourceExtractor

Reason for reland:
No longer using ZipFile. Hopefully this was the reason
for the dirty memory.

TBR=yfriedman@chromium.org,torne@chromium.org,wnwen@chromium.org,qyearsley@ch...
BUG=606413,607598
==========

[commit-bot: I haz the power, 2016-05-19 21:17:20]

Committed patchset #2 (id:40001)

[commit-bot: I haz the power, 2016-05-19 21:19:07]

Description was changed from

==========
Reland of Check for 0-length files in ResourceExtractor

Reason for reland:
No longer using ZipFile. Hopefully this was the reason
for the dirty memory.

TBR=yfriedman@chromium.org,torne@chromium.org,wnwen@chromium.org,qyearsley@ch...
BUG=606413,607598
==========

to

==========
Reland of Check for 0-length files in ResourceExtractor

Reason for reland:
No longer using ZipFile. Hopefully this was the reason
for the dirty memory.

TBR=yfriedman@chromium.org,torne@chromium.org,wnwen@chromium.org,qyearsley@ch...
BUG=606413,607598

Committed: https://crrev.com/f17984d06c7a736aa2cf7aac3d8aa6dc2628ef9a
Cr-Commit-Position: refs/heads/master@{#394866}
==========

[commit-bot: I haz the power, 2016-05-19 21:19:08]

Patchset 2 (id:??) landed as
https://crrev.com/f17984d06c7a736aa2cf7aac3d8aa6dc2628ef9a
Cr-Commit-Position: refs/heads/master@{#394866}