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

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

Issue 2827553002: MD Settings: Use a browser proxy from the languages page. Re-enable test. (Closed)
Patch Set: Remove accidental usage of getInputMethodPrivate() on non-Cros 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_browser_proxy.js
diff --git a/chrome/browser/resources/settings/languages_page/languages_browser_proxy.js b/chrome/browser/resources/settings/languages_page/languages_browser_proxy.js
new file mode 100644
index 0000000000000000000000000000000000000000..a36bb2e5bf757b297d00c6f436fd177e9e67146c
--- /dev/null
+++ b/chrome/browser/resources/settings/languages_page/languages_browser_proxy.js
@@ -0,0 +1,76 @@
+// Copyright 2017 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+/**
+ * @fileoverview A helper object used from the languages section
+ * to interact with the browser.
+ */
+
+cr.define('settings', function() {
+ /** @interface */
+ function LanguagesBrowserProxy() {}
+
+ LanguagesBrowserProxy.prototype = {
+// <if expr="chromeos or is_win">
+ /**
+ * Sets the prospective UI language to the chosen language. This won't
+ * affect the actual UI language until a restart.
+ * @param {string} languageCode
+ */
+ setProspectiveUILanguage: function(languageCode) {},
+
+ /** @return {!Promise<string>} */
+ getProspectiveUILanguage: function() {},
+// </if>
+
+ /** @return {!LanguageSettingsPrivate} */
+ getLanguageSettingsPrivate: function() {},
+
+// <if expr="chromeos">
+ /** @return {!InputMethodPrivate} */
+ getInputMethodPrivate: function() {},
+// </if>
+ };
+
+ /**
+ * @constructor
+ * @implements {settings.LanguagesBrowserProxy}
+ */
+ function LanguagesBrowserProxyImpl() {}
+ // The singleton instance_ is replaced with a test version of this wrapper
+ // during testing.
+ cr.addSingletonGetter(LanguagesBrowserProxyImpl);
+
+ LanguagesBrowserProxyImpl.prototype = {
+// <if expr="chromeos or is_win">
+ /** @override */
+ setProspectiveUILanguage: function(languageCode) {
+ chrome.send('setProspectiveUILanguage', [languageCode]);
+ },
+
+ /** @override */
+ getProspectiveUILanguage: function() {
+ return cr.sendWithPromise('getProspectiveUILanguage');
+ },
+// </if>
+
+ /** @override */
+ getLanguageSettingsPrivate: function() {
+ return /** @type {!LanguageSettingsPrivate} */ (
+ chrome.languageSettingsPrivate);
+ },
+
+// <if expr="chromeos">
+ /** @override */
+ getInputMethodPrivate: function() {
+ return /** @type {!InputMethodPrivate} */ (chrome.inputMethodPrivate);
+ },
+// </if>
+ };
+
+ return {
+ LanguagesBrowserProxy: LanguagesBrowserProxy,
+ LanguagesBrowserProxyImpl: LanguagesBrowserProxyImpl,
+ };
+});

Powered by Google App Engine
This is Rietveld 408576698