MD Settings: Dual-role charger selection

chrome/browser/resources/settings/device_page/device_page.html

Index: chrome/browser/resources/settings/device_page/device_page.html
diff --git a/chrome/browser/resources/settings/device_page/device_page.html b/chrome/browser/resources/settings/device_page/device_page.html
index feb35dc531596ccccda68ec9bf4ef4f13d3ed5e4..e12ef3b64b9062abb1d737ae2b9304784536d540 100644
--- a/chrome/browser/resources/settings/device_page/device_page.html
+++ b/chrome/browser/resources/settings/device_page/device_page.html
@@ -1,4 +1,5 @@
 <link rel="import" href="chrome://resources/html/i18n_behavior.html">
+<link rel="import" href="chrome://resources/html/md_select_css.html">
 <link rel="import" href="chrome://resources/html/polymer.html">
 <link rel="import" href="chrome://resources/html/web_ui_listener_behavior.html">
 <link rel="import" href="chrome://resources/polymer/v1_0/iron-icon/iron-icon.html">
@@ -19,7 +20,7 @@
 
 <dom-module id="settings-device-page">
   <template>
-    <style include="settings-shared"></style>
+    <style include="settings-shared md-select"></style>
     <settings-animated-pages id="pages" section="device">
       <neon-animatable id="main" route-path="default">
         <div id="pointersRow" class="settings-box first"
@@ -57,6 +58,35 @@
           <div class="middle">$i18n{storageTitle}</div>
           <button class="subpage-arrow" is="paper-icon-button-light"></button>
         </div>
+        <template is="dom-if" if="[[enablePowerSettings_]]">
+          <div id="powerRow" class="settings-box two-line">
+            <iron-icon icon="[[batteryIcon_]]"></iron-icon>
+            <div class="middle">
+              <div>[[powerLabel_]]</div>
+              <div class="secondary">[[batteryStatus_.statusText]]</div>
+            </div>
+            <div class="md-select-wrapper"
+                hidden$="[[!showPowerDropdown_]]">
+              <select id="powerSource" class="md-select"
+                  on-change="onPowerSourceChange_">
+                <option value=""
+                    selected$="[[isEqual_('', selectedPowerSourceId_)]]">

[stevenjb, 2017/01/13 17:22:42]

nit: Can we easily use a descriptive value for battery instead of ""?

[Daniel Erat, 2017/01/13 17:44:54]

the IDs are opaque strings that come from powerd. the field from powerd is
actually specified to be external, so an empty string is used to indicate no
external power source:

  // ID of the PowerSource that is currently providing power to the system.
  optional string external_power_source_id = 17;

so i'm not sure that it makes sense to map it to something else here.

[stevenjb, 2017/01/13 18:03:41]

OK, that makes sense.

+                  $i18n{powerSourceBattery}
+                </option>
+                <template is="dom-repeat" items="[[powerSources_]]">
+                  <option value="[[item.id]]"
+                      selected$="[[isEqual_(item.id, selectedPowerSourceId_)]]">
+                    [[item.description]]
+                  </option>
+                </template>
+              </select>
+              <span class="md-select-underline"></span>
+            </div>
+            <div hidden$="[[showPowerDropdown_]]">
+              [[powerSourceName_]]
+            </div>
+          </div>
+        </template>
       </neon-animatable>
       <template is="dom-if" route-path="/pointer-overlay">
         <settings-subpage