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

Unified Diff: chrome/browser/autofill/wallet/wallet_client.cc

Issue 12221040: Interactive autofill: Handle Online Wallet being unavailable. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: . Created 7 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/autofill/wallet/wallet_client.cc
diff --git a/chrome/browser/autofill/wallet/wallet_client.cc b/chrome/browser/autofill/wallet/wallet_client.cc
index ea5e381bb5e3540eaa854ed341731075d0fc7811..bf2f7b26d88a8538a90086d3f1a99765f26bc059 100644
--- a/chrome/browser/autofill/wallet/wallet_client.cc
+++ b/chrome/browser/autofill/wallet/wallet_client.cc
@@ -244,11 +244,15 @@ void WalletClient::SendAutocheckoutStatus(
MakeWalletRequest(GetSendStatusUrl(), post_body, observer, kJsonMimeType);
}
+bool WalletClient::IsRequestInProgress() const {
+ return request_.get() != NULL;
+}
+
void WalletClient::MakeWalletRequest(const GURL& url,
const std::string& post_body,
WalletClientObserver* observer,
const std::string& content_type) {
- DCHECK(!request_.get()) << "Tried to fetch two things at once!";
+ DCHECK(!IsRequestInProgress()) << "Tried to fetch two things at once!";
Ilya Sherman 2013/02/11 06:07:01 nit: As long as you're here, please remove the log
Dan Beam 2013/02/11 21:53:54 Done.
DCHECK(observer);
observer_ = observer;

Powered by Google App Engine
This is Rietveld 408576698