Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(343)

Unified Diff: tools/telemetry/telemetry/core/backends/chrome/cros_browser_backend.py

Issue 246353004: Remove files in /var/lib/whitelist before attempting login with telemetry. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Default false Created 6 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « tools/telemetry/telemetry/core/backends/chrome/chrome_browser_options.py ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/telemetry/telemetry/core/backends/chrome/cros_browser_backend.py
diff --git a/tools/telemetry/telemetry/core/backends/chrome/cros_browser_backend.py b/tools/telemetry/telemetry/core/backends/chrome/cros_browser_backend.py
index 5c8db7138c13308df3e1d589a1208443e24275dc..606f9aba8dc083703ce4702d2464b52901d07fc0 100644
--- a/tools/telemetry/telemetry/core/backends/chrome/cros_browser_backend.py
+++ b/tools/telemetry/telemetry/core/backends/chrome/cros_browser_backend.py
@@ -47,8 +47,6 @@ class CrOSBrowserBackend(chrome_browser_backend.ChromeBrowserBackend):
self._remote_debugging_port = self._cri.GetRemotePort()
self._port = self._remote_debugging_port
- self._SetBranchNumber(self._GetChromeVersion())
-
# Copy extensions to temp directories on the device.
# Note that we also perform this copy locally to ensure that
# the owner of the extensions is set to chronos.
@@ -59,9 +57,11 @@ class CrOSBrowserBackend(chrome_browser_backend.ChromeBrowserBackend):
cri.Chown(extension_dir)
e.local_path = os.path.join(extension_dir, os.path.basename(e.path))
- # Ensure the UI is running and logged out.
- self._RestartUI()
- util.WaitFor(self.IsBrowserRunning, 20)
+ self._cri.RunCmdOnDevice(['stop', 'ui'])
+
+ if self.browser_options.clear_enterprise_policy:
+ self._cri.RmRF('/var/lib/whitelist/*')
+ self._cri.RmRF('/home/chronos/Local\ State')
# Delete test user's cryptohome vault (user data directory).
if not self.browser_options.dont_override_profile:
@@ -73,6 +73,11 @@ class CrOSBrowserBackend(chrome_browser_backend.ChromeBrowserBackend):
self.profile_directory)
cri.Chown(self.profile_directory)
+ self._cri.RunCmdOnDevice(['start', 'ui'])
+ util.WaitFor(self.IsBrowserRunning, 20)
+
+ self._SetBranchNumber(self._GetChromeVersion())
+
def GetBrowserStartupArgs(self):
args = super(CrOSBrowserBackend, self).GetBrowserStartupArgs()
args.extend([
« no previous file with comments | « tools/telemetry/telemetry/core/backends/chrome/chrome_browser_options.py ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698