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

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

Issue 9696018: Update pyauto tests to work with the uber page. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: rebase Created 8 years, 9 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 | « chrome/test/functional/policy.py ('k') | chrome/test/functional/special_tabs.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/test/functional/policy_prefs_ui.py
===================================================================
--- chrome/test/functional/policy_prefs_ui.py (revision 126676)
+++ chrome/test/functional/policy_prefs_ui.py (working copy)
@@ -31,21 +31,16 @@
"""Tests policies and their impact on the prefs UI."""
settings_pages = [
- 'chrome://settings/browser',
- 'chrome://settings/personal',
- 'chrome://settings/advanced',
- 'chrome://settings/searchEngines',
- 'chrome://settings/passwords',
- 'chrome://settings/autofill',
- 'chrome://settings/content',
- 'chrome://settings/languages',
- 'chrome://settings/extensions',
+ 'chrome://settings-frame',
+ 'chrome://settings-frame/searchEngines',
+ 'chrome://settings-frame/passwords',
+ 'chrome://settings-frame/autofill',
+ 'chrome://settings-frame/content',
+ 'chrome://settings-frame/languages',
]
if pyauto.PyUITest.IsChromeOS():
settings_pages += [
- 'chrome://settings/system',
- 'chrome://settings/internet',
- 'chrome://settings/accounts',
+ 'chrome://settings-frame/accounts',
]
def GetPlatform(self):
@@ -62,6 +57,8 @@
def IsBannerVisible(self):
"""Returns true if the managed-banner is visible in the current page."""
+ # TODO(csilv): This logic assumes there is only one banner, it needs to be
+ # updated to work with uber page.
ret = self.ExecuteJavascript("""
var visible = false;
var banner = document.getElementById('managed-prefs-banner');
« no previous file with comments | « chrome/test/functional/policy.py ('k') | chrome/test/functional/special_tabs.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698