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

Unified Diff: chrome/browser/resources/settings/basic_page/basic_page.js

Issue 2660383002: [MD settings] idle load basic and advanced pages (Closed)
Patch Set: merge with master; changed test name Created 3 years, 11 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/browser/resources/settings/basic_page/basic_page.js
diff --git a/chrome/browser/resources/settings/basic_page/basic_page.js b/chrome/browser/resources/settings/basic_page/basic_page.js
index bc14ed22a91762bfb3a84c109dc936b817fa8baf..82e9add52da31876ba558caf8c8a3136501948d6 100644
--- a/chrome/browser/resources/settings/basic_page/basic_page.js
+++ b/chrome/browser/resources/settings/basic_page/basic_page.js
@@ -29,6 +29,7 @@ Polymer({
advancedToggleExpanded: {
type: Boolean,
notify: true,
+ observer: 'advancedToggleExpandedChanged_',
},
/**
@@ -101,7 +102,7 @@ Polymer({
if (this.pageVisibility.advancedSettings !== false) {
assert(whenSearchDone === settings.getSearchManager().search(
- query, assert(this.$$('#advancedPage'))));
+ query, assert(this.$.advancedPageTemplate.get())));
}
return whenSearchDone;
@@ -131,6 +132,15 @@ Polymer({
},
/**
+ * Render the advanced page now (don't wait for idle).
+ * @private
+ */
+ advancedToggleExpandedChanged_: function() {
+ if (this.advancedToggleExpanded)
+ this.$.advancedPageTemplate.get();
+ },
+
+ /**
* @param {boolean} inSearchMode
* @param {boolean} hasExpandedSection
* @return {boolean}

Powered by Google App Engine
This is Rietveld 408576698