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

Side by Side Diff: components/password_manager/core/browser/password_form_manager.h

Issue 870513002: [PasswordManager] Improve detection of ignorable change password forms. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Addresses Vaclav's review inputs. Created 5 years, 10 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
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 COMPONENTS_PASSWORD_MANAGER_CORE_BROWSER_PASSWORD_FORM_MANAGER_H_ 5 #ifndef COMPONENTS_PASSWORD_MANAGER_CORE_BROWSER_PASSWORD_FORM_MANAGER_H_
6 #define COMPONENTS_PASSWORD_MANAGER_CORE_BROWSER_PASSWORD_FORM_MANAGER_H_ 6 #define COMPONENTS_PASSWORD_MANAGER_CORE_BROWSER_PASSWORD_FORM_MANAGER_H_
7 7
8 #include <string> 8 #include <string>
9 #include <vector> 9 #include <vector>
10 10
(...skipping 264 matching lines...) Expand 10 before | Expand all | Expand 10 after
275 275
276 // Remove possible_usernames that may contains sensitive information and 276 // Remove possible_usernames that may contains sensitive information and
277 // duplicates. 277 // duplicates.
278 void SanitizePossibleUsernames(autofill::PasswordForm* form); 278 void SanitizePossibleUsernames(autofill::PasswordForm* form);
279 279
280 // Helper function to delegate uploading to the AutofillManager. 280 // Helper function to delegate uploading to the AutofillManager.
281 virtual void UploadPasswordForm( 281 virtual void UploadPasswordForm(
282 const autofill::FormData& form_data, 282 const autofill::FormData& form_data,
283 const autofill::ServerFieldType& password_type); 283 const autofill::ServerFieldType& password_type);
284 284
285 // Returns true if |candidate|'s registry controlled domain for origin and
vabr (Chromium) 2015/02/02 14:18:50 Please update the comment. Also mention that |matc
Pritam Nikam 2015/02/05 06:12:07 Removed this function.
286 // password matches with one of the stored password form within
287 // |best_matches_|; otherwise false. On success, |matching_form| holds the
288 // matching password form.
289 bool HasMatchingCredentialsInStore(
290 const autofill::PasswordForm& candidate,
291 autofill::PasswordForm* matching_form) const;
292
285 // Set of PasswordForms from the DB that best match the form 293 // Set of PasswordForms from the DB that best match the form
286 // being managed by this. Use a map instead of vector, because we most 294 // being managed by this. Use a map instead of vector, because we most
287 // frequently require lookups by username value in IsNewLogin. 295 // frequently require lookups by username value in IsNewLogin.
288 autofill::PasswordFormMap best_matches_; 296 autofill::PasswordFormMap best_matches_;
289 297
290 // Cleans up when best_matches_ goes out of scope. 298 // Cleans up when best_matches_ goes out of scope.
291 STLValueDeleter<autofill::PasswordFormMap> best_matches_deleter_; 299 STLValueDeleter<autofill::PasswordFormMap> best_matches_deleter_;
292 300
293 // The PasswordForm from the page or dialog managed by |this|. 301 // The PasswordForm from the page or dialog managed by |this|.
294 const autofill::PasswordForm observed_form_; 302 const autofill::PasswordForm observed_form_;
(...skipping 58 matching lines...) Expand 10 before | Expand all | Expand 10 after
353 ManagerAction manager_action_; 361 ManagerAction manager_action_;
354 UserAction user_action_; 362 UserAction user_action_;
355 SubmitResult submit_result_; 363 SubmitResult submit_result_;
356 364
357 DISALLOW_COPY_AND_ASSIGN(PasswordFormManager); 365 DISALLOW_COPY_AND_ASSIGN(PasswordFormManager);
358 }; 366 };
359 367
360 } // namespace password_manager 368 } // namespace password_manager
361 369
362 #endif // COMPONENTS_PASSWORD_MANAGER_CORE_BROWSER_PASSWORD_FORM_MANAGER_H_ 370 #endif // COMPONENTS_PASSWORD_MANAGER_CORE_BROWSER_PASSWORD_FORM_MANAGER_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698