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

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

Issue 228263004: Password manager internals page: Introduce logger in renderer (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Squash in little fixes Created 6 years, 8 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.h
diff --git a/components/password_manager/core/browser/password_manager.h b/components/password_manager/core/browser/password_manager.h
index 05a2cf3b0747dba8330b6a6785c55aeba66687cd..fe969e3ebd337e88507e43fa3cb7cf8ee6b1e434 100644
--- a/components/password_manager/core/browser/password_manager.h
+++ b/components/password_manager/core/browser/password_manager.h
@@ -99,6 +99,8 @@ class PasswordManager : public LoginModel {
virtual void OnPasswordFormSubmitted(
const autofill::PasswordForm& password_form);
+ PasswordManagerClient* client() const { return client_; }
Ilya Sherman 2014/04/09 21:10:08 This method returns a non-const pointer, so the me
vabr (Chromium) 2014/04/10 08:51:25 Done. Personally, I would agree with you if the cl
Ilya Sherman 2014/04/10 09:23:07 You should really never use const on a method that
vabr (Chromium) 2014/04/10 12:15:41 Thanks, Ilya, that's a very good point, and I agre
+
private:
enum ProvisionalSaveFailure {
SAVING_DISABLED,

Powered by Google App Engine
This is Rietveld 408576698