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

Unified Diff: chrome/browser/autofill/wallet/full_wallet.h

Issue 11777007: Adds wallet::RequiredAction for when we start interacting with Online Wallet. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: isherman@ review Created 7 years, 11 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/full_wallet.h
diff --git a/chrome/browser/autofill/wallet/full_wallet.h b/chrome/browser/autofill/wallet/full_wallet.h
index 848c2e162922b8aeba192b08dd157dc8e43dbb4e..9e9c94520bc86da675ad28ab62c24457caaef2ae 100644
--- a/chrome/browser/autofill/wallet/full_wallet.h
+++ b/chrome/browser/autofill/wallet/full_wallet.h
@@ -11,6 +11,7 @@
#include "base/basictypes.h"
#include "base/gtest_prod_util.h"
#include "base/memory/scoped_ptr.h"
+#include "chrome/browser/autofill/wallet/required_action.h"
#include "chrome/browser/autofill/wallet/wallet_address.h"
namespace base {
@@ -22,7 +23,10 @@ namespace wallet {
class FullWalletTest;
// FullWallet contains all the information a merchant requires from a user for
-// that user to make a purchase.
+// that user to make a purchase. This includes:
+// - billing information
+// - shipping information
+// - a proxy card for the "real" card selected from a user's wallet items
ahutter 2013/01/07 17:50:11 backing card.
Dan Beam 2013/01/07 18:29:08 Done.
class FullWallet {
public:
~FullWallet();
@@ -50,7 +54,7 @@ class FullWallet {
// |shipping_address_| might contain NULL.
const Address* shipping_address() const { return shipping_address_.get(); }
- const std::vector<std::string>& required_actions() const {
+ const std::vector<RequiredAction>& required_actions() const {
return required_actions_;
}
int expiration_month() const { return expiration_month_; }
@@ -66,7 +70,7 @@ class FullWallet {
const std::string& encrypted_rest,
scoped_ptr<Address> billing_address,
scoped_ptr<Address> shipping_address,
- const std::vector<std::string>& required_actions);
+ const std::vector<RequiredAction>& required_actions);
void DecryptCardInfo(uint8* otp, size_t length);
int expiration_month_;
int expiration_year_;
@@ -82,9 +86,7 @@ class FullWallet {
scoped_ptr<Address> shipping_address_;
// Actions that must be completed by the user before a FullWallet can be
// issued to them by the Online Wallet service.
- // TODO(ahutter): |required_actions_| should be members of an enum not
- // strings. See http://crbug.com/165195.
- std::vector<std::string> required_actions_;
+ std::vector<RequiredAction> required_actions_;
DISALLOW_COPY_AND_ASSIGN(FullWallet);
};
« no previous file with comments | « no previous file | chrome/browser/autofill/wallet/full_wallet.cc » ('j') | chrome/browser/autofill/wallet/full_wallet.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698