Merge code which extracts assets from APK to file

Merge code which extracts assets from APK to file

BUG=609122
Committed: https://crrev.com/6efc98a62239ae265d3a643a8e5fffa4ee30358a
Cr-Commit-Position: refs/heads/master@{#396620}

[pkotwicz, 2016-05-24 20:54:54]

pkotwicz@chromium.org changed reviewers:
+ yfriedman@chromium.org

[pkotwicz, 2016-05-24 20:54:54]

Yaron, can you please take a look?

[Yaron, 2016-05-24 21:36:27]

https://codereview.chromium.org/2006703004/diff/1/base/android/java/src/org/c...
File base/android/java/src/org/chromium/base/FileUtils.java (right):

https://codereview.chromium.org/2006703004/diff/1/base/android/java/src/org/c...
base/android/java/src/org/chromium/base/FileUtils.java:46: try (InputStream
inputStream = context.getAssets().open(assetName);
unfortunately we can't use try-with-resources because we support older versions

[pkotwicz, 2016-05-25 15:38:27]

Yaron, can you please take another look? I am not using try-with-resources
anymore

[Yaron, 2016-05-25 18:39:29]

lgtm

[pkotwicz, 2016-05-25 19:43:40]

pkotwicz@chromium.org changed reviewers:
+ mef@chromium.org, michaelbai@chromium.org

[pkotwicz, 2016-05-25 19:43:41]

michaelbai@ for android_webview/
mef@ for components/cronet

[michaelbai, 2016-05-25 20:00:57]

android_webview LGTM

[pkotwicz, 2016-05-26 23:10:21]

mef@ Ping!

[mef, 2016-05-27 00:44:35]

cronet/ lgtm

[pkotwicz, 2016-05-27 15:20:39]

The CQ bit was checked by pkotwicz@chromium.org

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

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

[commit-bot: I haz the power, 2016-05-27 15:28:18]

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

[commit-bot: I haz the power, 2016-05-27 15:28:18]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[pkotwicz, 2016-05-27 17:51:52]

Description was changed from

==========
Merge code which extracts assets from APK to file

BUG=609122
==========

to

==========
Merge code which extracts assets from APK to file

BUG=609122
NO_PRESUBMIT=true
==========

[pkotwicz, 2016-05-27 17:52:15]

Description was changed from

==========
Merge code which extracts assets from APK to file

BUG=609122
NO_PRESUBMIT=true
==========

to

==========
Merge code which extracts assets from APK to file

BUG=609122
==========

[pkotwicz, 2016-05-27 18:52:56]

The CQ bit was checked by pkotwicz@chromium.org

[pkotwicz, 2016-05-27 18:52:57]

The patchset sent to the CQ was uploaded after l-g-t-m from
michaelbai@chromium.org, yfriedman@chromium.org, mef@chromium.org
Link to the patchset: https://codereview.chromium.org/2006703004/#ps40001
(title: " ")

[pkotwicz, 2016-05-27 18:53:05]

The CQ bit was unchecked by pkotwicz@chromium.org

[pkotwicz, 2016-05-27 18:54:11]

Patchset #3 (id:40001) has been deleted

[pkotwicz, 2016-05-27 18:54:14]

The CQ bit was checked by pkotwicz@chromium.org

[pkotwicz, 2016-05-27 18:54:15]

The patchset sent to the CQ was uploaded after l-g-t-m from
michaelbai@chromium.org, yfriedman@chromium.org, mef@chromium.org
Link to the patchset: https://codereview.chromium.org/2006703004/#ps60001
(title: " ")

[commit-bot: I haz the power, 2016-05-27 18:54:40]

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

[commit-bot: I haz the power, 2016-05-27 21:28:04]

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

[commit-bot: I haz the power, 2016-05-27 21:28:05]

Try jobs failed on following builders:
  linux_android_rel_ng on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[pkotwicz, 2016-05-28 00:27:37]

The CQ bit was checked by pkotwicz@chromium.org

[commit-bot: I haz the power, 2016-05-28 00:27:55]

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

[commit-bot: I haz the power, 2016-05-28 01:33:28]

Description was changed from

==========
Merge code which extracts assets from APK to file

BUG=609122
==========

to

==========
Merge code which extracts assets from APK to file

BUG=609122
==========

[commit-bot: I haz the power, 2016-05-28 01:33:29]

Committed patchset #3 (id:60001)

[commit-bot: I haz the power, 2016-05-28 01:37:17]

Description was changed from

==========
Merge code which extracts assets from APK to file

BUG=609122
==========

to

==========
Merge code which extracts assets from APK to file

BUG=609122

Committed: https://crrev.com/6efc98a62239ae265d3a643a8e5fffa4ee30358a
Cr-Commit-Position: refs/heads/master@{#396620}
==========

[commit-bot: I haz the power, 2016-05-28 01:37:18]

Patchset 3 (id:??) landed as
https://crrev.com/6efc98a62239ae265d3a643a8e5fffa4ee30358a
Cr-Commit-Position: refs/heads/master@{#396620}