Password manager internals page: Improve security

chrome/browser/password_manager/chrome_password_manager_client.h

Index: chrome/browser/password_manager/chrome_password_manager_client.h
diff --git a/chrome/browser/password_manager/chrome_password_manager_client.h b/chrome/browser/password_manager/chrome_password_manager_client.h
index 1a8b5c2b38b6ab9fc628cae44714e81d6e216052..d14fb6230b15519019ce662f8df5dce001fa11b7 100644
--- a/chrome/browser/password_manager/chrome_password_manager_client.h
+++ b/chrome/browser/password_manager/chrome_password_manager_client.h
@@ -53,7 +53,8 @@ class ChromePasswordManagerClient
   virtual bool IsPasswordSyncEnabled() OVERRIDE;
   virtual void SetLogger(password_manager::PasswordManagerLogger* logger)
       OVERRIDE;
-  virtual void LogSavePasswordProgress(const std::string& text) OVERRIDE;
+  virtual void LogSavePasswordProgress(const std::vector<
+      autofill::SavePasswordProgressLogger::StructuredLog>& logs) OVERRIDE;

[Ilya Sherman, 2014/04/12 00:00:51]

Wow, is this really how clang-format formats this line?  That's not great :/

[vabr (Chromium), 2014/04/12 09:11:38]

I can confirm this was clang-formated.
Have you suggestions for a better split? I actually don't see any, the type name
got monstrously long. 

[vabr (Chromium), 2014/04/16 20:55:36]

Resolved: The type got shorter again.
On 2014/04/12 09:11:38, vabr (Chromium) wrote:

   virtual bool IsLoggingActive() const OVERRIDE;
 
   // Hides any visible generation UI.