Enable import/export of passwords into/from Password Manager

chrome/browser/resources/options/password_manager.html

Index: chrome/browser/resources/options/password_manager.html
diff --git a/chrome/browser/resources/options/password_manager.html b/chrome/browser/resources/options/password_manager.html
index 3e2584572b7fa0876bf3b3413ebeef7a93ef6aa6..c38963d4c8afbaf1c959d4b50705b8c3a2167e38 100644
--- a/chrome/browser/resources/options/password_manager.html
+++ b/chrome/browser/resources/options/password_manager.html
@@ -2,38 +2,48 @@
   <div class="close-button"></div>
   <h1 i18n-content="passwordsPage"></h1>
   <div class="content-area">
-    <div id="auto-signin-block" class="checkbox" hidden>
-      <label>
-        <input pref="credentials_enable_autosignin" type="checkbox">
-        <span i18n-content="autoSigninTitle"></span>
-      </label>
-      <div class="setting-extra-description">
-        <span i18n-content="autoSigninDescription"></span>
+    <div class="password-manager-home">
+      <div id="auto-signin-block" class="checkbox" hidden>
+        <label>
+          <input pref="credentials_enable_autosignin" type="checkbox">
+          <span i18n-content="autoSigninTitle"></span>
+        </label>
+        <div class="setting-extra-description">
+          <span i18n-content="autoSigninDescription"></span>
+        </div>
       </div>
-    </div>
-    <div id="password-list-headers">
-      <div id="passwords-title">
-        <h3 i18n-content="savedPasswordsTitle"></h3>
+      <div id="password-list-headers">
+        <div id="passwords-title">
+          <h3 i18n-content="savedPasswordsTitle"></h3>
+        </div>
+        <div id="password-search-column">
+          <input id="password-search-box" type="search"
+              i18n-values="placeholder:passwordSearchPlaceholder" incremental>
+        </div>
       </div>
-      <div id="password-search-column">
-        <input id="password-search-box" type="search"
-            i18n-values="placeholder:passwordSearchPlaceholder" incremental>
+      <list id="saved-passwords-list" class="settings-list"></list>
+      <div id="saved-passwords-list-empty-placeholder"
+           class="settings-list-empty" hidden>
+        <span i18n-content="passwordsNoPasswordsDescription"></span>
+        <a target="_blank" i18n-content="learnMore"
+           i18n-values="href:passwordManagerLearnMoreURL"></a>
+      </div>
+      <div class="import-export-button-strip" align="right">

[Evan Stade, 2016/04/08 17:28:28]

I don't think this is correct either. Other button-strip elements in other webui
don't use align. I don't get why you need to worry about this at all. Isn't it
handled (with the spacer div) by ui/webui/resources/css/overlay.css? Just add
the .button-strip class

[xunlu, 2016/04/08 19:10:20]

But then how do I flush the button to the right of the button-strip div?

[Evan Stade, 2016/04/08 19:23:18]

how do other dialogs do it?

[xunlu, 2016/04/08 20:49:40]

Done.

+        <button id="password-manager-import" class="password-manager-home"
+            i18n-content="passwordManagerImportPasswordButtonText" hidden>
+        </button>
+        <button id="password-manager-export" class="password-manager-home"
+            i18n-content="passwordManagerExportPasswordButtonText" hidden>
+        </button>
+      </div>
+      <h3 i18n-content="passwordExceptionsTitle"></h3>
+      <list id="password-exceptions-list" class="settings-list"></list>
+      <div id="password-exceptions-list-empty-placeholder" hidden
+           class="settings-list-empty">
+        <span i18n-content="passwordsNoExceptionsDescription"></span>
+        <a id="exceptions-learn-more" target="_blank" i18n-content="learnMore"
+           i18n-values="href:passwordManagerLearnMoreURL"></a>
       </div>
-    </div>
-    <list id="saved-passwords-list" class="settings-list"></list>
-    <div id="saved-passwords-list-empty-placeholder"
-         class="settings-list-empty" hidden>
-      <span i18n-content="passwordsNoPasswordsDescription"></span>
-      <a target="_blank" i18n-content="learnMore"
-         i18n-values="href:passwordManagerLearnMoreURL"></a>
-    </div>
-    <h3 i18n-content="passwordExceptionsTitle"></h3>
-    <list id="password-exceptions-list" class="settings-list"></list>
-    <div id="password-exceptions-list-empty-placeholder" hidden
-         class="settings-list-empty">
-      <span i18n-content="passwordsNoExceptionsDescription"></span>
-      <a id="exceptions-learn-more" target="_blank" i18n-content="learnMore"
-         i18n-values="href:passwordManagerLearnMoreURL"></a>
     </div>
   </div>
   <div class="action-area">
@@ -46,7 +56,8 @@
     </span>
     <div class="spacer-div"></div>
     <div class="button-strip">
-      <button id="password-manager-confirm" i18n-content="done"></button>
+      <button id="password-manager-confirm" class="password-manager-home"
+          i18n-content="done"></button>
     </div>
   </div>
 </div>