Make permission errors non-fatal

Some devices disallow setting app permissions via adb. This CL makes
sure that we still can run benchmarks on them.

Bug: 427152872
Change-Id: Ibf502b2d44a43457ce394ee39facb37d36c5e210
Reviewed-on: https://chromium-review.googlesource.com/c/crossbench/+/6663058
Reviewed-by: Camillo Bruni <[email protected]>
Commit-Queue: Mikhail Khokhlov <[email protected]>

Author: khokhlov962

crossbench/browsers/chromium/webdriver.py

@@ -27,6 +27,7 @@
 from crossbench.helper.path_finder import ChromiumBuildBinaryFinder
 from crossbench.parse import NumberParser
 from crossbench.plt.android_adb import AndroidAdbPlatform
+from crossbench.plt.base import SubprocessError
 from crossbench.plt.bin import Binaries
 from crossbench.plt.chromeos_ssh import ChromeOsSshPlatform
 from crossbench.plt.linux_ssh import LinuxSshPlatform
@@ -113,7 +114,7 @@ def _start_driver(self, session: BrowserSessionRunGroup,
     self.adb_force_stop()
     if session.browser.wipe_system_user_data:
       self.adb_force_clear()
-      self.platform.adb.grant_permissions(self.android_package)
+      self._setup_binary_permissions()
     self._backup_chrome_flags()
     return self._start_chromedriver(session, driver_path)
 
@@ -187,7 +188,10 @@ def _setup_binary(self) -> None:  # pytype: disable=override-error
     self._setup_binary_permissions()
 
   def _setup_binary_permissions(self) -> None:
-    self.platform.adb.grant_permissions(self.android_package)
+    try:
+      self.platform.adb.grant_permissions(self.android_package)
+    except SubprocessError as e:
+      logging.warning("Error setting app permissions: %s", e)
 
   @override
   def _setup_window(self) -> None:  # pytype: disable=override-error