[devil] Always set permissions on install

[devil] Always set permissions on install

Grant required app permissions on apk Install, regardless of whether
the apk was already installed or not.

This caused some issues where developers manually installed apks and
then test runners would fail due to the missing permissions.

BUG=chromium:732724
Review-Url: https://codereview.chromium.org/2963303002
Committed: https://chromium.googlesource.com/external/github.com/catapult-project/catapult/+/c8957d6c0705d7d93063430b911b453bfa3356bf

[perezju, 2017-06-30 14:50:51]

perezju@chromium.org changed reviewers:
+ jbudorick@chromium.org, primiano@chromium.org

[perezju, 2017-06-30 14:50:51]

Also verified that this fixes the issue on crbug.com/732724; the tests now would
have run as expected.

[Primiano Tucci (use gerrit), 2017-07-03 09:57:49]

LGTM thanks

[jbudorick, 2017-07-05 16:05:53]

lgtm

[perezju, 2017-07-05 16:19:52]

The CQ bit was checked by perezju@chromium.org

[commit-bot: I haz the power, 2017-07-05 16:20:03]

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-05 17:19:28]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1499271592955650, "parent_rev":
"90b65c4c0e2199cc37d061006064e25546fc37ce", "commit_rev":
"c8957d6c0705d7d93063430b911b453bfa3356bf"}

[commit-bot: I haz the power, 2017-07-05 17:19:31]

Description was changed from

==========
[devil] Always set permissions on install

Grant required app permissions on apk Install, regardless of whether
the apk was already installed or not.

This caused some issues where developers manually installed apks and
then test runners would fail due to the missing permissions.

BUG=chromium:732724
==========

to

==========
[devil] Always set permissions on install

Grant required app permissions on apk Install, regardless of whether
the apk was already installed or not.

This caused some issues where developers manually installed apks and
then test runners would fail due to the missing permissions.

BUG=chromium:732724

Review-Url: https://codereview.chromium.org/2963303002
Committed:
https://chromium.googlesource.com/external/github.com/catapult-project/catapu...
==========

[commit-bot: I haz the power, 2017-07-05 17:19:32]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/external/github.com/catapult-project/catapu...

[jbudorick, 2017-07-05 22:50:14]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/2967283002/ by jbudorick@chromium.org.

The reason for reverting is: breaking a few different suites:
https://bugs.chromium.org/p/chromium/issues/detail?id=739531.