Add accessibility settings metrics

chrome/browser/resources/options/browser_options.js

Index: chrome/browser/resources/options/browser_options.js
diff --git a/chrome/browser/resources/options/browser_options.js b/chrome/browser/resources/options/browser_options.js
index c5fd816d04aa3b2ab4c2fbad9fefd0501d113a92..768c50376006b2a3397f3e9c458b9c3113e627ab 100644
--- a/chrome/browser/resources/options/browser_options.js
+++ b/chrome/browser/resources/options/browser_options.js
@@ -496,6 +496,11 @@ cr.define('options', function() {
         Preferences.getInstance().addEventListener(
             'settings.accessibility',
             updateAccessibilitySettingsButton);
+        $('accessibility-learn-more').onclick = function(event) {

[michaelpg, 2014/06/13 22:05:05]

nit-ish: unused parameter

[stevenjb, 2014/06/13 22:11:01]

Hmm, pattern elsewhere seems to be to include the parameter even when unused.

[Dan Beam, 2014/06/16 19:08:24]

personal preference: no unused params
Super Official WebUI Reviewer Dude Opinion™: doesn't matter

[stevenjb, 2014/06/16 21:06:16]

I discussed this with some Blink developers and we basically agreed that either
kind of suck. (Unused variables are confusing, but apparently 'event' is
sometimes implicitly declared). We all thought that the C++-ish 'unused_event'
was a reasonable compromise, so I'll use that within this section and we can
make it consistent over time.

+          window.open(loadTimeData.getString('accessibilityLearnMoreURL'));
+          chrome.send('coreOptionsUserMetricsAction',
+                      ['Options_AccessibilityLearnMore']);
+        };
         $('accessibility-settings-button').onclick = function(event) {
           window.open(loadTimeData.getString('accessibilitySettingsURL'));
         };