MD Settings: Migrating C++ handlers for profile reset page.

chrome/browser/resources/settings/reset_page/reset_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.
 
 /**
  * @fileoverview
  * 'settings-reset-page' is the settings page containing reset
  * settings.
  *
  * Example:
  *
  *    <iron-animated-pages>
  *      <settings-reset-page prefs="{{prefs}}">
  *      </settings-reset-page>
  *      ... other pages ...
  *    </iron-animated-pages>
  *
  * @group Chrome Settings Elements
  * @element settings-reset-page
  */
 Polymer({
   is: 'settings-reset-page',
 
+  properties: {
+    feedbackInfo_: String,
+  },
+
+  attached: function() {
+    cr.define('SettingsResetPage', function() {
+      return {
+        doneResetting: function() {
+          this.$.resetSpinner.active = false;
+          this.$.resetDialog.close();
+        }.bind(this),
+
+        setFeedbackInfo: function(data) {
+          this.set('feedbackInfo_', data.feedbackInfo);
+          this.async(function() {
+            this.$.resetDialog.center();
+          });
+        }.bind(this),
+      };
+    }.bind(this));
+  },
+
   /** @private */
   onShowDialog_: function() {
     this.$.resetDialog.open();
+    chrome.send('onShowResetProfileDialog');
   },
 
   /** @private */
   onCancelTap_: function() {
     this.$.resetDialog.close();
+    chrome.send('onHideResetProfileDialog');
   },
 
   /** @private */
   onResetTap_: function() {
-    // TODO(dpapad): Set up C++ handlers and figure out when it is OK to close
-    // the dialog.
-    this.$.resetDialog.close();
+    this.$.resetSpinner.active = true;
+    chrome.send('performResetProfileSettings', [this.$.sendSettings.checked]);
   },
 
   /** @private */
   onLearnMoreTap_: function() {
     window.open(loadTimeData.getString('resetPageLearnMoreUrl'));
+  },
+
+  /** @private */
+  onCheckboxChange_: function() {

[Dan Beam, 2015/11/09 23:05:19]

arguable nit: alphabetize these private methods

[dpapad, 2015/11/09 23:37:49]

Indeed this is arguable. I am failing to see the benefit of alphabetizing the
order of methods. Let's have the
discussion in person if you think it is worth it.

[Dan Beam, 2015/11/10 02:53:25]

I thought it was part of the style guide (JS or C++) but it's not.  fwiw: if
there's no other useful way to order them, alphabetically ordering makes them
easier to scan as they're sorted.  that's why I do this.

+    // TODO(dpapad): Update how settings info is surfaced when final mocks
+    // exist.
+    this.$.settings.hidden = !this.$.sendSettings.checked;
+    this.$.resetDialog.center();
   }
 });