MD Settings: Reset profile dialog, display settings in new tab.

chrome/browser/resources/settings/reset_page/reset_profile_dialog.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-profile-dialog' is the dialog shown for clearing profile
  * settings.
  */
 Polymer({
   is: 'settings-reset-profile-dialog',
 
   behaviors: [WebUIListenerBehavior],
 
-  properties: {
-    feedbackInfo_: String,
-  },
-
   /** @private {!settings.ResetBrowserProxy} */
   browserProxy_: null,
 
   /** @override */
   ready: function() {
     this.browserProxy_ = settings.ResetBrowserProxyImpl.getInstance();
 
     this.addEventListener('iron-overlay-canceled', function() {
       this.browserProxy_.onHideResetProfileDialog();
     }.bind(this));
-
-    this.addWebUIListener('feedback-info-changed', function(feedbackInfo) {
-      this.feedbackInfo_ = feedbackInfo;
-    }.bind(this));
   },
 
   open: function() {
     this.$.dialog.open();
     this.browserProxy_.onShowResetProfileDialog();
   },
 
   /** @private */
   onCancelTap_: function() {
     this.$.dialog.cancel();
   },
 
   /** @private */
   onResetTap_: function() {
     this.$.resetSpinner.active = true;
     this.browserProxy_.performResetProfileSettings(
         this.$.sendSettings.checked).then(function() {
       this.$.resetSpinner.active = false;
       this.$.dialog.close();
       this.dispatchEvent(new CustomEvent('reset-done'));
     }.bind(this));
   },
 
-  /** @private */
-  onSendSettingsChange_: function() {
-    // TODO(dpapad): Update how settings info is surfaced when final mocks
-    // exist.
-    this.$.settings.hidden = !this.$.sendSettings.checked;
-    this.$.dialog.center();
+  /**
+   * Displays the settings that will be reported in a new tab.
+   * @private
+   */
+  onShowReportedSettingsTap_: function() {
+    this.browserProxy_.getReportedSettings().then(function(settings) {
+      var feedbackObj = {};
+      settings.forEach(function(entry) {

[tommycli, 2016/04/20 23:23:18]

Maybe add a comment that says something like: Replace arrays containing only one
value with just the value (for display purposes only).

[dpapad, 2016/04/21 00:27:10]

Added comment.

+        var values = entry.value.split('\n');
+        feedbackObj[entry.key] = values.length == 1 ? values[0] : values;
+      });
+      var win = window.open('about:blank');
+      var div = win.document.createElement('div');
+      div.textContent = JSON.stringify(feedbackObj, null, 2 /* spaces */);
+      div.style.whiteSpace = 'pre';

[tommycli, 2016/04/20 23:23:18]

Optional: Since you are styling it, should we just make it within
<pre><code></code></pre>? So the JSON lines up?

[dpapad, 2016/04/21 00:27:10]

Does it not already line up (see http://i.imgur.com/FdgmiEC.png)? I prefer to
keep this as is since it is shorter than dynamically creating
"<pre><code></code></pre>" and also because using HTML to style is a sort of
obsolete (in the same sense that the <b></b> tag for bold text is obsolete
compore to CSS font-weight property).

+      win.document.body.appendChild(div);
+    });
   },
 });