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

Unified Diff: chrome/test/data/webui/settings/languages_page_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/languages_page_browsertest.js
diff --git a/chrome/test/data/webui/settings/languages_page_browsertest.js b/chrome/test/data/webui/settings/languages_page_browsertest.js
index 8869a965fe05e78811c96ad370dc73497f3cda30..ea269e62f798e378c3761327ed661a1c380913bb 100644
--- a/chrome/test/data/webui/settings/languages_page_browsertest.js
+++ b/chrome/test/data/webui/settings/languages_page_browsertest.js
@@ -43,12 +43,15 @@ GEN('#endif');
// Runs languages page tests.
TEST_F('SettingsLanguagesPageBrowserTest', 'MAYBE_LanguagesPage', function() {
+ var self = this;
+
+ suiteSetup(function() {
+ self.toggleAdvanced();
+ });
+
suite('languages page', function() {
testing.Test.disableAnimationsAndTransitions();
- this.toggleAdvanced();
- var page = this.getPage('basic');
-
var languagesSection;
var languagesPage;
var languagesCollapse;
@@ -81,6 +84,7 @@ TEST_F('SettingsLanguagesPageBrowserTest', 'MAYBE_LanguagesPage', function() {
}
suiteSetup(function() {
+ var page = self.basicPage;
page.set('pageVisibility.languages', true);
Polymer.dom.flush();
« no previous file with comments | « chrome/test/data/webui/settings/help_page_browsertest.js ('k') | chrome/test/data/webui/settings/on_startup_browsertest.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698