| Index: components/password_manager/core/browser/password_form_manager.cc
|
| diff --git a/components/password_manager/core/browser/password_form_manager.cc b/components/password_manager/core/browser/password_form_manager.cc
|
| index 959e5a7016f97eb2dcc22998ce7c5cf8e20eb502..247d539498daa6df089b315eec3ee742a2dfb698 100644
|
| --- a/components/password_manager/core/browser/password_form_manager.cc
|
| +++ b/components/password_manager/core/browser/password_form_manager.cc
|
| @@ -11,6 +11,7 @@
|
| #include <set>
|
| #include <utility>
|
|
|
| +#include "base/memory/ptr_util.h"
|
| #include "base/metrics/histogram_macros.h"
|
| #include "base/strings/string_split.h"
|
| #include "base/strings/string_util.h"
|
| @@ -93,7 +94,7 @@ bool IsProbablyNotUsername(const base::string16& s) {
|
|
|
| // Splits federated matches from |store_results| into a separate vector and
|
| // returns that.
|
| -std::vector<scoped_ptr<autofill::PasswordForm>> SplitFederatedMatches(
|
| +std::vector<std::unique_ptr<autofill::PasswordForm>> SplitFederatedMatches(
|
| ScopedVector<PasswordForm>* store_results) {
|
| auto first_federated =
|
| std::partition(store_results->begin(), store_results->end(),
|
| @@ -101,11 +102,11 @@ std::vector<scoped_ptr<autofill::PasswordForm>> SplitFederatedMatches(
|
| return form->federation_origin.unique();
|
| });
|
|
|
| - std::vector<scoped_ptr<autofill::PasswordForm>> federated_matches;
|
| + std::vector<std::unique_ptr<autofill::PasswordForm>> federated_matches;
|
| federated_matches.reserve(store_results->end() - first_federated);
|
| for (auto federated = first_federated; federated != store_results->end();
|
| ++federated) {
|
| - federated_matches.push_back(make_scoped_ptr(*federated));
|
| + federated_matches.push_back(base::WrapUnique(*federated));
|
| *federated = nullptr;
|
| }
|
| store_results->weak_erase(first_federated, store_results->end());
|
| @@ -279,7 +280,7 @@ void PasswordFormManager::ProvisionallySave(
|
| DCHECK(state_ == MATCHING_PHASE || state_ == POST_MATCHING_PHASE) << state_;
|
| DCHECK_NE(RESULT_NO_MATCH, DoesManage(credentials));
|
|
|
| - scoped_ptr<autofill::PasswordForm> mutable_provisionally_saved_form(
|
| + std::unique_ptr<autofill::PasswordForm> mutable_provisionally_saved_form(
|
| new PasswordForm(credentials));
|
| if (credentials.IsPossibleChangePasswordForm() &&
|
| !credentials.username_value.empty() &&
|
| @@ -339,7 +340,7 @@ void PasswordFormManager::FetchDataFromPasswordStore() {
|
| return;
|
| }
|
|
|
| - scoped_ptr<BrowserSavePasswordProgressLogger> logger;
|
| + std::unique_ptr<BrowserSavePasswordProgressLogger> logger;
|
| if (password_manager_util::IsLoggingActive(client_)) {
|
| logger.reset(
|
| new BrowserSavePasswordProgressLogger(client_->GetLogManager()));
|
| @@ -411,7 +412,7 @@ void PasswordFormManager::OnRequestDone(
|
| blacklisted_matches_.clear();
|
| const size_t logins_result_size = logins_result.size();
|
|
|
| - scoped_ptr<BrowserSavePasswordProgressLogger> logger;
|
| + std::unique_ptr<BrowserSavePasswordProgressLogger> logger;
|
| if (password_manager_util::IsLoggingActive(client_)) {
|
| logger.reset(
|
| new BrowserSavePasswordProgressLogger(client_->GetLogManager()));
|
| @@ -458,7 +459,7 @@ void PasswordFormManager::OnRequestDone(
|
| // Fill |best_matches_| with the best-scoring credentials for each username.
|
| for (size_t i = 0; i < logins_result.size(); ++i) {
|
| // Take ownership of the PasswordForm from the ScopedVector.
|
| - scoped_ptr<PasswordForm> login(logins_result[i]);
|
| + std::unique_ptr<PasswordForm> login(logins_result[i]);
|
| logins_result[i] = nullptr;
|
| DCHECK(!login->blacklisted_by_user);
|
| const base::string16& username = login->username_value;
|
| @@ -558,7 +559,7 @@ void PasswordFormManager::OnGetPasswordStoreResults(
|
| return;
|
| }
|
|
|
| - scoped_ptr<BrowserSavePasswordProgressLogger> logger;
|
| + std::unique_ptr<BrowserSavePasswordProgressLogger> logger;
|
| if (password_manager_util::IsLoggingActive(client_)) {
|
| logger.reset(
|
| new BrowserSavePasswordProgressLogger(client_->GetLogManager()));
|
| @@ -585,7 +586,7 @@ void PasswordFormManager::OnGetPasswordStoreResults(
|
| }
|
|
|
| void PasswordFormManager::OnGetSiteStatistics(
|
| - scoped_ptr<std::vector<scoped_ptr<InteractionsStats>>> stats) {
|
| + std::unique_ptr<std::vector<std::unique_ptr<InteractionsStats>>> stats) {
|
| // On Windows the password request may be resolved after the statistics due to
|
| // importing from IE.
|
| DCHECK(state_ == MATCHING_PHASE || state_ == POST_MATCHING_PHASE) << state_;
|
|
|