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

Side by Side Diff: components/autofill/content/browser/wallet/wallet_client.cc

Issue 151383003: Updates to wallet for i18n (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fix blasted unit tests Created 6 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 | Annotate | Revision Log
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 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 "components/autofill/content/browser/wallet/wallet_client.h" 5 #include "components/autofill/content/browser/wallet/wallet_client.h"
6 6
7 #include "base/bind.h" 7 #include "base/bind.h"
8 #include "base/json/json_reader.h" 8 #include "base/json/json_reader.h"
9 #include "base/json/json_writer.h" 9 #include "base/json/json_writer.h"
10 #include "base/logging.h" 10 #include "base/logging.h"
(...skipping 534 matching lines...) Expand 10 before | Expand all | Expand 10 after
545 545
546 DCHECK_EQ(source, request_.get()); 546 DCHECK_EQ(source, request_.get());
547 DVLOG(1) << "Got response from " << source->GetOriginalURL(); 547 DVLOG(1) << "Got response from " << source->GetOriginalURL();
548 548
549 // |request_|, which is aliased to |source|, might continue to be used in this 549 // |request_|, which is aliased to |source|, might continue to be used in this
550 // |method, but should be freed once control leaves the method. 550 // |method, but should be freed once control leaves the method.
551 scoped_ptr<net::URLFetcher> scoped_request(request_.Pass()); 551 scoped_ptr<net::URLFetcher> scoped_request(request_.Pass());
552 552
553 std::string data; 553 std::string data;
554 source->GetResponseAsString(&data); 554 source->GetResponseAsString(&data);
555 DVLOG(1) << "Response body: " << data; 555 NOTIMPLEMENTED() << "Response body: " << data;
Dan Beam 2014/02/12 21:34:33 revert
Evan Stade 2014/02/12 23:32:13 you unluckily reviewed an old patchset
556 556
557 scoped_ptr<base::DictionaryValue> response_dict; 557 scoped_ptr<base::DictionaryValue> response_dict;
558 558
559 int response_code = source->GetResponseCode(); 559 int response_code = source->GetResponseCode();
560 delegate_->GetMetricLogger().LogWalletResponseCode(response_code); 560 delegate_->GetMetricLogger().LogWalletResponseCode(response_code);
561 561
562 switch (response_code) { 562 switch (response_code) {
563 // HTTP_BAD_REQUEST means the arguments are invalid. No point retrying. 563 // HTTP_BAD_REQUEST means the arguments are invalid. No point retrying.
564 case net::HTTP_BAD_REQUEST: { 564 case net::HTTP_BAD_REQUEST: {
565 request_type_ = NO_REQUEST; 565 request_type_ = NO_REQUEST;
(...skipping 210 matching lines...) Expand 10 before | Expand all | Expand 10 after
776 NOTREACHED(); 776 NOTREACHED();
777 return AutofillMetrics::UNKNOWN_API_CALL; 777 return AutofillMetrics::UNKNOWN_API_CALL;
778 } 778 }
779 779
780 NOTREACHED(); 780 NOTREACHED();
781 return AutofillMetrics::UNKNOWN_API_CALL; 781 return AutofillMetrics::UNKNOWN_API_CALL;
782 } 782 }
783 783
784 } // namespace wallet 784 } // namespace wallet
785 } // namespace autofill 785 } // namespace autofill
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698