Remove instances of chrome.is_logged_in
In desktopui_AudioFeedback, don't need to wait for browser to come up or check is_logged_in.
Remove is_logged_in, and add a comment to wait_for_browser_to_come_up.
BUG=chromium:233834
TEST=these are the tests
Change-Id: Ib4c5c4d32c00442d43664066b1775ffdf5a0f7bd
Reviewed-on: https://chromium-review.googlesource.com/172633
Reviewed-by: Dean Liao <deanliao@chromium.org>
Commit-Queue: Achuith Bhandarkar <achuith@chromium.org>
Tested-by: Achuith Bhandarkar <achuith@chromium.org>
diff --git a/client/common_lib/cros/chrome.py b/client/common_lib/cros/chrome.py
index a0ed856..c59d3e5 100644
--- a/client/common_lib/cros/chrome.py
+++ b/client/common_lib/cros/chrome.py
@@ -2,8 +2,6 @@
# Use of this source code is governed by a BSD-style license that can be
# found in the LICENSE file.
-import os
-
from telemetry.core import browser_finder, browser_options, exceptions
from telemetry.core import extension_to_load, util
@@ -71,7 +69,9 @@
def wait_for_browser_to_come_up(self):
- """Waits for the browser to come up."""
+ """Waits for the browser to come up. This should only be called after a
+ browser crash.
+ """
def _BrowserReady(cr):
try:
tab = cr.browser.tabs.New()
@@ -96,8 +96,3 @@
return True
return False
-
- def is_logged_in(self):
- """Returns true iff logged in."""
- # TODO(achuith): Do this better.
- return os.path.exists('/var/run/state/logged-in')