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

Side by Side Diff: chrome/browser/password_manager/chrome_password_manager_client.cc

Issue 866983003: GetLoginsRequest: Use ScopedVector to express ownership of forms (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@324291_scopedvector
Patch Set: Second fix of the rebase 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 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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 #include "chrome/browser/password_manager/chrome_password_manager_client.h" 5 #include "chrome/browser/password_manager/chrome_password_manager_client.h"
6 6
7 #include "base/bind_helpers.h" 7 #include "base/bind_helpers.h"
8 #include "base/command_line.h" 8 #include "base/command_line.h"
9 #include "base/memory/singleton.h" 9 #include "base/memory/singleton.h"
10 #include "base/metrics/histogram.h" 10 #include "base/metrics/histogram.h"
(...skipping 190 matching lines...) Expand 10 before | Expand all | Expand 10 after
201 password_manager::metrics_util::GroupIdToString( 201 password_manager::metrics_util::GroupIdToString(
202 password_manager::metrics_util::MonitoredDomainGroupId( 202 password_manager::metrics_util::MonitoredDomainGroupId(
203 form_to_save->realm(), GetPrefs()))); 203 form_to_save->realm(), GetPrefs())));
204 SavePasswordInfoBarDelegate::Create( 204 SavePasswordInfoBarDelegate::Create(
205 web_contents(), form_to_save.Pass(), uma_histogram_suffix); 205 web_contents(), form_to_save.Pass(), uma_histogram_suffix);
206 } 206 }
207 return true; 207 return true;
208 } 208 }
209 209
210 bool ChromePasswordManagerClient::PromptUserToChooseCredentials( 210 bool ChromePasswordManagerClient::PromptUserToChooseCredentials(
211 const std::vector<autofill::PasswordForm*>& local_forms, 211 ScopedVector<autofill::PasswordForm> local_forms,
212 const std::vector<autofill::PasswordForm*>& federated_forms, 212 ScopedVector<autofill::PasswordForm> federated_forms,
213 base::Callback<void(const password_manager::CredentialInfo&)> callback) { 213 base::Callback<void(const password_manager::CredentialInfo&)> callback) {
214 // Take ownership of all the password form objects in the forms vectors.
215 ScopedVector<autofill::PasswordForm> local_entries;
216 local_entries.assign(local_forms.begin(), local_forms.end());
217 ScopedVector<autofill::PasswordForm> federated_entries;
218 federated_entries.assign(federated_forms.begin(), federated_forms.end());
219
220 ManagePasswordsUIController* manage_passwords_ui_controller = 214 ManagePasswordsUIController* manage_passwords_ui_controller =
221 ManagePasswordsUIController::FromWebContents(web_contents()); 215 ManagePasswordsUIController::FromWebContents(web_contents());
222 return manage_passwords_ui_controller->OnChooseCredentials( 216 return manage_passwords_ui_controller->OnChooseCredentials(
223 local_entries.Pass(), federated_entries.Pass(), callback); 217 local_forms.Pass(), federated_forms.Pass(), callback);
224 } 218 }
225 219
226 void ChromePasswordManagerClient::AutomaticPasswordSave( 220 void ChromePasswordManagerClient::AutomaticPasswordSave(
227 scoped_ptr<password_manager::PasswordFormManager> saved_form) { 221 scoped_ptr<password_manager::PasswordFormManager> saved_form) {
228 #if defined(OS_ANDROID) 222 #if defined(OS_ANDROID)
229 GeneratedPasswordSavedInfoBarDelegateAndroid::Create(web_contents()); 223 GeneratedPasswordSavedInfoBarDelegateAndroid::Create(web_contents());
230 #else 224 #else
231 if (IsTheHotNewBubbleUIEnabled()) { 225 if (IsTheHotNewBubbleUIEnabled()) {
232 ManagePasswordsUIController* manage_passwords_ui_controller = 226 ManagePasswordsUIController* manage_passwords_ui_controller =
233 ManagePasswordsUIController::FromWebContents(web_contents()); 227 ManagePasswordsUIController::FromWebContents(web_contents());
(...skipping 330 matching lines...) Expand 10 before | Expand all | Expand 10 after
564 autofill_sync_state_ = DISALLOW_SYNC_CREDENTIALS; 558 autofill_sync_state_ = DISALLOW_SYNC_CREDENTIALS;
565 } else { 559 } else {
566 // Allow by default. 560 // Allow by default.
567 autofill_sync_state_ = ALLOW_SYNC_CREDENTIALS; 561 autofill_sync_state_ = ALLOW_SYNC_CREDENTIALS;
568 } 562 }
569 } 563 }
570 564
571 const GURL& ChromePasswordManagerClient::GetMainFrameURL() { 565 const GURL& ChromePasswordManagerClient::GetMainFrameURL() {
572 return web_contents()->GetVisibleURL(); 566 return web_contents()->GetVisibleURL();
573 } 567 }
OLDNEW
« no previous file with comments | « chrome/browser/password_manager/chrome_password_manager_client.h ('k') | chrome/browser/password_manager/password_store_mac.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698