| 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 a574778a6ca01b9c743221342436977986d6f31a..0a6f36a390861b104012e0dff8bf35b8b5e0934e 100644
|
| --- a/tools/telemetry/telemetry/core/backends/chrome/cros_browser_backend.py
|
| +++ b/tools/telemetry/telemetry/core/backends/chrome/cros_browser_backend.py
|
| @@ -143,33 +143,27 @@ class CrOSBrowserBackend(chrome_browser_backend.ChromeBrowserBackend):
|
| https=None,
|
| dns=None), forwarding_flag='L')
|
|
|
| - try:
|
| - self._WaitForBrowserToComeUp(wait_for_extensions=False)
|
| - self._PostBrowserStartupInitialization()
|
| - except:
|
| - import traceback
|
| - traceback.print_exc()
|
| - self.Close()
|
| - raise
|
| -
|
| + # Wait for oobe.
|
| + self._WaitForBrowserToComeUp(wait_for_extensions=False)
|
| + self._PostBrowserStartupInitialization()
|
| util.WaitFor(lambda: self.oobe_exists, 10)
|
|
|
| if self.browser_options.auto_login:
|
| - if self._is_guest:
|
| - pid = self.pid
|
| - self.oobe.NavigateGuestLogin()
|
| - # Guest browsing shuts down the current browser and launches an
|
| - # incognito browser in a separate process, which we need to wait for.
|
| - util.WaitFor(lambda: pid != self.pid, 10)
|
| - elif self.browser_options.gaia_login:
|
| - try:
|
| + try:
|
| + if self._is_guest:
|
| + pid = self.pid
|
| + self.oobe.NavigateGuestLogin()
|
| + # Guest browsing shuts down the current browser and launches an
|
| + # incognito browser in a separate process, which we need to wait for.
|
| + util.WaitFor(lambda: pid != self.pid, 10)
|
| + elif self.browser_options.gaia_login:
|
| self.oobe.NavigateGaiaLogin(self._username, self._password)
|
| - except util.TimeoutException:
|
| - self._cri.TakeScreenShot('gaia-login')
|
| - raise
|
| - else:
|
| - self.oobe.NavigateFakeLogin(self._username, self._password)
|
| - self._WaitForLogin()
|
| + else:
|
| + self.oobe.NavigateFakeLogin(self._username, self._password)
|
| + self._WaitForLogin()
|
| + except util.TimeoutException:
|
| + self._cri.TakeScreenShot('login-screen')
|
| + raise exceptions.LoginException('Timed out going through login screen')
|
|
|
| logging.info('Browser is up!')
|
|
|
| @@ -244,19 +238,11 @@ class CrOSBrowserBackend(chrome_browser_backend.ChromeBrowserBackend):
|
| util.WaitFor(self._IsCryptohomeMounted, 30)
|
| return
|
|
|
| - try:
|
| - util.WaitFor(self._IsLoggedIn, 60)
|
| - except util.TimeoutException:
|
| - self._cri.TakeScreenShot('login-screen')
|
| - raise exceptions.LoginException('Timed out going through login screen')
|
| + # Wait for cryptohome to mount.
|
| + util.WaitFor(self._IsLoggedIn, 60)
|
|
|
| # Wait for extensions to load.
|
| - try:
|
| - self._WaitForBrowserToComeUp()
|
| - except util.TimeoutException:
|
| - logging.error('Chrome args: %s' % self._cri.GetChromeProcess()['args'])
|
| - self._cri.TakeScreenShot('extension-timeout')
|
| - raise
|
| + self._WaitForBrowserToComeUp()
|
|
|
| # Workaround for crbug.com/329271, crbug.com/334726.
|
| retries = 3
|
|
|