Expand the workaround for Strict Mode issues on L Consumer.

Expand the workaround for Strict Mode issues on L Consumer.

Expands the code section that allows I/O during
ChromeActivity#setContentView. Now I/O is also allowed during the resource
inflation of the toolbar container.

This works around Strict Mode violations we're seeing on tests running on
the L Consumer configuration.

BUG=639352
Committed: https://crrev.com/a1235d7c247cbb55a06f648cda088272fb3c8729
Cr-Commit-Position: refs/heads/master@{#413838}

[Donn Denman, 2016-08-23 00:36:22]

donnd@chromium.org changed reviewers:
+ tedchoc@chromium.org

[Donn Denman, 2016-08-23 00:36:22]

Ted, PTAL.

I'm thinking we should also verify that there are not additional problems
lurking.  I need to get a consumer L device to run the tests on to verify,
right?

[Donn Denman, 2016-08-23 16:19:52]

[cc Peter]

[Peter Wen, 2016-08-23 17:36:45]

wnwen@chromium.org changed reviewers:
+ wnwen@chromium.org

[Peter Wen, 2016-08-23 17:36:46]

lgtm

Unfortunately this is out of our control.

https://codereview.chromium.org/2270563002/diff/1/chrome/android/java/src/org...
File chrome/android/java/src/org/chromium/chrome/browser/ChromeActivity.java
(right):

https://codereview.chromium.org/2270563002/diff/1/chrome/android/java/src/org...
chrome/android/java/src/org/chromium/chrome/browser/ChromeActivity.java:346: //
On certain android configurations this setup sequence results in disk writes
outside
Mind changing this to "On certain android devices..."?

[Donn Denman, 2016-08-23 17:50:51]

Ted, I'm thinking it's easiest to just try this to see if it works on the bots,
OK?

https://codereview.chromium.org/2270563002/diff/1/chrome/android/java/src/org...
File chrome/android/java/src/org/chromium/chrome/browser/ChromeActivity.java
(right):

https://codereview.chromium.org/2270563002/diff/1/chrome/android/java/src/org...
chrome/android/java/src/org/chromium/chrome/browser/ChromeActivity.java:346: //
On certain android configurations this setup sequence results in disk writes
outside
On 2016/08/23 17:36:46, Peter Wen wrote:
> Mind changing this to "On certain android devices..."?

Done.

[Ted C, 2016-08-23 18:13:48]

lgtm

[Donn Denman, 2016-08-23 19:35:07]

The CQ bit was checked by donnd@chromium.org

[Donn Denman, 2016-08-23 19:35:08]

The patchset sent to the CQ was uploaded after l-g-t-m from wnwen@chromium.org
Link to the patchset: https://chromiumcodereview.appspot.com/2270563002/#ps20001
(title: "Just changed a comment.")

[commit-bot: I haz the power, 2016-08-23 19:35:45]

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, 2016-08-23 21:19:41]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-08-23 21:23:11]

Description was changed from

==========
Expand the workaround for Strict Mode issues on L Consumer.

Expands the code section that allows I/O during
ChromeActivity#setContentView. Now I/O is also allowed during the resource
inflation of the toolbar container.

This works around Strict Mode violations we're seeing on tests running on
the L Consumer configuration.

BUG=639352
==========

to

==========
Expand the workaround for Strict Mode issues on L Consumer.

Expands the code section that allows I/O during
ChromeActivity#setContentView. Now I/O is also allowed during the resource
inflation of the toolbar container.

This works around Strict Mode violations we're seeing on tests running on
the L Consumer configuration.

BUG=639352

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

[commit-bot: I haz the power, 2016-08-23 21:23:12]

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