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

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

Issue 1548203002: Convert Pass()→std::move() in //components/[n-z]* (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix bad headers Created 5 years 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.cc
diff --git a/components/password_manager/core/browser/password_manager.cc b/components/password_manager/core/browser/password_manager.cc
index bd851730384755abbff27cc0fea6696d5c4fcd9f..dda28e840931cddbb7a706ee83a2a8deda9afbcc 100644
--- a/components/password_manager/core/browser/password_manager.cc
+++ b/components/password_manager/core/browser/password_manager.cc
@@ -5,8 +5,8 @@
#include "components/password_manager/core/browser/password_manager.h"
#include <stddef.h>
-
#include <map>
+#include <utility>
#include "base/command_line.h"
#include "base/metrics/field_trial.h"
@@ -704,7 +704,7 @@ void PasswordManager::OnLoginSuccessful() {
provisional_save_manager_->password_overridden() ||
provisional_save_manager_->retry_password_form_password_update();
if (client_->PromptUserToSaveOrUpdatePassword(
- provisional_save_manager_.Pass(),
+ std::move(provisional_save_manager_),
CredentialSourceType::CREDENTIAL_SOURCE_PASSWORD_MANAGER,
update_password)) {
if (logger)
@@ -716,7 +716,7 @@ void PasswordManager::OnLoginSuccessful() {
provisional_save_manager_->Save();
if (provisional_save_manager_->has_generated_password()) {
- client_->AutomaticPasswordSave(provisional_save_manager_.Pass());
+ client_->AutomaticPasswordSave(std::move(provisional_save_manager_));
} else {
provisional_save_manager_.reset();
}

Powered by Google App Engine
This is Rietveld 408576698