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

Unified Diff: chrome/test/data/webui/settings/on_startup_browsertest.js

Issue 2754563002: MD Settings: Lazy load the contents of the "advanced" settings. (Closed)
Patch Set: Address comments. Created 3 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
Index: chrome/test/data/webui/settings/on_startup_browsertest.js
diff --git a/chrome/test/data/webui/settings/on_startup_browsertest.js b/chrome/test/data/webui/settings/on_startup_browsertest.js
index a3b75cbc9c9b053ffd392aaa91379bca64dade70..76546308ffa179e09c76104aba22b2b9231b2f82 100644
--- a/chrome/test/data/webui/settings/on_startup_browsertest.js
+++ b/chrome/test/data/webui/settings/on_startup_browsertest.js
@@ -28,7 +28,7 @@ OnStartupSettingsBrowserTest.prototype = {
/** @return {Element} */
getPageElement: function(selector) {
- var section = this.getSection(this.getPage('basic'), 'onStartup');
+ var section = this.getSection(this.basicPage, 'onStartup');
assertTrue(!!section);
var module = section.querySelector('settings-on-startup-page');
assertTrue(!!module);
@@ -59,7 +59,7 @@ TEST_F('OnStartupSettingsBrowserTest', 'uiTests', function() {
suite('OnStartupHandler', function() {
suiteSetup(function() {
- self.getPage('basic').set('pageVisibility.onStartup', true);
+ self.basicPage.set('pageVisibility.onStartup', true);
Polymer.dom.flush();
settingsPrefs = document.querySelector('settings-ui').$$(

Powered by Google App Engine
This is Rietveld 408576698