Commit e3546a69 authored by Andrew Grieve's avatar Andrew Grieve Committed by Commit Bot

apk_operations.py: Don't auto-grant app permissions

Doing so is a bit surprising, and makes local development not match what
users see.

Bug: 1027123
Change-Id: I5f05de15b75c0ba0bf5d022ae9be50405415df99
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2069544
Auto-Submit: Andrew Grieve <agrieve@chromium.org>
Reviewed-by: default avatarPeter Wen <wnwen@chromium.org>
Commit-Queue: Peter Wen <wnwen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#743997}
parent 8e26ee77
...@@ -66,9 +66,9 @@ def _Colorize(text, style=''): ...@@ -66,9 +66,9 @@ def _Colorize(text, style=''):
def _InstallApk(devices, apk, install_dict): def _InstallApk(devices, apk, install_dict):
def install(device): def install(device):
if install_dict: if install_dict:
installer.Install(device, install_dict, apk=apk) installer.Install(device, install_dict, apk=apk, permissions=[])
else: else:
device.Install(apk, allow_downgrade=True, reinstall=True) device.Install(apk, permissions=[], allow_downgrade=True, reinstall=True)
logging.info('Installing %sincremental apk.', '' if install_dict else 'non-') logging.info('Installing %sincremental apk.', '' if install_dict else 'non-')
device_utils.DeviceUtils.parallel(devices).pMap(install) device_utils.DeviceUtils.parallel(devices).pMap(install)
...@@ -151,6 +151,7 @@ def _InstallBundle(devices, apk_helper_instance, package_name, ...@@ -151,6 +151,7 @@ def _InstallBundle(devices, apk_helper_instance, package_name,
device.Install( device.Install(
apk_helper_instance, apk_helper_instance,
permissions=[],
modules=modules, modules=modules,
fake_modules=fake_modules, fake_modules=fake_modules,
allow_downgrade=True) allow_downgrade=True)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment