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

Side by Side Diff: chrome/browser/password_manager/native_backend_kwallet_x.h

Issue 283563002: Password Login Database: report correct changes from AddLogin(). (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: the nit addressed 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef CHROME_BROWSER_PASSWORD_MANAGER_NATIVE_BACKEND_KWALLET_X_H_ 5 #ifndef CHROME_BROWSER_PASSWORD_MANAGER_NATIVE_BACKEND_KWALLET_X_H_
6 #define CHROME_BROWSER_PASSWORD_MANAGER_NATIVE_BACKEND_KWALLET_X_H_ 6 #define CHROME_BROWSER_PASSWORD_MANAGER_NATIVE_BACKEND_KWALLET_X_H_
7 7
8 #include <string> 8 #include <string>
9 9
10 #include "base/basictypes.h" 10 #include "base/basictypes.h"
(...skipping 22 matching lines...) Expand all
33 // NativeBackend implementation using KWallet. 33 // NativeBackend implementation using KWallet.
34 class NativeBackendKWallet : public PasswordStoreX::NativeBackend { 34 class NativeBackendKWallet : public PasswordStoreX::NativeBackend {
35 public: 35 public:
36 explicit NativeBackendKWallet(LocalProfileId id); 36 explicit NativeBackendKWallet(LocalProfileId id);
37 37
38 virtual ~NativeBackendKWallet(); 38 virtual ~NativeBackendKWallet();
39 39
40 virtual bool Init() OVERRIDE; 40 virtual bool Init() OVERRIDE;
41 41
42 // Implements NativeBackend interface. 42 // Implements NativeBackend interface.
43 virtual bool AddLogin(const autofill::PasswordForm& form) OVERRIDE; 43 virtual password_manager::PasswordStoreChangeList AddLogin(
44 const autofill::PasswordForm& form) OVERRIDE;
44 virtual bool UpdateLogin(const autofill::PasswordForm& form) OVERRIDE; 45 virtual bool UpdateLogin(const autofill::PasswordForm& form) OVERRIDE;
45 virtual bool RemoveLogin(const autofill::PasswordForm& form) OVERRIDE; 46 virtual bool RemoveLogin(const autofill::PasswordForm& form) OVERRIDE;
46 virtual bool RemoveLoginsCreatedBetween( 47 virtual bool RemoveLoginsCreatedBetween(
47 const base::Time& delete_begin, const base::Time& delete_end) OVERRIDE; 48 const base::Time& delete_begin, const base::Time& delete_end) OVERRIDE;
48 virtual bool GetLogins(const autofill::PasswordForm& form, 49 virtual bool GetLogins(const autofill::PasswordForm& form,
49 PasswordFormList* forms) OVERRIDE; 50 PasswordFormList* forms) OVERRIDE;
50 virtual bool GetLoginsCreatedBetween(const base::Time& get_begin, 51 virtual bool GetLoginsCreatedBetween(const base::Time& get_begin,
51 const base::Time& get_end, 52 const base::Time& get_end,
52 PasswordFormList* forms) OVERRIDE; 53 PasswordFormList* forms) OVERRIDE;
53 virtual bool GetAutofillableLogins(PasswordFormList* forms) OVERRIDE; 54 virtual bool GetAutofillableLogins(PasswordFormList* forms) OVERRIDE;
(...skipping 89 matching lines...) Expand 10 before | Expand all | Expand 10 after
143 144
144 // The name of the wallet we've opened. Set during Init(). 145 // The name of the wallet we've opened. Set during Init().
145 std::string wallet_name_; 146 std::string wallet_name_;
146 // The application name (e.g. "Chromium"), shown in KWallet auth dialogs. 147 // The application name (e.g. "Chromium"), shown in KWallet auth dialogs.
147 const std::string app_name_; 148 const std::string app_name_;
148 149
149 DISALLOW_COPY_AND_ASSIGN(NativeBackendKWallet); 150 DISALLOW_COPY_AND_ASSIGN(NativeBackendKWallet);
150 }; 151 };
151 152
152 #endif // CHROME_BROWSER_PASSWORD_MANAGER_NATIVE_BACKEND_KWALLET_X_H_ 153 #endif // CHROME_BROWSER_PASSWORD_MANAGER_NATIVE_BACKEND_KWALLET_X_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698