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

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

Issue 1415533013: Fix password manager internals renderer reporting (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: All comments addressed Created 5 years, 1 month 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/log_manager.h
diff --git a/components/password_manager/core/browser/log_manager.h b/components/password_manager/core/browser/log_manager.h
new file mode 100644
index 0000000000000000000000000000000000000000..875f0b8d42124e8acab2d9629bc3fd61f599bccd
--- /dev/null
+++ b/components/password_manager/core/browser/log_manager.h
@@ -0,0 +1,51 @@
+// Copyright 2015 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 COMPONENTS_PASSWORD_MANAGER_CORE_BROWSER_LOG_MANAGER_H_
+#define COMPONENTS_PASSWORD_MANAGER_CORE_BROWSER_LOG_MANAGER_H_
+
+#include <string>
+
+#include "base/callback.h"
+#include "base/macros.h"
+#include "base/memory/scoped_ptr.h"
+
+namespace password_manager {
+
+class LogRouter;
+
+// This interface is used by the password management code to receive and display
+// logs about progress of actions like saving a password.
+class LogManager {
+ public:
+ virtual ~LogManager() = default;
+
+ // This method is called by a LogRouter, after the LogManager registers with
+ // one. If |router_can_be_used| is true, logs can be sent to LogRouter after
+ // the return from OnLogRouterAvailabilityChanged and will reach at least one
+ // LogReceiver instance. If |router_can_be_used| is false, no logs should be
+ // sent to the LogRouter.
+ virtual void OnLogRouterAvailabilityChanged(bool router_can_be_used) = 0;
+
+ // The owner of the LogManager can call this to start or end suspending the
+ // logging, by setting |suspended| to true or false, respectively.
+ virtual void SetSuspended(bool suspended) = 0;
+
+ // Forward |text| for display to the LogRouter (if registered with one).
+ virtual void LogSavePasswordProgress(const std::string& text) const = 0;
+
+ // Returns true if logs recorded via LogSavePasswordProgress will be
+ // displayed, and false otherwise.
+ virtual bool IsLoggingActive() const = 0;
+
+ // Returns the production code implementation of LogManager. If |log_router|
+ // is null, the manager will do nothing. |notification_callback| will be
+ // called every time the activity status of logging changes.
+ static scoped_ptr<LogManager> Create(LogRouter* log_router,
+ base::Closure notification_callback);
+};
+
+} // namespace password_manager
+
+#endif // COMPONENTS_PASSWORD_MANAGER_CORE_BROWSER_LOG_MANAGER_H_

Powered by Google App Engine
This is Rietveld 408576698