cros: Allow user to configure note taking app in md-settings.

chrome/browser/resources/settings/device_page/stylus.js

Index: chrome/browser/resources/settings/device_page/stylus.js
diff --git a/chrome/browser/resources/settings/device_page/stylus.js b/chrome/browser/resources/settings/device_page/stylus.js
index a31df1d433c5b0290731fa10c112a73d6955aa59..1f2497bd5002be7bc8aae68f46516a0014a2ca88 100644
--- a/chrome/browser/resources/settings/device_page/stylus.js
+++ b/chrome/browser/resources/settings/device_page/stylus.js
@@ -7,6 +7,9 @@
  * 'settings-stylus' is the settings subpage with stylus-specific settings.
  */
 
+/** @const */ var FIND_MORE_APPS_URL = 'https://play.google.com/store/apps/' +
+    'collection/promotion_30023cb_stylus_apps';
+
 Polymer({
   is: 'settings-stylus',
 
@@ -16,5 +19,69 @@ Polymer({
       type: Object,
       notify: true,
     },
-  }
+
+    /**
+     * Note taking apps the user can pick between.
+     * @type {Array<{name:string, value:string, preferred:boolean}>}
+     * @private
+     */
+    appChoices_: {
+      type: Array,
+      value: () => {}

[stevenjb, 2017/01/25 01:40:08]

Unfortunately we can no longer use ES6 in settings (it breaks vulcanization),
specifically =>, for..of, and let.

[jdufault, 2017/02/03 21:32:08]

Done.

+    },
+
+    /**
+     * True if the ARC container has not finished starting yet.
+     * @private
+     */
+    waitingForAndroid_: {
+      type: Boolean,
+      value: false
+    },
+  },
+
+  ready: function() {
+    settings.DevicePageBrowserProxyImpl.getInstance().onNoteTakingAppsUpdated(
+        this.onNoteAppsUpdated_.bind(this));
+    settings.DevicePageBrowserProxyImpl.getInstance().requestNoteTakingApps();

[stevenjb, 2017/01/25 01:40:08]

Since we have several proxy calls here, we should probably set a local
browserProxy_, e.g.:
https://cs.chromium.org/chromium/src/chrome/browser/resources/settings/appear...

[jdufault, 2017/02/03 21:32:08]

Done.

+  },
+
+  /** @private */
+  onSelectedAppChanged_: function() {
+    settings.DevicePageBrowserProxyImpl.getInstance().setPreferredNoteTakingApp(
+        this.$.menu.value);
+  },
+
+  /**
+   * @param {Array<{name:string, value:string, preferred:boolean}>} apps

[stevenjb, 2017/01/25 01:40:08]

{name:string, value:string, preferred:boolean} should be typed in the proxy JS,
e.g.:
https://cs.chromium.org/chromium/src/chrome/browser/resources/md_user_manager...

[jdufault, 2017/02/03 21:32:09]

Done.

+   * @param {boolean} waitingForAndroid
+   * @private
+   */
+  onNoteAppsUpdated_: function(apps, waitingForAndroid) {
+    this.waitingForAndroid_ = waitingForAndroid;
+    this.appChoices_ = apps;
+  },
+
+  /**
+   * @param {Array<{name:string, value:string, preferred:boolean}>} apps
+   * @param {boolean} waitingForAndroid
+   * @private
+   */
+  showNoApps_: function(apps, waitingForAndroid) {
+    return apps.length == 0 && !waitingForAndroid;
+  },
+
+  /**
+   * @param {Array<{name:string, value:string, preferred:boolean}>} apps
+   * @param {boolean} waitingForAndroid
+   * @private
+   */
+  showApps_: function(apps, waitingForAndroid) {
+    return apps.length > 0 && !waitingForAndroid;
+  },
+
+  /** @private */
+  onFindAppsTap_: function() {
+    window.open(FIND_MORE_APPS_URL);
+  },
 });