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

Unified Diff: chrome/test/functional/chromeos_time.py

Issue 9539004: Fix testTimezoneIsEditable on chromeos (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 10 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 | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/test/functional/chromeos_time.py
diff --git a/chrome/test/functional/chromeos_time.py b/chrome/test/functional/chromeos_time.py
index bc53c71ae9633b3e51ad845b06ca341656c05f17..9ac7c4d293b6eca6d96be0e1864e3e3968268755 100755
--- a/chrome/test/functional/chromeos_time.py
+++ b/chrome/test/functional/chromeos_time.py
@@ -43,12 +43,12 @@ class ChromeosTime(pyauto.PyUITest):
True, if timezone dropdown is enabled
False, otherwise
"""
- self.NavigateToURL('chrome://settings/system')
+ self.NavigateToURL('chrome://settings-frame/advanced')
ret = self.ExecuteJavascript("""
var enabled = false;
var timezone = document.getElementById('timezone-select');
if (timezone)
- enabled = banner.enabled;
+ enabled = !timezone.disabled;
domAutomationController.send(enabled.toString());
""")
return ret == 'true'
@@ -57,7 +57,7 @@ class ChromeosTime(pyauto.PyUITest):
"""Test that the timezone is always editable."""
# This test only makes sense if we are not running as the owner.
self.assertFalse(self.GetLoginInfo()['is_owner'])
- enabled = _IsTimezoneEditable()
+ enabled = self._IsTimezoneEditable()
self.assertTrue(enabled, msg='Timezone is not editable when not owner.')
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698