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

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

Issue 2827553002: MD Settings: Use a browser proxy from the languages page. Re-enable test. (Closed)
Patch Set: Make inputMethodPrivate CrOS only Created 3 years, 8 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 499164b8ee049be06ca08a3736f6abd6a8145cbe..e6ab30e44b6b6146c9805bf781e61b6f7e8287f9 100644
--- a/chrome/browser/resources/settings/languages_page/languages_page.js
+++ b/chrome/browser/resources/settings/languages_page/languages_page.js
@@ -81,6 +81,14 @@ Polymer({
},
},
+ /** @private {?settings.LanguagesBrowserProxy} */
+ browserProxy_: null,
+
+ /** @override */
+ attached: function() {
+ this.browserProxy_ = settings.LanguagesBrowserProxyImpl.getInstance();
+ },
+
/**
* Stamps and opens the Add Languages dialog, registering a listener to
* disable the dialog's dom-if again on close.
@@ -190,7 +198,8 @@ Polymer({
return;
// Set the input method.
- this.languageHelper.setCurrentInputMethod(e.model.item.id);
+ this.browserProxy_.getInputMethodPrivate().setCurrentInputMethod(
+ e.model.item.id);
},
/**
@@ -200,7 +209,8 @@ Polymer({
* @private
*/
onInputMethodOptionsTap_: function(e) {
- this.languageHelper.openInputMethodOptions(e.model.item.id);
+ this.browserProxy_.getInputMethodPrivate().openOptionsPage(
+ e.model.item.id);
},
// </if>
@@ -259,7 +269,7 @@ Polymer({
// We don't support unchecking this checkbox. TODO(michaelpg): Ask for a
// simpler widget.
assert(e.target.checked);
- this.languageHelper.setProspectiveUILanguage(
+ this.browserProxy_.setProspectiveUILanguage(
this.detailLanguage_.language.code);
this.closeMenuSoon_();

Powered by Google App Engine
This is Rietveld 408576698