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

chrome/browser/extensions/api/passwords_private/passwords_private_api.cc

 // 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.
 
 #include "chrome/browser/extensions/api/passwords_private/passwords_private_api.h"
 
 #include "base/values.h"
 #include "chrome/browser/extensions/api/passwords_private/passwords_private_delegate_factory.h"
 #include "chrome/common/extensions/api/passwords_private.h"
 #include "components/password_manager/core/common/experiments.h"
@@ skipping 67 matching lines @@
 
   delegate->RequestShowPassword(parameters->login_pair.origin_url,
                                 parameters->login_pair.username,
                                 GetSenderWebContents());
 
   // No response given from this API function; instead, listeners wait for the
   // chrome.passwordsPrivate.onPlaintextPasswordRetrieved event to fire.
   return RespondNow(NoArguments());
 }
 
+////////////////////////////////////////////////////////////////////////////////
+// PasswordsPrivateRequestSavedPasswordListFunction
+
+PasswordsPrivateRequestSavedPasswordListFunction::
+    ~PasswordsPrivateRequestSavedPasswordListFunction() {}
+
+ExtensionFunction::ResponseAction
+PasswordsPrivateRequestSavedPasswordListFunction::Run() {
+  PasswordsPrivateDelegate* delegate =
+      PasswordsPrivateDelegateFactory::GetForBrowserContext(browser_context(),
+                                                            true /* create */);
+  std::unique_ptr<base::ListValue> event_args =
+      api::passwords_private::OnSavedPasswordsListChanged::Create(
+          *(delegate->GetSavedPasswordsList()));
+  base::ListValue* entries = nullptr;
+  event_args->GetList(0, &entries);

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

I believe that if this is called with no entries argument this will crash
chrome? It would be better to call SetError("Invalid arguments") and return
false if event_args is null.

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

I fixed my issue. I was creating an event object and taking the first
param from it. I should have been using the result creation method for
my new API functions. Updated.

[stevenjb, 2016/04/19 23:13:38]

Ah. Yes, this is better.

+  return RespondNow(OneArgument(entries->CreateDeepCopy()));
+}
+
+////////////////////////////////////////////////////////////////////////////////
+// PasswordsPrivateRequestPasswordExceptionListFunction
+
+PasswordsPrivateRequestPasswordExceptionListFunction::
+    ~PasswordsPrivateRequestPasswordExceptionListFunction() {}
+
+ExtensionFunction::ResponseAction
+PasswordsPrivateRequestPasswordExceptionListFunction::Run() {
+  PasswordsPrivateDelegate* delegate =
+      PasswordsPrivateDelegateFactory::GetForBrowserContext(browser_context(),
+                                                            true /* create */);
+  std::unique_ptr<base::ListValue> event_args =
+      api::passwords_private::OnPasswordExceptionsListChanged::Create(
+          *(delegate->GetPasswordExceptionsList()));
+  base::ListValue* exceptions = nullptr;
+  event_args->GetList(0, &exceptions);

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

Same here.

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

Same changes here.

+  return RespondNow(OneArgument(exceptions->CreateDeepCopy()));
+}
+
 }  // namespace extensions