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

chrome/browser/resources/settings/reset_page/reset_page.js

Index: chrome/browser/resources/settings/reset_page/reset_page.js
diff --git a/chrome/browser/resources/settings/reset_page/reset_page.js b/chrome/browser/resources/settings/reset_page/reset_page.js
index f27b892e832b2bd32899b2415dc91fff9d91f525..7d33367721df982114596b4b5d004dcfb7f28af2 100644
--- a/chrome/browser/resources/settings/reset_page/reset_page.js
+++ b/chrome/browser/resources/settings/reset_page/reset_page.js
@@ -21,25 +21,56 @@
 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();
   }
 });