Enable import/export of passwords into/from Password Manager

chrome/browser/resources/options/password_manager.js

 // Copyright (c) 2012 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.
 
 cr.define('options', function() {
   /** @const */ var Page = cr.ui.pageManager.Page;
   /** @const */ var PageManager = cr.ui.pageManager.PageManager;
   /** @const */ var ArrayDataModel = cr.ui.ArrayDataModel;
 
   /////////////////////////////////////////////////////////////////////////////
@@ skipping 41 matching lines @@
      * The most recent search query, or null if the query is empty.
      * @type {?string}
      * @private
      */
     lastQuery_: null,
 
     /** @override */
     initializePage: function() {
       Page.prototype.initializePage.call(this);
 
+      var self = this;

[Evan Stade, 2016/04/06 21:25:22]

is this used somewhere?

[xunlu, 2016/04/08 15:38:21]

Done.

+
       $('auto-signin-block').hidden =
           !loadTimeData.getBoolean('enableCredentialManagerAPI');
 
       $('password-manager-confirm').onclick = function() {
         PageManager.closeOverlay();
       };
 
+      $('password-manager-import').onclick = function() {
+        chrome.send('importPassword');
+      };
+
+      $('password-manager-export').onclick = function() {
+        chrome.send('exportPassword');
+      };
+
       $('password-search-box').addEventListener('search',
-          this.handleSearchQueryChange_.bind(this));
+        this.handleSearchQueryChange_.bind(this));

[Evan Stade, 2016/04/06 21:25:22]

seems like the indentation was correct before

[xunlu, 2016/04/08 15:38:21]

Done.

 
       $('exceptions-learn-more').onclick = function() {
         chrome.send('coreOptionsUserMetricsAction',
                     ['Options_PasswordManagerExceptionsLearnMore']);
         return true;  // Always follow the href
       };
 
       this.createSavedPasswordsList_();
       this.createPasswordExceptionsList_();
     },
@@ skipping 182 matching lines @@
             index = i;
             break;
           }
         }
       }
 
       // Reveal the password in the UI.
       var item = this.savedPasswordsList_.getListItemByIndex(index);
       item.showPassword(password);
     },
+
+    /**
+     * @param {boolean} visible Whether the link should be visible.
+     * @private
+     */
+    setManageAccountLinkVisibility_: function(visible) {
+      $('manage-passwords-span').hidden = !visible;
+    },
+
+    showImportExportButton_: function() {
+      $('password-manager-import').disabled = false;
+      $('password-manager-export').disabled = false;
+    },
   };
 
   /**
    * Removes a saved password.
    * @param {number} rowIndex indicating the row to remove.
    */
   PasswordManager.removeSavedPassword = function(rowIndex) {
-      chrome.send('removeSavedPassword', [String(rowIndex)]);
-      chrome.send('coreOptionsUserMetricsAction',
+    chrome.send('removeSavedPassword', [String(rowIndex)]);
+    chrome.send('coreOptionsUserMetricsAction',
                   ['Options_PasswordManagerDeletePassword']);

[Evan Stade, 2016/04/06 21:25:22]

I think this indent needs to change too

[xunlu, 2016/04/08 15:38:21]

Done.

   };
 
   /**
    * Removes a password exception.
    * @param {number} rowIndex indicating the row to remove.
    */
   PasswordManager.removePasswordException = function(rowIndex) {
-      chrome.send('removePasswordException', [String(rowIndex)]);
+    chrome.send('removePasswordException', [String(rowIndex)]);
   };
 
   PasswordManager.requestShowPassword = function(index) {
     chrome.send('requestShowPassword', [index]);
   };
 
   // Forward public APIs to private implementations on the singleton instance.
   cr.makePublic(PasswordManager, [
     'setSavedPasswordsList',
     'setPasswordExceptionsList',
-    'showPassword'
+    'showPassword',
+    'showImportExportButton'

[Evan Stade, 2016/04/06 21:25:22]

nit: add a final comma
nit 2: alphabetize

[xunlu, 2016/04/08 15:38:21]

Done.

   ]);
 
   // Export
   return {
     PasswordManager: PasswordManager
   };
 
 });