[MD settings] move extra options in category settings

chrome/browser/resources/settings/privacy_page/privacy_page.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-privacy-page' is the settings page containing privacy and
  * security settings.
  */
 Polymer({
@@ skipping 147 matching lines @@
   /**
    * The sub-page title for the site or content settings.
    * @return {string}
    * @private
    */
   siteSettingsPageTitle_: function() {
     return loadTimeData.getBoolean('enableSiteSettings') ?
         loadTimeData.getString('siteSettings') :
         loadTimeData.getString('contentSettings');
   },
+
+<if expr="chromeos">
+  /** @private */
+  onAdobeFlashStorageClicked_: function() {
+    window.open('https://www.macromedia.com/support/' +
+        'documentation/en/flashplayer/help/settings_manager07.html');

[tommycli, 2016/12/09 00:13:19]

Interesting -- in other places, I've seen URLs be placed in the C++ and loaded
in via load time data. Also... is it intentional that the documentation is
specifically for the english version? maybe defer this to separate cl

[dschuyler, 2016/12/13 23:57:38]

This code is moved. I didn't look into the details of the implementation. (I did
check that it worked though).
There may be a bug here, but I think that should be fixed by a specific CL
rather than a refactor.

+  },
+</if>
 });