Improve logging in password manager internals page

components/password_manager/core/browser/password_manager_unittest.cc

 // 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.
 
 #include "components/password_manager/core/browser/password_manager.h"
 
 #include <string>
 #include <vector>
 
 #include "base/message_loop/message_loop.h"
@@ skipping 36 matching lines @@
   MOCK_CONST_METHOD0(IsPasswordManagerEnabledForCurrentPage, bool());
   MOCK_CONST_METHOD2(IsSyncAccountCredential,
                      bool(const std::string&, const std::string&));
   MOCK_METHOD1(PromptUserToSavePasswordPtr, void(PasswordFormManager*));
   MOCK_METHOD1(AutomaticPasswordSavePtr, void(PasswordFormManager*));
   MOCK_METHOD0(GetPasswordStore, PasswordStore*());
   MOCK_METHOD0(GetPrefs, PrefService*());
   MOCK_METHOD0(GetDriver, PasswordManagerDriver*());
 
   // Workaround for scoped_ptr<> lacking a copy constructor.
-  virtual void PromptUserToSavePassword(
+  virtual bool PromptUserToSavePassword(
       scoped_ptr<PasswordFormManager> manager) OVERRIDE {
     PromptUserToSavePasswordPtr(manager.release());
+    return false;
   }
   virtual void AutomaticPasswordSave(
       scoped_ptr<PasswordFormManager> manager) OVERRIDE {
     AutomaticPasswordSavePtr(manager.release());
   }
 };
 
 class MockPasswordManagerDriver : public StubPasswordManagerDriver {
  public:
   MOCK_METHOD1(FillPasswordForm, void(const autofill::PasswordFormFillData&));
@@ skipping 751 matching lines @@
   EXPECT_CALL(client_, PromptUserToSavePasswordPtr(_)).Times(0);
 
   // Now the password manager waits for the login to complete successfully.
   observed.clear();
   manager()->OnPasswordFormsParsed(observed);    // The post-navigation load.
   manager()->OnPasswordFormsRendered(observed,
                                      true);  // The post-navigation layout.
 }
 
 }  // namespace password_manager