Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(610)

Unified Diff: components/password_manager/core/browser/password_manager_client.h

Issue 269513003: Password manager internals page service: wiring it in (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Adding forgotten s/NotifyCanUseLogRouter/OnLogRouterAvailabilityChanged in tests Created 6 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: components/password_manager/core/browser/password_manager_client.h
diff --git a/components/password_manager/core/browser/password_manager_client.h b/components/password_manager/core/browser/password_manager_client.h
index 37938650373804e2c467d99420ac2b60ada07df4..e1c765e4507554816cc5d0807f9f96572f568ca6 100644
--- a/components/password_manager/core/browser/password_manager_client.h
+++ b/components/password_manager/core/browser/password_manager_client.h
@@ -16,7 +16,6 @@ namespace password_manager {
class PasswordFormManager;
class PasswordManagerDriver;
class PasswordStore;
-class PasswordManagerLogger;
// An abstraction of operations that depend on the embedders (e.g. Chrome)
// environment.
@@ -73,10 +72,13 @@ class PasswordManagerClient {
// implementation returns false.
virtual bool IsPasswordSyncEnabled();
- // Attach or detach (setting NULL) a logger for this client.
- virtual void SetLogger(PasswordManagerLogger* logger);
+ // Only for clients which registered with a LogRouter: If called with
+ // |router_can_be_used| set to false, the client may no longer use the
+ // LogRouter. If |router_can_be_used| is true, the LogRouter can be used after
+ // the return from OnLogRouterAvailabilityChanged.
+ virtual void OnLogRouterAvailabilityChanged(bool router_can_be_used);
- // Send |text| to the logger.
+ // Forward |text| for display to the LogRouter (if registered with one).
virtual void LogSavePasswordProgress(const std::string& text);
// Returns true if logs recorded via LogSavePasswordProgress will be

Powered by Google App Engine
This is Rietveld 408576698