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

Unified Diff: components/autofill/core/browser/payments/full_card_request.h

Issue 1899893002: Card unmasking without form filling (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Use the credit card number field in metrics tests, because ios single-field form fill will not requ… Created 4 years, 8 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: components/autofill/core/browser/payments/full_card_request.h
diff --git a/components/autofill/core/browser/payments/full_card_request.h b/components/autofill/core/browser/payments/full_card_request.h
new file mode 100644
index 0000000000000000000000000000000000000000..77303d3a1599cd60c2d26738538f664257525734
--- /dev/null
+++ b/components/autofill/core/browser/payments/full_card_request.h
@@ -0,0 +1,94 @@
+// Copyright 2016 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef COMPONENTS_AUTOFILL_CORE_BROWSER_PAYMENTS_FULL_CARD_REQUEST_H_
+#define COMPONENTS_AUTOFILL_CORE_BROWSER_PAYMENTS_FULL_CARD_REQUEST_H_
+
+#include <memory>
+#include <string>
+
+#include "base/macros.h"
+#include "base/memory/weak_ptr.h"
+#include "base/strings/string16.h"
+#include "base/time/time.h"
+#include "components/autofill/core/browser/card_unmask_delegate.h"
+#include "components/autofill/core/browser/payments/payments_client.h"
+
+namespace autofill {
+
+class AutofillClient;
+class CreditCard;
+class PersonalDataManager;
+
+namespace payments {
+
+// Retrieves the full card details, including the pan and the cvc.
+class FullCardRequest : public CardUnmaskDelegate {
+ public:
+ // The interface for receiving the full card details.
+ class Delegate {
+ public:
+ virtual void OnFullCardDetails(const CreditCard& card,
+ const base::string16& cvc) = 0;
+ virtual void OnFullCardError() = 0;
+ };
+
+ // The parameters should outlive the FullCardRequest.
+ FullCardRequest(AutofillClient* autofill_client,
+ payments::PaymentsClient* payments_client,
+ PersonalDataManager* personal_data_manager);
+ ~FullCardRequest();
+
+ // Retrieves the pan and cvc for |card| and invokes
+ // Delegate::OnFullCardDetails() or Delegate::OnFullCardError(). Only one
+ // request should be active at a time.
+ void GetFullCard(const CreditCard& card,
+ AutofillClient::UnmaskCardReason reason,
+ base::WeakPtr<Delegate> delegate);
+
+ // Returns true if there's a pending request to get the full card.
+ bool IsGettingFullCard() const;
+
+ // Called by the payments client when a card has been unmasked.
+ void OnDidGetRealPan(AutofillClient::PaymentsRpcResult result,
+ const std::string& real_pan);
+ private:
+ // CardUnmaskDelegate:
+ void OnUnmaskResponse(const UnmaskResponse& response) override;
+ void OnUnmaskPromptClosed() override;
+
+ // Called by autofill client when the risk data has been loaded.
+ void OnDidGetUnmaskRiskData(const std::string& risk_data);
+
+ // Responsible for showing the UI that prompts the user for the CVC and/or the
+ // updated expiration date.
+ AutofillClient* const autofill_client_;
+
+ // Responsible for unmasking a masked server card.
+ payments::PaymentsClient* const payments_client_;
+
+ // Responsible for updating the server card on disk after it's been unmasked.
+ PersonalDataManager* const personal_data_manager_;
+
+ // Receiver of the full PAN and CVC.
+ base::WeakPtr<Delegate> delegate_;
+
+ // The pending request to get a card's full PAN and CVC.
+ std::unique_ptr<payments::PaymentsClient::UnmaskRequestDetails> request_;
+
+ // The timestamp when the full PAN was requested from a server. For
+ // histograms.
+ base::Time real_pan_request_timestamp_;
+
+ // Enables destroying FullCardRequest while CVC prompt is showing or a server
+ // communication is pending.
+ base::WeakPtrFactory<FullCardRequest> weak_ptr_factory_;
+
+ DISALLOW_COPY_AND_ASSIGN(FullCardRequest);
+};
+
+} // namespace payments
+} // namespace autofill
+
+#endif // COMPONENTS_AUTOFILL_CORE_BROWSER_PAYMENTS_FULL_CARD_REQUEST_H_

Powered by Google App Engine
This is Rietveld 408576698