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

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

Issue 2573643005: Language settings: Fix Restart and Remove buttons shown at wrong times (Closed)
Patch Set: feedback Created 4 years 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
« no previous file with comments | « chrome/browser/ui/webui/settings/languages_handler.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/test/data/webui/settings/languages_tests.js
diff --git a/chrome/test/data/webui/settings/languages_tests.js b/chrome/test/data/webui/settings/languages_tests.js
index f334126277bc3508d25671ed92d3890b4c633d80..07fa7054bb779ac2b083a2cb1f62d8c9a21d8011 100644
--- a/chrome/test/data/webui/settings/languages_tests.js
+++ b/chrome/test/data/webui/settings/languages_tests.js
@@ -90,12 +90,23 @@ cr.define('settings-languages', function() {
languageSettingsPrivate.setSettingsPrefs(settingsPrefs);
settings.languageSettingsPrivateApiForTest = languageSettingsPrivate;
+ var getProspectiveUILanguageCalled = false;
+ registerMessageCallback('getProspectiveUILanguage', null,
+ function(callbackId) {
+ assertFalse(getProspectiveUILanguageCalled);
+ getProspectiveUILanguageCalled = true;
+ cr.webUIResponse(callbackId, true, 'en-US');
+ });
+
languageHelper = document.createElement('settings-languages');
// Prefs would normally be data-bound to settings-languages.
fakeDataBind(settingsPrefs, languageHelper, 'prefs');
- return languageHelper.whenReady();
+ return languageHelper.whenReady().then(function() {
+ assertEquals(
+ cr.isChromeOS || cr.isWindows, getProspectiveUILanguageCalled);
+ });
});
test('languages model', function() {
« no previous file with comments | « chrome/browser/ui/webui/settings/languages_handler.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698