[Android] Set permissions when clearing app state in test runner platform mode.

[Android] Set permissions when clearing app state in test runner platform mode.

BUG=609226
Committed: https://crrev.com/7b6552417873165924fd7f62cc428fe5e6b12dc1
Cr-Commit-Position: refs/heads/master@{#392871}

[rnephew (Reviews Here), 2016-05-04 20:56:46]

rnephew@chromium.org changed reviewers:
+ jbudorick@chromium.org, mikecase@chromium.org

[rnephew (Reviews Here), 2016-05-04 20:56:47]

https://codereview.chromium.org/1952813002/diff/1/build/android/test_runner.p...
File build/android/test_runner.pydeps (left):

https://codereview.chromium.org/1952813002/diff/1/build/android/test_runner.p...
build/android/test_runner.pydeps:6:
../../third_party/appurify-python/src/appurify/utils.py
Not sure why I had to run this... but I kept getting the following without it:


** Presubmit ERRORS **
File is stale: build/android/test_runner.pydeps
To regenerate, run:

    build/print_python_deps.py --root build/android --output
build/android/test_runner.pydeps build/android/test_runner.py

[jbudorick, 2016-05-04 20:59:45]

https://codereview.chromium.org/1952813002/diff/20001/build/android/test_runn...
File build/android/test_runner.pydeps (left):

https://codereview.chromium.org/1952813002/diff/20001/build/android/test_runn...
build/android/test_runner.pydeps:3:
../../third_party/appurify-python/src/appurify/__init__.py
This probably isn't right?

[rnephew (Reviews Here), 2016-05-04 21:02:05]

On 2016/05/04 20:59:45, jbudorick wrote:
>
https://codereview.chromium.org/1952813002/diff/20001/build/android/test_runn...
> File build/android/test_runner.pydeps (left):
> 
>
https://codereview.chromium.org/1952813002/diff/20001/build/android/test_runn...
> build/android/test_runner.pydeps:3:
> ../../third_party/appurify-python/src/appurify/__init__.py
> This probably isn't right?

See comment in first patch set.. I have no clue what is making it run. Or what
it is for that matter.

[jbudorick, 2016-05-04 21:22:26]

On 2016/05/04 21:02:05, rnephew1 wrote:
> On 2016/05/04 20:59:45, jbudorick wrote:
> >
>
https://codereview.chromium.org/1952813002/diff/20001/build/android/test_runn...
> > File build/android/test_runner.pydeps (left):
> > 
> >
>
https://codereview.chromium.org/1952813002/diff/20001/build/android/test_runn...
> > build/android/test_runner.pydeps:3:
> > ../../third_party/appurify-python/src/appurify/__init__.py
> > This probably isn't right?
> 
> See comment in first patch set.. I have no clue what is making it run. Or what
> it is for that matter.

It's a presubmit agrieve added a month or so ago to determine what .py files are
required by the test runner for gn+isolate.

If you're seeing this happen, it probably means something is up with your local
repos.

[jbudorick, 2016-05-10 23:23:29]

https://codereview.chromium.org/1952813002/diff/40001/build/android/pylib/loc...
File build/android/pylib/local/device/local_device_instrumentation_test_run.py
(right):

https://codereview.chromium.org/1952813002/diff/40001/build/android/pylib/loc...
build/android/pylib/local/device/local_device_instrumentation_test_run.py:272:
self._test_instance.apk_under_test
I think this is backwards.

[rnephew (Reviews Here), 2016-05-10 23:27:33]

https://codereview.chromium.org/1952813002/diff/40001/build/android/pylib/loc...
File build/android/pylib/local/device/local_device_instrumentation_test_run.py
(right):

https://codereview.chromium.org/1952813002/diff/40001/build/android/pylib/loc...
build/android/pylib/local/device/local_device_instrumentation_test_run.py:272:
self._test_instance.apk_under_test
On 2016/05/10 23:23:29, jbudorick wrote:
> I think this is backwards.

Done.

[jbudorick, 2016-05-11 03:04:02]

lgtm

[rnephew (Reviews Here), 2016-05-11 06:25:09]

The CQ bit was checked by rnephew@chromium.org

[commit-bot: I haz the power, 2016-05-11 06:25:19]

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

[commit-bot: I haz the power, 2016-05-11 07:22:24]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-05-11 07:23:59]

Description was changed from

==========
[Android] Set permissions when clearing app state in test runner platform mode.

BUG=609226
==========

to

==========
[Android] Set permissions when clearing app state in test runner platform mode.

BUG=609226

Committed: https://crrev.com/7b6552417873165924fd7f62cc428fe5e6b12dc1
Cr-Commit-Position: refs/heads/master@{#392871}
==========

[commit-bot: I haz the power, 2016-05-11 07:24:00]

Patchset 4 (id:??) landed as
https://crrev.com/7b6552417873165924fd7f62cc428fe5e6b12dc1
Cr-Commit-Position: refs/heads/master@{#392871}