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

Side by Side Diff: chrome/test/data/webui/settings/languages_page_browsertest.js

Issue 2255023002: Settings: Remove /advanced usage as URL. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fix Created 4 years, 4 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 unified diff | Download patch
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 /** @fileoverview Suite of tests for settings-languages-page. */ 5 /** @fileoverview Suite of tests for settings-languages-page. */
6 6
7 /** @const {string} Path to root from chrome/test/data/webui/settings/. */ 7 /** @const {string} Path to root from chrome/test/data/webui/settings/. */
8 var ROOT_PATH = '../../../../../'; 8 var ROOT_PATH = '../../../../../';
9 9
10 // Polymer BrowserTest fixture. 10 // Polymer BrowserTest fixture.
11 GEN_INCLUDE( 11 GEN_INCLUDE(
12 [ROOT_PATH + 'chrome/test/data/webui/polymer_browser_test_base.js']); 12 [ROOT_PATH + 'chrome/test/data/webui/polymer_browser_test_base.js']);
13 // SettingsPageBrowserTest fixture. 13 // SettingsPageBrowserTest fixture.
14 GEN_INCLUDE([ROOT_PATH + 14 GEN_INCLUDE([ROOT_PATH +
15 'chrome/test/data/webui/settings/settings_page_browsertest.js']); 15 'chrome/test/data/webui/settings/settings_page_browsertest.js']);
16 16
17 var languageSettings = languageSettings || {}; 17 var languageSettings = languageSettings || {};
18 18
19 /** 19 /**
20 * Test class for settings-languages-page UI. 20 * Test class for settings-languages-page UI.
21 * @constructor 21 * @constructor
22 * @extends {SettingsPageBrowserTest} 22 * @extends {SettingsPageBrowserTest}
23 */ 23 */
24 function SettingsLanguagesPageBrowserTest() {} 24 function SettingsLanguagesPageBrowserTest() {}
25 25
26 SettingsLanguagesPageBrowserTest.prototype = { 26 SettingsLanguagesPageBrowserTest.prototype = {
27 __proto__: SettingsPageBrowserTest.prototype, 27 __proto__: SettingsPageBrowserTest.prototype,
28 28
29 /** @override */ 29 /** @override */
30 browsePreload: 'chrome://md-settings/advanced',
31
32 /** @override */
33 preLoad: function() { 30 preLoad: function() {
34 SettingsPageBrowserTest.prototype.preLoad.call(this); 31 SettingsPageBrowserTest.prototype.preLoad.call(this);
35 settingsHidePagesByDefaultForTest = true; 32 settingsHidePagesByDefaultForTest = true;
36 }, 33 },
37 }; 34 };
38 35
39 // May time out on debug builders and memory bots because 36 // May time out on debug builders and memory bots because
40 // the Settings page can take several seconds to load in a Release build 37 // the Settings page can take several seconds to load in a Release build
41 // and several times that in a Debug build. See https://crbug.com/558434. 38 // and several times that in a Debug build. See https://crbug.com/558434.
42 GEN('#if defined(MEMORY_SANITIZER) || !defined(NDEBUG)'); 39 GEN('#if defined(MEMORY_SANITIZER) || !defined(NDEBUG)');
43 GEN('#define MAYBE_LanguagesPage DISABLED_LanguagesPage'); 40 GEN('#define MAYBE_LanguagesPage DISABLED_LanguagesPage');
44 GEN('#else'); 41 GEN('#else');
45 GEN('#define MAYBE_LanguagesPage LanguagesPage'); 42 GEN('#define MAYBE_LanguagesPage LanguagesPage');
46 GEN('#endif'); 43 GEN('#endif');
47 44
48 // Runs languages page tests. 45 // Runs languages page tests.
49 TEST_F('SettingsLanguagesPageBrowserTest', 'MAYBE_LanguagesPage', function() { 46 TEST_F('SettingsLanguagesPageBrowserTest', 'MAYBE_LanguagesPage', function() {
50 suite('languages page', function() { 47 suite('languages page', function() {
51 testing.Test.disableAnimationsAndTransitions(); 48 testing.Test.disableAnimationsAndTransitions();
52 49
50 this.toggleAdvanced();
53 var advanced = this.getPage('advanced'); 51 var advanced = this.getPage('advanced');
54 52
55 var languagesSection; 53 var languagesSection;
56 var languagesPage; 54 var languagesPage;
57 var languageHelper; 55 var languageHelper;
58 suiteSetup(function() { 56 suiteSetup(function() {
59 advanced.set('pageVisibility.languages', true); 57 advanced.set('pageVisibility.languages', true);
60 Polymer.dom.flush(); 58 Polymer.dom.flush();
61 59
62 languagesSection = this.getSection(advanced, 'languages'); 60 languagesSection = this.getSection(advanced, 'languages');
(...skipping 74 matching lines...) Expand 10 before | Expand all | Expand 10 after
137 assertTrue(!!languagesPage.$$('settings-edit-dictionary-page')); 135 assertTrue(!!languagesPage.$$('settings-edit-dictionary-page'));
138 } 136 }
139 }); 137 });
140 }.bind(this)); 138 }.bind(this));
141 139
142 // TODO(michaelpg): Test more aspects of the languages UI. 140 // TODO(michaelpg): Test more aspects of the languages UI.
143 141
144 // Run all registered tests. 142 // Run all registered tests.
145 mocha.run(); 143 mocha.run();
146 }); 144 });
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698