Incorporate reset button in post-automatic-reset settings-banner.

chrome/browser/resources/options/automatic_settings_reset_banner.js

Index: chrome/browser/resources/options/automatic_settings_reset_banner.js
diff --git a/chrome/browser/resources/options/automatic_settings_reset_banner.js b/chrome/browser/resources/options/automatic_settings_reset_banner.js
index 15877ad4b7fdf8d69f24123a2546c291b873813d..37ed64161a508f5159970b9975f06ccfb6fa1d3c 100644
--- a/chrome/browser/resources/options/automatic_settings_reset_banner.js
+++ b/chrome/browser/resources/options/automatic_settings_reset_banner.js
@@ -39,6 +39,12 @@ cr.define('options', function() {
         chrome.send('metricsHandler:recordAction',
             ['AutomaticSettingsReset_WebUIBanner_LearnMoreClicked']);
       };
+      $('automatic-settings-reset-banner-activate-reset').onclick =
+          function(event) {
+            chrome.send('metricsHandler:recordAction',
+                ['AutomaticSettingsReset_WebUIBanner_ResetClicked']);
+            OptionsPage.navigateToPage('resetProfileSettings');
+          };

[robertshield, 2014/07/28 21:09:42]

not sure, but seems like this should line up with the $( on line 42. 

[gab, 2014/07/28 21:22:52]

Wasn't sure either but since I indented "function(event) {" above so I indented
this "}" to match that.

I couldn't find an example of such a wrapping caused by the onclick target's
name being too long...

UI owners will know I assume.

[Bernhard Bauer, 2014/07/28 22:29:38]

I think what you do here is indent line 43 four spaces, then continue with two
space indent for the rest of the function (and then no space for the closing
brace and semicolon).

[gab, 2014/07/29 14:54:40]

Okay so just like I have it, but bring the closing "};" back.

Done.

[Bernhard Bauer, 2014/07/29 15:01:43]

Sorry, what I meant was this:

$('foo').onclick =
    function() {
  chrome.send('moose!');
};

[gab, 2014/07/29 15:41:55]

Ah I see, thanks for clarifying, done!

     },
   };