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

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

Issue 2690263002: MD Settings: add empty list message for spell-check edit page. (Closed)
Patch Set: fix 80 col Created 3 years, 10 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 99efbc2a2c1e6619fa101f26c94402336f0b41aa..8869a965fe05e78811c96ad370dc73497f3cda30 100644
--- a/chrome/test/data/webui/settings/languages_page_browsertest.js
+++ b/chrome/test/data/webui/settings/languages_page_browsertest.js
@@ -449,23 +449,6 @@ TEST_F('SettingsLanguagesPageBrowserTest', 'MAYBE_LanguagesPage', function() {
0, triggerRow.querySelector('.secondary').textContent.length);
}
});
-
- if (!cr.isMac) {
scottchen 2017/02/15 19:34:46 This is being moved to the spellcheck-specific tes
- test('spellcheck edit dictionary page add word validation', function() {
- assertFalse(!!languagesPage.$$('settings-edit-dictionary-page'));
- var spellCheckCollapse = languagesPage.$.spellCheckCollapse;
- MockInteractions.tap(
- spellCheckCollapse.querySelector('.list-button'));
- var editDictPage = languagesPage.$$('settings-edit-dictionary-page');
- assertTrue(!!editDictPage);
- var addWordButton = editDictPage.root.querySelector('paper-button');
- editDictPage.$.newWord.value = '';
- assertTrue(!!addWordButton);
- assertTrue(addWordButton.disabled);
- editDictPage.$.newWord.value = 'valid word';
- assertFalse(addWordButton.disabled);
- });
- }
}.bind(this));
// TODO(michaelpg): Test more aspects of the languages UI.

Powered by Google App Engine
This is Rietveld 408576698