Implement logging of the Credential Manager API calls to chrome://password-manager-internals.

components/password_manager/content/browser/credential_manager_impl.cc

Index: components/password_manager/content/browser/credential_manager_impl.cc
diff --git a/components/password_manager/content/browser/credential_manager_impl.cc b/components/password_manager/content/browser/credential_manager_impl.cc
index 9a0a1b43c757a16a6eb96d63b8a7b4a415d304f2..0a8d57cde0903278b6a82a395e640a4c63b51370 100644
--- a/components/password_manager/content/browser/credential_manager_impl.cc
+++ b/components/password_manager/content/browser/credential_manager_impl.cc
@@ -14,6 +14,7 @@
 #include "components/password_manager/content/browser/content_password_manager_driver.h"
 #include "components/password_manager/content/browser/content_password_manager_driver_factory.h"
 #include "components/password_manager/core/browser/affiliated_match_helper.h"
+#include "components/password_manager/core/browser/credential_manager_logger.h"
 #include "components/password_manager/core/browser/password_manager_client.h"
 #include "components/password_manager/core/browser/password_store.h"
 #include "components/password_manager/core/common/credential_manager_types.h"
@@ -52,6 +53,10 @@ void CredentialManagerImpl::Store(const CredentialInfo& credential,
                                   const StoreCallback& callback) {
   DCHECK_NE(CredentialType::CREDENTIAL_TYPE_EMPTY, credential.type);
 
+  CredentialManagerLogger(client_->GetLogManager())

[vabr (Chromium), 2016/08/29 13:10:44]

For all the current logging the code checks LogManager::IsLoggingActive() as
soon as possible, so that the performance impact on the code is minimised in
cases when the internals page is not open. Please follow that pattern here as
well, do not construct the logger before you know you need it.

[vasilii, 2016/08/29 15:12:39]

Done.

+      .LogStoreCredential(web_contents()->GetLastCommittedURL(),
+                          credential.type);
+
   // Send acknowledge response back.
   callback.Run();
 
@@ -104,6 +109,8 @@ void CredentialManagerImpl::OnProvisionalSaveComplete() {
 
 void CredentialManagerImpl::RequireUserMediation(
     const RequireUserMediationCallback& callback) {
+  CredentialManagerLogger(client_->GetLogManager())
+      .LogRequireUserMediation(web_contents()->GetLastCommittedURL());
   PasswordStore* store = GetPasswordStore();
   if (!store || !IsUpdatingCredentialAllowed()) {
     callback.Run();
@@ -150,6 +157,9 @@ void CredentialManagerImpl::Get(bool zero_click_only,
                                 const std::vector<GURL>& federations,
                                 const GetCallback& callback) {
   PasswordStore* store = GetPasswordStore();
+  CredentialManagerLogger(client_->GetLogManager())
+      .LogRequestCredential(web_contents()->GetLastCommittedURL(),
+                            zero_click_only, federations);
   if (pending_request_ || !store) {
     // Callback error.
     callback.Run(pending_request_
@@ -225,6 +235,8 @@ void CredentialManagerImpl::SendCredential(
   DCHECK(pending_request_);
   DCHECK(send_callback.Equals(pending_request_->send_callback()));
 
+  CredentialManagerLogger(client_->GetLogManager())
+      .LogSendCredential(web_contents()->GetLastCommittedURL(), info.type);
   send_callback.Run(info);
   pending_request_.reset();
 }