Desktop MD Site Settings: Re-order the permission list.

chrome/browser/resources/settings/site_settings/site_details.html

 <link rel="import" href="chrome://resources/html/polymer.html">
 <link rel="import" href="chrome://resources/polymer/v1_0/iron-flex-layout/iron-flex-layout.html">
 <link rel="import" href="chrome://resources/polymer/v1_0/paper-button/paper-button.html">
 <link rel="import" href="chrome://resources/polymer/v1_0/paper-icon-button/paper-icon-button.html">
 <link rel="import" href="chrome://md-settings/icons.html">
 <link rel="import" href="chrome://md-settings/settings_shared_css.html">
 <link rel="import" href="chrome://md-settings/site_settings/constants.html">
 <link rel="import" href="chrome://md-settings/site_settings/site_details_permission.html">
 <link rel="import" href="chrome://md-settings/site_settings/site_settings_behavior.html">
 <link rel="import" href="chrome://md-settings/site_settings/website_usage_private_api.html">
 
 <dom-module id="site-details">
   <link rel="import" type="css" href="site_details.css">
   <template>
     <style include="settings-shared"></style>
     <div class="settings-box block">
       <div class="horizontal layout origin-box">
         <div class="website-icon" style$="[[computeSiteIcon(site)]]"></div>
         <div class="origin flex">[[site.originForDisplay]]</div>
       </div>
       <h2 id="usage" hidden$="[[!storedData_]]">$i18n{siteSettingsUsage}</h2>
       <paper-item id="storage" hidden$="[[!storedData_]]">
         <div class="flex">[[storedData_]]</div>
         <paper-icon-button icon="settings:delete"
             on-tap="onClearStorage_"
             alt="$i18n{siteSettingsDelete}"></paper-icon-button>
       </paper-item>
       <h2>$i18n{siteSettingsPermissions}</h2>
+
+      <!-- The order of the following items from: http://crbug.com/610358. -->

[michaelpg, 2016/05/23 18:14:22]

nit: we can't comment every feature that's specified in a bug. we just generally
assume our code is written as intended. if the desired ordering changes or a
permission type is added or removed, the comment will be out of date. we should
trust that nobody will re-order these without good reason.

[Finnur, 2016/05/25 18:41:29]

My thinking behind this was that random people are going to be adding new types
in the future and I might not catch all of them. History tells me they would be
tempted to add theirs in somewhere random, not knowing that there's both a
pre-determined order and multiple places to update.

But, I don't feel strongly about this, so I've removed the comment from both
locations.

       <site-details-permission site="[[site]]" id="cookies"
           category="{{ContentSettingsTypes.COOKIES}}">
       </site-details-permission>
+      <site-details-permission site="[[site]]" id="geolocation"
+          category="{{ContentSettingsTypes.GEOLOCATION}}">
+      </site-details-permission>
+      <site-details-permission site="[[site]]" id="camera"
+          category="{{ContentSettingsTypes.CAMERA}}">
+      </site-details-permission>
+      <site-details-permission site="[[site]]" id="mic"
+          category="{{ContentSettingsTypes.MIC}}">
+      </site-details-permission>
+      <site-details-permission site="[[site]]" id="notification"
+          category="{{ContentSettingsTypes.NOTIFICATIONS}}">
+      </site-details-permission>
       <site-details-permission site="[[site]]" id="javascript"
           category="{{ContentSettingsTypes.JAVASCRIPT}}">
       </site-details-permission>
       <site-details-permission site="[[site]]" id="popups"
           category="{{ContentSettingsTypes.POPUPS}}">
       </site-details-permission>
-      <site-details-permission site="[[site]]" id="geolocation"
-          category="{{ContentSettingsTypes.GEOLOCATION}}">
-      </site-details-permission>
-      <site-details-permission site="[[site]]" id="notification"
-          category="{{ContentSettingsTypes.NOTIFICATIONS}}">
-      </site-details-permission>
       <site-details-permission site="[[site]]" id="fullscreen"
           category="{{ContentSettingsTypes.FULLSCREEN}}">
       </site-details-permission>
-      <site-details-permission site="[[site]]" id="camera"
-          category="{{ContentSettingsTypes.CAMERA}}">
-      </site-details-permission>
-      <site-details-permission site="[[site]]" id="mic"
-          category="{{ContentSettingsTypes.MIC}}">
-      </site-details-permission>
+
       <paper-button on-tap="onClearAndReset_" raised class="reset-button">
         $i18n{siteSettingsClearAndReset}
       </paper-button>
     </div>
     <website-usage-private-api id="usageApi"
         website-data-usage="{{storedData_}}"
         website-storage-type="{{storageType_}}">
     </website-usage-private-api>
   </template>
   <script src="site_details.js"></script>
 </dom-module>