| Index: components/autofill/core/browser/payments/payments_client.cc
|
| diff --git a/components/autofill/core/browser/payments/payments_client.cc b/components/autofill/core/browser/payments/payments_client.cc
|
| index b039e01e3abcdd1d3ce7e78c432d146ae36da6dd..a519b52b762a8634f0b8f655e2ca0800b227c06d 100644
|
| --- a/components/autofill/core/browser/payments/payments_client.cc
|
| +++ b/components/autofill/core/browser/payments/payments_client.cc
|
| @@ -4,12 +4,13 @@
|
|
|
| #include "components/autofill/core/browser/payments/payments_client.h"
|
|
|
| +#include <memory>
|
| #include <utility>
|
|
|
| #include "base/command_line.h"
|
| #include "base/json/json_reader.h"
|
| #include "base/json/json_writer.h"
|
| -#include "base/memory/scoped_ptr.h"
|
| +#include "base/memory/ptr_util.h"
|
| #include "base/strings/string_number_conversions.h"
|
| #include "base/strings/string_util.h"
|
| #include "base/strings/stringprintf.h"
|
| @@ -95,9 +96,9 @@ GURL GetRequestUrl(const std::string& path) {
|
| return base.Resolve(path);
|
| }
|
|
|
| -scoped_ptr<base::DictionaryValue> BuildRiskDictionary(
|
| +std::unique_ptr<base::DictionaryValue> BuildRiskDictionary(
|
| const std::string& encoded_risk_data) {
|
| - scoped_ptr<base::DictionaryValue> risk_data(new base::DictionaryValue());
|
| + std::unique_ptr<base::DictionaryValue> risk_data(new base::DictionaryValue());
|
| #if defined(OS_IOS)
|
| // Browser fingerprinting is not available on iOS. Instead, we generate
|
| // RiskAdvisoryData.
|
| @@ -132,15 +133,16 @@ void AppendStringIfNotEmpty(const AutofillProfile& profile,
|
| list->AppendString(value);
|
| }
|
|
|
| -scoped_ptr<base::DictionaryValue> BuildAddressDictionary(
|
| +std::unique_ptr<base::DictionaryValue> BuildAddressDictionary(
|
| const AutofillProfile& profile,
|
| const std::string& app_locale) {
|
| - scoped_ptr<base::DictionaryValue> postal_address(new base::DictionaryValue());
|
| + std::unique_ptr<base::DictionaryValue> postal_address(
|
| + new base::DictionaryValue());
|
|
|
| SetStringIfNotEmpty(profile, NAME_FULL, app_locale, "recipient_name",
|
| postal_address.get());
|
|
|
| - scoped_ptr<base::ListValue> address_lines(new base::ListValue());
|
| + std::unique_ptr<base::ListValue> address_lines(new base::ListValue());
|
| AppendStringIfNotEmpty(profile, ADDRESS_HOME_LINE1, app_locale,
|
| address_lines.get());
|
| AppendStringIfNotEmpty(profile, ADDRESS_HOME_LINE2, app_locale,
|
| @@ -162,7 +164,7 @@ scoped_ptr<base::DictionaryValue> BuildAddressDictionary(
|
| if (!country_code.empty())
|
| postal_address->SetString("country_name_code", country_code);
|
|
|
| - scoped_ptr<base::DictionaryValue> address(new base::DictionaryValue());
|
| + std::unique_ptr<base::DictionaryValue> address(new base::DictionaryValue());
|
| address->Set("postal_address", std::move(postal_address));
|
| SetStringIfNotEmpty(profile, PHONE_HOME_WHOLE_NUMBER, app_locale,
|
| "phone_number", address.get());
|
| @@ -191,7 +193,7 @@ class UnmaskCardRequest : public PaymentsRequest {
|
| request_dict.SetString("credit_card_id", request_details_.card.server_id());
|
| request_dict.Set("risk_data_encoded",
|
| BuildRiskDictionary(request_details_.risk_data));
|
| - request_dict.Set("context", make_scoped_ptr(new base::DictionaryValue()));
|
| + request_dict.Set("context", base::WrapUnique(new base::DictionaryValue()));
|
|
|
| int value = 0;
|
| if (base::StringToInt(request_details_.user_response.exp_month, &value))
|
| @@ -211,7 +213,7 @@ class UnmaskCardRequest : public PaymentsRequest {
|
| return request_content;
|
| }
|
|
|
| - void ParseResponse(scoped_ptr<base::DictionaryValue> response) override {
|
| + void ParseResponse(std::unique_ptr<base::DictionaryValue> response) override {
|
| response->GetString("pan", &real_pan_);
|
| }
|
|
|
| @@ -241,7 +243,7 @@ class GetUploadDetailsRequest : public PaymentsRequest {
|
|
|
| std::string GetRequestContent() override {
|
| base::DictionaryValue request_dict;
|
| - scoped_ptr<base::DictionaryValue> context(new base::DictionaryValue());
|
| + std::unique_ptr<base::DictionaryValue> context(new base::DictionaryValue());
|
| context->SetString("language_code", app_locale_);
|
| request_dict.Set("context", std::move(context));
|
|
|
| @@ -251,7 +253,7 @@ class GetUploadDetailsRequest : public PaymentsRequest {
|
| return request_content;
|
| }
|
|
|
| - void ParseResponse(scoped_ptr<base::DictionaryValue> response) override {
|
| + void ParseResponse(std::unique_ptr<base::DictionaryValue> response) override {
|
| response->GetString("context_token", &context_token_);
|
| base::DictionaryValue* unowned_legal_message;
|
| if (response->GetDictionary("legal_message", &unowned_legal_message))
|
| @@ -271,7 +273,7 @@ class GetUploadDetailsRequest : public PaymentsRequest {
|
| private:
|
| std::string app_locale_;
|
| base::string16 context_token_;
|
| - scoped_ptr<base::DictionaryValue> legal_message_;
|
| + std::unique_ptr<base::DictionaryValue> legal_message_;
|
| };
|
|
|
| class UploadCardRequest : public PaymentsRequest {
|
| @@ -294,14 +296,14 @@ class UploadCardRequest : public PaymentsRequest {
|
| BuildRiskDictionary(request_details_.risk_data));
|
|
|
| const std::string& app_locale = request_details_.app_locale;
|
| - scoped_ptr<base::DictionaryValue> context(new base::DictionaryValue());
|
| + std::unique_ptr<base::DictionaryValue> context(new base::DictionaryValue());
|
| context->SetString("language_code", app_locale);
|
| request_dict.Set("context", std::move(context));
|
|
|
| SetStringIfNotEmpty(request_details_.card, CREDIT_CARD_NAME_FULL,
|
| app_locale, "cardholder_name", &request_dict);
|
|
|
| - scoped_ptr<base::ListValue> addresses(new base::ListValue());
|
| + std::unique_ptr<base::ListValue> addresses(new base::ListValue());
|
| for (const AutofillProfile& profile : request_details_.profiles) {
|
| addresses->Append(BuildAddressDictionary(profile, app_locale));
|
| }
|
| @@ -334,7 +336,8 @@ class UploadCardRequest : public PaymentsRequest {
|
| return request_content;
|
| }
|
|
|
| - void ParseResponse(scoped_ptr<base::DictionaryValue> response) override {}
|
| + void ParseResponse(std::unique_ptr<base::DictionaryValue> response) override {
|
| + }
|
|
|
| bool IsResponseComplete() override { return true; }
|
|
|
| @@ -376,19 +379,20 @@ void PaymentsClient::Prepare() {
|
|
|
| void PaymentsClient::UnmaskCard(
|
| const PaymentsClient::UnmaskRequestDetails& request_details) {
|
| - IssueRequest(make_scoped_ptr(new UnmaskCardRequest(request_details)), true);
|
| + IssueRequest(base::WrapUnique(new UnmaskCardRequest(request_details)), true);
|
| }
|
|
|
| void PaymentsClient::GetUploadDetails(const std::string& app_locale) {
|
| - IssueRequest(make_scoped_ptr(new GetUploadDetailsRequest(app_locale)), false);
|
| + IssueRequest(base::WrapUnique(new GetUploadDetailsRequest(app_locale)),
|
| + false);
|
| }
|
|
|
| void PaymentsClient::UploadCard(
|
| const PaymentsClient::UploadRequestDetails& request_details) {
|
| - IssueRequest(make_scoped_ptr(new UploadCardRequest(request_details)), true);
|
| + IssueRequest(base::WrapUnique(new UploadCardRequest(request_details)), true);
|
| }
|
|
|
| -void PaymentsClient::IssueRequest(scoped_ptr<PaymentsRequest> request,
|
| +void PaymentsClient::IssueRequest(std::unique_ptr<PaymentsRequest> request,
|
| bool authenticate) {
|
| request_ = std::move(request);
|
| has_retried_authorization_ = false;
|
| @@ -431,8 +435,8 @@ void PaymentsClient::OnURLFetchComplete(const net::URLFetcher* source) {
|
|
|
| // |url_fetcher_|, which is aliased to |source|, might continue to be used in
|
| // this method, but should be freed once control leaves the method.
|
| - scoped_ptr<net::URLFetcher> scoped_url_fetcher(std::move(url_fetcher_));
|
| - scoped_ptr<base::DictionaryValue> response_dict;
|
| + std::unique_ptr<net::URLFetcher> scoped_url_fetcher(std::move(url_fetcher_));
|
| + std::unique_ptr<base::DictionaryValue> response_dict;
|
| int response_code = source->GetResponseCode();
|
| std::string data;
|
| source->GetResponseAsString(&data);
|
| @@ -444,7 +448,7 @@ void PaymentsClient::OnURLFetchComplete(const net::URLFetcher* source) {
|
| // Valid response.
|
| case net::HTTP_OK: {
|
| std::string error_code;
|
| - scoped_ptr<base::Value> message_value = base::JSONReader::Read(data);
|
| + std::unique_ptr<base::Value> message_value = base::JSONReader::Read(data);
|
| if (message_value.get() &&
|
| message_value->IsType(base::Value::TYPE_DICTIONARY)) {
|
| response_dict.reset(
|
|
|