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

Unified Diff: chrome/browser/resources/settings/languages_page/languages_page.js

Issue 2252323002: MD Settings: reduce complexity and overhead of Languages singleton (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@RemoveLanguageInputMethods
Patch Set: Reduce diff 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/resources/settings/languages_page/languages_page.js
diff --git a/chrome/browser/resources/settings/languages_page/languages_page.js b/chrome/browser/resources/settings/languages_page/languages_page.js
index f36e18a738a5f5716cc606a2b77f64028616e1a4..c16e69770ad4df9448550b0e0e94628859a68032 100644
--- a/chrome/browser/resources/settings/languages_page/languages_page.js
+++ b/chrome/browser/resources/settings/languages_page/languages_page.js
@@ -31,6 +31,9 @@ Polymer({
notify: true,
},
+ /** @type {!LanguageHelper} */
+ languageHelper: Object,
+
/** @private */
spellCheckSecondary_: {
type: String,
@@ -43,14 +46,6 @@ Polymer({
* @private
*/
detailLanguage_: Object,
-
- /** @private {!LanguageHelper} */
- languageHelper_: Object,
- },
-
- /** @override */
- created: function() {
- this.languageHelper_ = LanguageHelperImpl.getInstance();
},
/**
@@ -66,7 +61,7 @@ Polymer({
// Set the prospective UI language. This won't take effect until a restart.
var tapEvent = /** @type {!{model: !{item: !LanguageState}}} */(e);
if (tapEvent.model.item.language.supportsUI)
- this.languageHelper_.setUILanguage(tapEvent.model.item.language.code);
+ this.languageHelper.setUILanguage(tapEvent.model.item.language.code);
},
/**
@@ -84,8 +79,8 @@ Polymer({
* @param {!{target: Element, model: !{item: !LanguageState}}} e
*/
onSpellCheckChange_: function(e) {
- this.languageHelper_.toggleSpellCheck(e.model.item.language.code,
- e.target.checked);
+ this.languageHelper.toggleSpellCheck(e.model.item.language.code,
+ e.target.checked);
},
/** @private */
@@ -138,7 +133,7 @@ Polymer({
* @private
*/
onMoveUpTap_: function(e) {
- this.languageHelper_.moveLanguage(e.model.item.language.code, -1);
+ this.languageHelper.moveLanguage(e.model.item.language.code, -1);
},
/**
@@ -147,7 +142,7 @@ Polymer({
* @private
*/
onMoveDownTap_: function(e) {
- this.languageHelper_.moveLanguage(e.model.item.language.code, 1);
+ this.languageHelper.moveLanguage(e.model.item.language.code, 1);
},
/**
@@ -183,7 +178,7 @@ Polymer({
return;
// Set the input method.
- this.languageHelper_.setCurrentInputMethod(e.model.item.id);
+ this.languageHelper.setCurrentInputMethod(e.model.item.id);
},
/**
@@ -194,7 +189,7 @@ Polymer({
*/
onInputMethodOptionsTap_: function(e) {
assert(cr.isChromeOS);
- this.languageHelper_.openInputMethodOptions(e.model.item.id);
+ this.languageHelper.openInputMethodOptions(e.model.item.id);
},
/**
@@ -257,7 +252,7 @@ Polymer({
*/
isProspectiveUILanguage_: function(languageCode, prospectiveUILanguage) {
assert(cr.isChromeOS || cr.isWindows);
- return languageCode == this.languageHelper_.getProspectiveUILanguage();
+ return languageCode == this.languageHelper.getProspectiveUILanguage();
},
/**
@@ -265,8 +260,8 @@ Polymer({
* @private
*/
getProspectiveUILanguageName_: function() {
- return this.languageHelper_.getLanguage(
- this.languageHelper_.getProspectiveUILanguage()).displayName;
+ return this.languageHelper.getLanguage(
+ this.languageHelper.getProspectiveUILanguage()).displayName;
},
/**

Powered by Google App Engine
This is Rietveld 408576698