Add functions to the passwords private api that update the lists.

chrome/common/extensions/api/passwords_private.idl

 // 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.
 
 // Use the <code>chrome.passwordsPrivate</code> API to add or remove password
 // data from the settings UI.
 namespace passwordsPrivate {
   // Pair of origin URL and login saved for that URL.
   dictionary LoginPair {
     // The human-readable origin for the URL where the password is used.
@@ skipping 21 matching lines @@
 
   // Dictionary passed to listeners for the onPlaintextPasswordRetrieved event.
   dictionary PlaintextPasswordEventParameters {
     // The LoginPair associated with the retrieved password.
     LoginPair loginPair;
 
     // The password in plaintext.
     DOMString plaintextPassword;
   };
 
+  callback passwordListCallback = void(PasswordUiEntry[] entries);
+  callback exceptionListCallback = void(DOMString[] exceptions);
+
   interface Functions {
     // Removes the saved password corresponding to |loginPair|. If no saved
     // password for this pair exists, this function is a no-op.
     //
     // |loginPair|: The LoginPair corresponding to the entry to remove.
     static void removeSavedPassword(LoginPair loginPair);
 
     // Removes the saved password exception corresponding to |exceptionUrl|. If
     // no exception with this URL exists, this function is a no-op.
     //
     // |exceptionUrl|: The URL corresponding to the exception to remove.
     static void removePasswordException(DOMString exceptionUrl);
 
     // Returns the plaintext password corresponding to |loginPair|. Note that on
     // some operating systems, this call may result in an OS-level
     // reauthentication. Once the password has been fetched, it will be returned
     // via the onPlaintextPasswordRetrieved event.
+    // TODO(hcarmona): Make this use a callback for consistency.

[stevenjb, 2016/04/19 20:59:17]

This is a bit more complicated. I don't remember the details off hand, but that
change might be tricky. Maybe s/Make this use/Investigate using/.

[hcarmona, 2016/04/19 23:07:45]

Sounds good. I haven't investigated this yet, I noticed it was different
and added the TODO.

     //
     // |loginPair|: The LoginPair corresponding to the entry whose password
     //     is to be returned.
     static void requestPlaintextPassword(LoginPair loginPair);
+
+    // Returns the list of saved passwords using |callback|.
+    //
+    // |callback|: Called with the list of saved passwords.
+    static void requestSavedPasswordList(passwordListCallback callback);

[stevenjb, 2016/04/19 20:59:17]

I think these should now be 'get' instead of 'request'.

[hcarmona, 2016/04/19 23:07:45]

Done.

+
+    // Returns the list of password exceptions using |callback|.
+    //
+    // |callback|: Called with the list of password exceptions.
+    static void requestPasswordExceptionList(exceptionListCallback callback);
   };
 
   interface Events {
     // Fired when the saved passwords list has changed, meaning that an entry
     // has been added or removed. Note that this event fires as soon as a
     //  listener is added.
     //
     // |entries|: The updated list of password entries.
     static void onSavedPasswordsListChanged(PasswordUiEntry[] entries);
 
     // Fired when the password exceptions list has changed, meaning that an
     // entry has been added or removed. Note that this event fires as soon as a
     // listener is added.
     //
     // |exceptions|: The updated list of password exceptions.
     static void onPasswordExceptionsListChanged(DOMString[] exceptions);
 
     // Fired when a plaintext password has been fetched in response to a call to
     // chrome.passwordsPrivate.requestPlaintextPassword().
     //
     // |loginPair|: The LoginPair whose password was found.
     // |plaintextPassword|: The plaintext password which was retrieved.
     static void onPlaintextPasswordRetrieved(
         PlaintextPasswordEventParameters dict);
   };
 };