Integrating Online Wallet into Chrome.

chrome/browser/autofill/wallet/wallet_items.cc

+// Copyright (c) 2012 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.
+
+#include "chrome/browser/autofill/wallet/wallet_items.h"
+
+#include "base/logging.h"
+#include "base/values.h"
+
+namespace wallet {
+
+WalletItems::MaskedInstrument::MaskedInstrument(
+    const std::string& descriptive_name,
+    WalletItems::MaskedInstrument::Type type,
+    std::vector<std::string> supported_currencies,
+    const std::string& last_four_digits,
+    int expiration_month,
+    int expiration_year,
+    const std::string& brand,
+    Address* address,
+    WalletItems::MaskedInstrument::Status status,
+    const std::string& object_id)
+    : descriptive_name_(descriptive_name),
+      type_(type),
+      supported_currencies_(supported_currencies),
+      last_four_digits_(last_four_digits),
+      expiration_month_(expiration_month),
+      expiration_year_(expiration_year),
+      brand_(brand),
+      address_(address),
+      status_(status),
+      object_id_(object_id) {}
+
+WalletItems::MaskedInstrument::~MaskedInstrument() {}
+
+WalletItems::MaskedInstrument*
+WalletItems::MaskedInstrument::CreateMaskedInstrument(

[Dan Beam, 2012/12/01 01:19:49]

nit: I think this should be 4 \s indented ?

[ahutter, 2012/12/01 04:06:51]

Done.

+    base::DictionaryValue* dictionary) {
+  DCHECK(dictionary);
+  std::string descriptive_name;
+  if (!dictionary->GetString("descriptive_name", &descriptive_name))
+    VLOG(1) << "Response from Google Wallet missing descriptive name";
+  std::string type_string;
+  Type type;
+  if (dictionary->GetString("type", &type_string)) {
+    type = TypeFromString(type_string);
+  } else {
+    DLOG(ERROR) << "Response from Google Wallet missing card type";
+    return NULL;
+  }
+  std::vector<std::string> supported_currencies;
+  ListValue* supported_currency_list;
+  if (dictionary->GetList("supported_currency", &supported_currency_list)) {
+    for (size_t i = 0; i < supported_currency_list->GetSize(); i++) {
+      std::string currency;
+      if (supported_currency_list->GetString(i, &currency))

[Dan Beam, 2012/12/01 01:19:49]

how would this fail?

[ahutter, 2012/12/01 04:06:51]

If the element at the i-th position wasn't a string.

+        supported_currencies.push_back(currency);
+    }
+  } else {
+    VLOG(1) << "Response from Google Wallet missing supported currency";
+  }
+  std::string last_four_digits;
+  if (!dictionary->GetString("last_four_digits", &last_four_digits)) {
+    DLOG(ERROR) << "Response from Google Wallet missing last four digits";
+    return NULL;
+  }
+  int expiration_month;
+  if (!dictionary->GetInteger("expiration_month", &expiration_month))
+    VLOG(1) << "Response from Google Wallet missing expiration month";
+  int expiration_year;
+  if (!dictionary->GetInteger("expiration_year", &expiration_year))
+    VLOG(1) << "Response from Google Wallet missing expiration year";
+  std::string brand;
+  if (!dictionary->GetString("brand", &brand))
+    VLOG(1) << "Response from Google Wallet missing brand";
+  std::string status_string;
+  Status status;
+  if (dictionary->GetString("status", &status_string)) {
+    status = StatusFromString(status_string);
+  } else {
+    DLOG(ERROR) << "Response from Google Wallet missing status";
+    return NULL;
+  }
+  std::string object_id;
+  if (!dictionary->GetString("object_id", &object_id)) {
+    DLOG(ERROR) << "Response from Google Wallet missing object id";
+    return NULL;
+  }
+  DictionaryValue* address_dict;
+  if (!dictionary->GetDictionary("billing_address", &address_dict)) {
+    DLOG(ERROR) << "Response from Google wallet missing address";
+    return NULL;
+  }
+  Address* address = Address::CreateDisplayAddress(address_dict);
+  if (!address) {
+    DLOG(ERROR) << "Response from Google wallet contained malformed address";
+    return NULL;
+  }

[Dan Beam, 2012/12/01 01:19:49]

60 lines without a \n makes baby seals cry :_(

[ahutter, 2012/12/01 04:06:51]

Sorry.  I'm fixing it I swear.

+
+  return new MaskedInstrument(descriptive_name,
+                              type,
+                              supported_currencies,
+                              last_four_digits,
+                              expiration_month,
+                              expiration_year,
+                              brand,
+                              address,
+                              status,
+                              object_id);
+}
+
+bool WalletItems::MaskedInstrument::operator==(
+    const WalletItems::MaskedInstrument& other) const {
+  if (descriptive_name_.compare(other.descriptive_name_) != 0)
+    return false;
+  if (type_ != other.type_)
+    return false;
+  if (supported_currencies_ != other.supported_currencies_)
+    return false;
+  if (last_four_digits_.compare(other.last_four_digits_) != 0)
+    return false;
+  if (expiration_month_ != other.expiration_month_)
+    return false;
+  if (expiration_year_ != other.expiration_year_)
+    return false;
+  if (brand_.compare(other.brand_) != 0)
+    return false;
+  if (address_.get()) {
+    if (other.address_.get()) {
+      if (*address_.get() != *other.address_.get())
+        return false;
+    } else {
+      return false;
+    }
+  } else if (other.address_.get()) {
+    return false;
+  }
+  if (status_ != other.status_)
+    return false;
+  if (object_id_.compare(other.object_id_) != 0)
+    return false;
+  return true;
+}
+
+bool WalletItems::MaskedInstrument::operator!=(
+    const WalletItems::MaskedInstrument& other) const {
+  return !(*this == other);
+}
+
+WalletItems::MaskedInstrument::Type
+WalletItems::MaskedInstrument::TypeFromString(
+    const std::string& type_string) {
+  if (type_string.compare("VISA") == 0)

[Dan Beam, 2012/12/01 01:19:49]

I think my == comment applies to here, as well (== > .compare() == 0)

[ahutter, 2012/12/01 04:06:51]

Done.

+    return VISA;
+  if (type_string.compare("MASTER_CARD") == 0)
+    return MASTER_CARD;
+  if (type_string.compare("AMEX") == 0)
+    return AMEX;
+  if (type_string.compare("DISCOVER") == 0)
+    return DISCOVER;
+  if (type_string.compare("SOLO") == 0)
+    return SOLO;
+  if (type_string.compare("MAESTRO") == 0)
+    return MAESTRO;
+  if (type_string.compare("SWITCH") == 0)
+    return SWITCH;

[Dan Beam, 2012/12/01 01:19:49]

nit: should there be a NOTREACHED() here? do we expect to get here often?

[ahutter, 2012/12/01 04:06:51]

UNKNOWN is an actual type coming from Wallet so it's possible.

+  return UNKNOWN;
+}
+
+WalletItems::MaskedInstrument::Status
+WalletItems::MaskedInstrument::StatusFromString(
+    const std::string& status_string) {
+  if (status_string.compare("PENDING") == 0)
+    return PENDING;
+  if (status_string.compare("VALID") == 0)
+    return VALID;
+  if (status_string.compare("DECLINED") == 0)
+    return DECLINED;
+  if (status_string.compare("UNSUPPORTED_COUNTRY") == 0)
+    return UNSUPPORTED_COUNTRY;
+  if (status_string.compare("EXPIRED") == 0)
+    return EXPIRED;
+  if (status_string.compare("BILLING_INCOMPLETE") == 0)
+    return BILLING_INCOMPLETE;

[Dan Beam, 2012/12/01 01:19:49]

nit: same question about NOTREACHED()

[ahutter, 2012/12/01 04:06:51]

Same answer.

+  return INAPPLICABLE;
+}
+
+WalletItems::LegalDocument::LegalDocument(const std::string& document_id,
+              const std::string& display_name,
+              const std::string& document_body)

[Dan Beam, 2012/12/01 01:19:49]

nit: indent is off

[ahutter, 2012/12/01 04:06:51]

Done.

+    : document_id_(document_id),
+      display_name_(display_name),
+      document_body_(document_body) {}
+
+WalletItems::LegalDocument::~LegalDocument() {}
+
+WalletItems::LegalDocument* WalletItems::LegalDocument::CreateLegalDocument(
+    base::DictionaryValue* dictionary) {
+  DCHECK(dictionary);
+
+  std::string document_id;
+  if (!dictionary->GetString("legal_document_id", &document_id)) {
+    DLOG(ERROR) << "Response from Google Wallet missing legal document id";
+    return NULL;
+  }
+  std::string display_name;
+  if (!dictionary->GetString("display_name", &display_name)) {
+    DLOG(ERROR) << "Response from Google Wallet missing display name";
+    return NULL;
+  }
+  std::string document_body;
+  if (!dictionary->GetString("document", &document_body)) {
+    DLOG(ERROR) << "Response from Google Wallet missing document body";
+    return NULL;
+  }
+  return new WalletItems::LegalDocument(document_id,
+                                        display_name,
+                                        document_body);
+}
+
+bool WalletItems::LegalDocument::operator==(const LegalDocument& other) const {
+  if (document_id_.compare(other.document_id_) != 0)
+    return false;
+  if (display_name_.compare(other.display_name_) != 0)
+    return false;
+  if (document_body_.compare(other.document_body_) != 0)
+    return false;
+  return true;
+}
+
+bool WalletItems::LegalDocument::operator!=(const LegalDocument& other) const {
+  return !(*this == other);
+}
+
+WalletItems::WalletItems(const std::vector<std::string> required_actions,

[Dan Beam, 2012/12/01 01:19:49]

nit: same question about passing this vector by value vs. const ref

[ahutter, 2012/12/01 04:06:51]

Done.

+                         const std::string& google_transaction_id,
+                         const std::string& default_instrument_id,
+                         const std::string& default_address_id)
+    : required_actions_(required_actions),
+      google_transaction_id_(google_transaction_id),
+      default_instrument_id_(default_instrument_id),
+      default_address_id_(default_address_id) {}
+
+WalletItems::~WalletItems() {}
+
+WalletItems* WalletItems::CreateWalletItems(
+    base::DictionaryValue* dictionary) {
+  DCHECK(dictionary);
+
+  std::vector<std::string> required_action;
+  ListValue* required_action_list;
+  if (dictionary->GetList("required_action", &required_action_list)) {
+    for (size_t i = 0; i < required_action_list->GetSize(); i++) {
+      std::string action;
+      if (required_action_list->GetString(i, &action))
+        required_action.push_back(action);
+    }
+    if (required_action.size() > 0) {

[Dan Beam, 2012/12/01 01:19:49]

nit: no curlies

[ahutter, 2012/12/01 04:06:51]

Done.

+      return new WalletItems(required_action, "", "", "");
+    }
+  } else {
+    VLOG(1) << "Response from Google wallet missing required actions";
+  }
+  std::string google_transaction_id;
+  if (!dictionary->GetString("google_transaction_id", &google_transaction_id)) {
+    DLOG(ERROR) << "Response from Google wallet missing google transaction id";
+    return NULL;
+  }
+  std::string default_instrument_id;
+  if (!dictionary->GetString("default_instrument_id", &default_instrument_id)) {
+    DLOG(ERROR) << "Response from Google wallet missing default instrument id";
+    return NULL;
+  }
+  std::string default_address_id;
+  if (!dictionary->GetString("default_address_id", &default_address_id))
+    VLOG(1) << "Response from Google wallet missing default_address_id";
+
+  WalletItems* wallet_items = new WalletItems(required_action,
+                                              google_transaction_id,
+                                              default_instrument_id,
+                                              default_address_id);
+
+  ListValue* instruments;
+  if (dictionary->GetList("instrument", &instruments)) {
+    for (size_t i = 0; i < instruments->GetSize(); i++) {
+      DictionaryValue* instrument_dict;
+      if (instruments->GetDictionary(i, &instrument_dict)) {
+        MaskedInstrument* instrument =
+            MaskedInstrument::CreateMaskedInstrument(instrument_dict);
+        if (instrument) {

[Dan Beam, 2012/12/01 01:19:49]

nit: no curlies

[ahutter, 2012/12/01 04:06:51]

Done.

+          wallet_items->AddInstrument(instrument);
+        }
+        // TODO(ahutter): Should I consider malformed input an error?
+      }
+    }
+  } else {
+    VLOG(1) << "Response from Google wallet missing instruments";
+  }
+  ListValue* addresses;
+  if (dictionary->GetList("address", &addresses)) {
+    for (size_t i = 0; i < addresses->GetSize(); i++) {
+      DictionaryValue* address_dict;
+      if (addresses->GetDictionary(i, &address_dict)) {
+        Address* address =
+            Address::CreateIdedAddress(address_dict);
+        if (address) {

[Dan Beam, 2012/12/01 01:19:49]

nit: no curlies

[ahutter, 2012/12/01 04:06:51]

Done.

+          wallet_items->AddAddress(address);
+        }
+        // TODO(ahutter): Should I consider malformed input an error?
+      }
+    }
+  } else {
+    VLOG(1) << "Response from Google wallet missing addresses";
+  }
+  ListValue* legal_docs;
+  if (dictionary->GetList("required_legal_document", &legal_docs)) {
+    for (size_t i = 0; i < legal_docs->GetSize(); i++) {
+      DictionaryValue* legal_doc_dict;
+      if (legal_docs->GetDictionary(i, &legal_doc_dict)) {
+        LegalDocument* legal_doc =
+            LegalDocument::CreateLegalDocument(legal_doc_dict);
+        if (legal_doc) {

[Dan Beam, 2012/12/01 01:19:49]

same nit

[ahutter, 2012/12/01 04:06:51]

Done.

+          wallet_items->AddLegalDocument(legal_doc);
+        }
+        // TODO(ahutter): Should I consider malformed input an error?
+      }
+    }
+  } else {
+    VLOG(1) << "Response from Google wallet missing legal docs";
+  }
+
+  return wallet_items;
+}
+
+bool WalletItems::operator==(const WalletItems& other) const {
+  if (google_transaction_id_.compare(other.google_transaction_id_) != 0)
+    return false;
+  if (default_instrument_id_.compare(other.default_instrument_id_))
+    return false;
+  if (default_address_id_.compare(other.default_address_id_))
+    return false;
+  if (required_actions_ != required_actions_)
+    return false;
+  // TODO(ahutter): how do i check the scoped vector equality...?
+  return true;
+}
+bool WalletItems::operator!=(const WalletItems& other) const {
+  return !(*this == other);
+}
+
+}  // end wallet namespace