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

Unified Diff: chrome/browser/resources/settings/device_page/device_page_browser_proxy.js

Issue 2630623002: cros: Allow user to configure note taking app in md-settings. (Closed)
Patch Set: NoteAppInfo => settings.NoteAppInfo Created 3 years, 10 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/device_page/device_page_browser_proxy.js
diff --git a/chrome/browser/resources/settings/device_page/device_page_browser_proxy.js b/chrome/browser/resources/settings/device_page/device_page_browser_proxy.js
index 0f7ed4dbc8dfc6cc859e6ada503cfb3b4c3047db..cd7cf7ddeff85ae850341975e1f07b4f3e3a3554 100644
--- a/chrome/browser/resources/settings/device_page/device_page_browser_proxy.js
+++ b/chrome/browser/resources/settings/device_page/device_page_browser_proxy.js
@@ -33,6 +33,11 @@ settings.PowerSource;
*/
settings.BatteryStatus;
+/**
+ * @typedef {{name:string, value:string, preferred:boolean}}
+ */
+settings.NoteAppInfo;
+
cr.define('settings', function() {
/** @interface */
function DevicePageBrowserProxy() {}
@@ -63,6 +68,26 @@ cr.define('settings', function() {
* battery (no external power source).
*/
setPowerSource: function(powerSourceId) {},
+
+ /**
+ * |callback| is run when there is new note-taking app information
+ * available or after |requestNoteTakingApps| has been called.
+ * @param {function(Array<settings.NoteAppInfo>, boolean):void} callback
+ */
+ setNoteTakingAppsUpdatedCallback: function(callback) {},
+
+ /**
+ * Request current note-taking app info. Invokes any callback registered in
+ * |onNoteTakingAppsUpdated|.
+ */
+ requestNoteTakingApps: function() {},
+
+ /**
+ * Changes the preferred note taking app.
+ * @param {string} appId The app id. This should be a value retrieved from a
+ * |onNoteTakingAppsUpdated| callback.
+ */
+ setPreferredNoteTakingApp: function(appId) {},
};
/**
@@ -105,6 +130,21 @@ cr.define('settings', function() {
setPowerSource: function(powerSourceId) {
chrome.send('setPowerSource', [powerSourceId]);
},
+
+ /** @override */
+ setNoteTakingAppsUpdatedCallback: function(callback) {
+ cr.addWebUIListener('onNoteTakingAppsUpdated', callback);
+ },
+
+ /** @override */
+ requestNoteTakingApps: function() {
+ chrome.send('requestNoteTakingApps');
+ },
+
+ /** @override */
+ setPreferredNoteTakingApp: function(appId) {
+ chrome.send('setPreferredNoteTakingApp', [appId]);
+ },
};
return {

Powered by Google App Engine
This is Rietveld 408576698