[MD settings] appearance theme and wallpaper

chrome/browser/resources/settings/appearance_page/appearance_page.js

 // Copyright 2015 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.
 
 /**
  * 'settings-appearance-page' is the settings page containing appearance
  * settings.
  *
  * Example:
  *
@@ skipping 10 matching lines @@
 
   properties: {
     /**
      * The current active route.
      */
     currentRoute: {
       notify: true,
       type: Object,
     },
 
+    /** @private {!settings.AppearanceBrowserProxy} */
+    browserProxy_: Object,
+
     /**
      * Preferences state.
      */
     prefs: {
       type: Object,
       notify: true,
     },
 
     /**
      * @private
@@ skipping 63 matching lines @@
 
     /** @private */
     themeSublabel_: String,
   },
 
   observers: [
     'themeChanged_(prefs.extensions.theme.id.value)',
     'zoomLevelChanged_(defaultZoomLevel_.value)',
   ],
 
+  created: function() {
+    this.browserProxy_ = settings.AppearanceBrowserProxyImpl.getInstance();
+  },
+
   ready: function() {
     this.$.defaultFontSize.menuOptions = this.fontSizeOptions_;
     this.$.pageZoom.menuOptions = this.pageZoomOptions_;
     // TODO(dschuyler): Look into adding a listener for the
     // default zoom percent.
     chrome.settingsPrivate.getDefaultZoomPercent(
         this.zoomPrefChanged_.bind(this));
   },
 
   /** @override */
   attached: function() {
     // Query the initial state.
-    cr.sendWithPromise('getResetThemeEnabled').then(
+    this.browserProxy_.getResetThemeEnabled().then(
         this.setResetThemeEnabled.bind(this));
 
     // Set up the change event listener.
     cr.addWebUIListener('reset-theme-enabled-changed',
                         this.setResetThemeEnabled.bind(this));
   },
 
   /**
    * @param {boolean} isNtp Whether to use the NTP as the home page.
    * @param {string} homepage If not using NTP, use this URL.
@@ skipping 16 matching lines @@
   /** @private */
   onCustomizeFontsTap_: function() {
     this.$.pages.setSubpageChain(['appearance-fonts']);
   },
 
   /** @private */
   openThemesGallery_: function() {
     window.open(loadTimeData.getString('themesGalleryUrl'));
   },
 
+  /**

[dpapad, 2016/04/19 23:32:40]

Same here, let's wrap with "if expr". I think you also need to mark this file as
"flattenhtml=true" in settings_resources.grd for this to work.

[dschuyler, 2016/04/20 00:19:07]

Done.

+   * ChromeOS only.
+   * @private
+   */
+  openWallpaperGallery_: function() {

[dpapad, 2016/04/19 23:32:40]

Nit(optional): Gallery vs Manager, can we stick with a single term?

[dschuyler, 2016/04/20 00:19:06]

Done.

+    this.browserProxy_.openWallpaperManager();
+  },
+
   /** @private */
   resetTheme_: function() {
-    chrome.send('resetTheme');
+    this.browserProxy_.resetTheme();
   },
 
   /** @private */
   showFontsPage_: function() {
     return this.currentRoute.subpage[0] == 'appearance-fonts';
   },
 
   /**
    * @param {string} themeId The theme ID.
    * @private
@@ skipping 21 matching lines @@
    * @param {number} percent The integer percentage of the page zoom.
    * @private
    */
   zoomLevelChanged_: function(percent) {
     // The |percent| may be undefined on startup.
     if (percent === undefined)
       return;
     chrome.settingsPrivate.setDefaultZoomPercent(percent);
   },
 });