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

Unified Diff: chrome/browser/ui/webui/settings/languages_handler.h

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
Index: chrome/browser/ui/webui/settings/languages_handler.h
diff --git a/chrome/browser/ui/webui/settings/languages_handler.h b/chrome/browser/ui/webui/settings/languages_handler.h
index d8ae17b8d028ec1ae9465248318eddea1a665e19..e1cfd55e98f5f80b67eceb9455b787fac1afbde9 100644
--- a/chrome/browser/ui/webui/settings/languages_handler.h
+++ b/chrome/browser/ui/webui/settings/languages_handler.h
@@ -32,8 +32,14 @@ class LanguagesHandler : public SettingsPageUIHandler {
void OnJavascriptDisallowed() override {}
private:
- // Changes the UI language, provided the user is allowed to do so.
- void HandleSetUILanguage(const base::ListValue* args);
+ // Returns the prospective UI language. May not match the actual UI language,
+ // depending on the user's permissions and whether the language is substituted
+ // for another locale.
+ void HandleGetProspectiveUILanguage(const base::ListValue* args);
+
+ // Changes the preferred UI language, provided the user is allowed to do so.
+ // The actual UI language will not change until the next restart.
+ void HandleSetProspectiveUILanguage(const base::ListValue* args);
Profile* profile_; // Weak pointer.

Powered by Google App Engine
This is Rietveld 408576698