Fix password manager internals renderer reporting

chrome/browser/password_manager/chrome_password_manager_client.h

 // Copyright 2014 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.
 
 #ifndef CHROME_BROWSER_PASSWORD_MANAGER_CHROME_PASSWORD_MANAGER_CLIENT_H_
 #define CHROME_BROWSER_PASSWORD_MANAGER_CHROME_PASSWORD_MANAGER_CLIENT_H_
 
 #include "base/basictypes.h"
 #include "base/compiler_specific.h"
 #include "base/memory/scoped_ptr.h"
@@ skipping 50 matching lines @@
   void ForceSavePassword() override;
   void NotifyUserAutoSignin(
       ScopedVector<autofill::PasswordForm> local_forms) override;
   void AutomaticPasswordSave(scoped_ptr<password_manager::PasswordFormManager>
                                  saved_form_manager) override;
   void PasswordWasAutofilled(const autofill::PasswordFormMap& best_matches,
                              const GURL& origin) const override;
   PrefService* GetPrefs() override;
   password_manager::PasswordStore* GetPasswordStore() const override;
   password_manager::PasswordSyncState GetPasswordSyncState() const override;
-  void OnLogRouterAvailabilityChanged(bool router_can_be_used) override;
-  void LogSavePasswordProgress(const std::string& text) const override;
-  bool IsLoggingActive() const override;
   bool WasLastNavigationHTTPError() const override;
   bool DidLastPageLoadEncounterSSLErrors() const override;
   bool IsOffTheRecord() const override;
   const password_manager::PasswordManager* GetPasswordManager() const override;
   autofill::AutofillManager* GetAutofillManagerForMainFrame() override;
   const GURL& GetMainFrameURL() const override;
   bool IsUpdatePasswordUIEnabled() const override;
   const GURL& GetLastCommittedEntryURL() const override;
   const password_manager::CredentialsFilter* GetStoreResultFilter()
       const override;
+  const password_manager::LogManager* GetLogManager() const override;
+  password_manager::LogRouter* GetLogRouter() const override;
 
   // Hides any visible generation UI.
   void HidePasswordGenerationPopup();
 
   static void CreateForWebContentsWithAutofillClient(
       content::WebContents* contents,
       autofill::AutofillClient* autofill_client);
 
   // Observer for PasswordGenerationPopup events. Used for testing.
   void SetTestObserver(autofill::PasswordGenerationPopupObserver* observer);
 
+  // Notifies all associated PasswordManagerDriver instances about whether
+  // logging is available.
+  void NotifyDriversAboutLoggingAvailability(bool is_available);

[vasilii, 2015/11/13 12:17:31]

Should be private.

[vabr (Chromium), 2015/11/13 20:48:18]

Done.

+
   // Returns true if the bubble UI is enabled, and false if we're still using
   // the sad old Infobar UI.
   static bool IsTheHotNewBubbleUIEnabled();
 
   // Returns true if the password manager should be enabled during sync signin.
   static bool EnabledForSyncSignin();
 
  protected:
   // Callable for tests.
   ChromePasswordManagerClient(content::WebContents* web_contents,
@@ skipping 25 matching lines @@
 
   // Causes the password editing UI to be shown anchored at |element_bounds|.
   void ShowPasswordEditingPopup(content::RenderFrameHost* render_frame_host,
                                 const gfx::RectF& bounds,
                                 const autofill::PasswordForm& form);
 
   // Notify the PasswordManager that generation is available for |form|. Used
   // for UMA stats.
   void GenerationAvailableForForm(const autofill::PasswordForm& form);
 
-  // Sends a message to the renderer with the current value of
-  // |can_use_log_router_|.
-  void NotifyRendererOfLoggingAvailability();
-
   Profile* const profile_;
 
   password_manager::PasswordManager password_manager_;
 
   password_manager::ContentPasswordManagerDriverFactory* driver_factory_;
 
   password_manager::CredentialManagerDispatcher
       credential_manager_dispatcher_;
 
   // Observer for password generation popup.
   autofill::PasswordGenerationPopupObserver* observer_;
 
   // Controls the popup
   base::WeakPtr<
     autofill::PasswordGenerationPopupControllerImpl> popup_controller_;
 
-  // True if |this| is registered with some LogRouter which can accept logs.
-  bool can_use_log_router_;
-
   // Set to false to disable password saving (will no longer ask if you
   // want to save passwords and also won't fill the passwords).
   BooleanPrefMember saving_and_filling_passwords_enabled_;
 
   const password_manager::SyncStoreResultFilter credentials_filter_;
 
+  scoped_ptr<password_manager::LogManager> log_manager_;
+
   DISALLOW_COPY_AND_ASSIGN(ChromePasswordManagerClient);
 };
 
 #endif  // CHROME_BROWSER_PASSWORD_MANAGER_CHROME_PASSWORD_MANAGER_CLIENT_H_