Show an extra privacy warning on Win8

chrome/browser/resources/options2/browser_options.js

Index: chrome/browser/resources/options2/browser_options.js
diff --git a/chrome/browser/resources/options2/browser_options.js b/chrome/browser/resources/options2/browser_options.js
index e0acdb745c6da8bcccdded3af1c293138a8b7fa6..3ebf7ddb5cea1766cab47f5597e9da62c4ac9c23 100644
--- a/chrome/browser/resources/options2/browser_options.js
+++ b/chrome/browser/resources/options2/browser_options.js
@@ -1191,6 +1191,14 @@ cr.define('options', function() {
     },
 
     /**
+     * Show/hide Windows 8 data privacy warning.
+     * @private
+     */
+    showWin8DataPrivacy_: function() {
+      $('privacy-win8-data-settings').hidden = false;

[arv (Not doing code reviews), 2012/08/15 14:33:31]

How about, in initializePage we do:

var m = /Windows NT (\d+(?:\.\d+))?/.exec(navigator.userAgent)
var isWin8 =  m && m[1] > 6.2;
var win8Element = $('privacy-win8-data-settings');
if (win8Element)
  win8Element.hidden = !isWin8;

[MAD, 2012/08/15 19:29:41]

Ho! Cool, I didn't think about that...
Thanks!... Done!

+    },
+
+    /**
      * Show/hide the display options button on the System settings page.
      * @private
      */
@@ -1318,6 +1326,7 @@ cr.define('options', function() {
     'showDisplayOptions',
     'showMouseControls',
     'showTouchpadControls',
+    'showWin8DataPrivacy',
     'updateAccountPicture',
     'updateAutoLaunchState',
     'updateDefaultBrowserState',