Revert "[Payments] Add billing_address_id and has_converted to autofill_table"

components/autofill/core/browser/webdata/autofill_table.cc

 // Copyright 2013 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 "components/autofill/core/browser/webdata/autofill_table.h"
 
 #include <stdint.h>
 
 #include <algorithm>
 #include <cmath>
@@ skipping 453 matching lines @@
       return MigrateToVersion65AddServerMetadataTables();
     case 66:
       *update_compatible_version = false;
       return MigrateToVersion66AddCardBillingAddress();
     case 67:
       *update_compatible_version = false;
       return MigrateToVersion67AddMaskedCardBillingAddress();
     case 70:
       *update_compatible_version = false;
       return MigrateToVersion70AddSyncMetadata();
-    case 71:
-      *update_compatible_version = true;
-      return MigrateToVersion71AddHasConvertedAndBillingAddressIdMetadata();
   }
   return true;
 }
 
 bool AutofillTable::AddFormFieldValues(
     const std::vector<FormFieldData>& elements,
     std::vector<AutofillChange>* changes) {
   return AddFormFieldValuesTime(elements, changes, Time::Now());
 }
 
@@ skipping 719 matching lines @@
 
   return s.Succeeded();
 }
 
 bool AutofillTable::GetServerCreditCards(
     std::vector<std::unique_ptr<CreditCard>>* credit_cards) const {
   credit_cards->clear();
 
   sql::Statement s(db_->GetUniqueStatement(
       "SELECT "
-      "card_number_encrypted, "       // 0
-      "last_four,"                    // 1
-      "masked.id,"                    // 2
-      "metadata.use_count,"           // 3
-      "metadata.use_date,"            // 4
-      "type,"                         // 5
-      "status,"                       // 6
-      "name_on_card,"                 // 7
-      "exp_month,"                    // 8
-      "exp_year,"                     // 9
-      "metadata.billing_address_id "  // 10
+      "card_number_encrypted, "  // 0
+      "last_four,"     // 1
+      "masked.id,"     // 2
+      "metadata.use_count,"     // 3
+      "metadata.use_date,"      // 4
+      "type,"          // 5
+      "status,"        // 6
+      "name_on_card,"  // 7
+      "exp_month,"     // 8
+      "exp_year,"      // 9
+      "billing_address_id "     // 10
       "FROM masked_credit_cards masked "
       "LEFT OUTER JOIN unmasked_credit_cards USING (id) "
       "LEFT OUTER JOIN server_card_metadata metadata USING (id)"));
   while (s.Step()) {
     int index = 0;
 
     // If the card_number_encrypted field is nonempty, we can assume this card
     // is a full card, otherwise it's masked.
     base::string16 full_card_number = UnencryptedCardFromColumn(s, index++);
     base::string16 last_four = s.ColumnString16(index++);
@@ skipping 38 matching lines @@
     const std::vector<CreditCard>& credit_cards) {
   sql::Transaction transaction(db_);
   if (!transaction.Begin())
     return;
 
   // Delete all old values.
   sql::Statement masked_delete(db_->GetUniqueStatement(
       "DELETE FROM masked_credit_cards"));
   masked_delete.Run();
 
-  sql::Statement masked_insert(
-      db_->GetUniqueStatement("INSERT INTO masked_credit_cards("
-                              "id,"            // 0
-                              "type,"          // 1
-                              "status,"        // 2
-                              "name_on_card,"  // 3
-                              "last_four,"     // 4
-                              "exp_month,"     // 5
-                              "exp_year)"      // 6
-                              "VALUES (?,?,?,?,?,?,?)"));
+  sql::Statement masked_insert(db_->GetUniqueStatement(
+      "INSERT INTO masked_credit_cards("
+      "id,"            // 0
+      "type,"          // 1
+      "status,"        // 2
+      "name_on_card,"  // 3
+      "last_four,"     // 4
+      "exp_month,"     // 5
+      "exp_year,"      // 6
+      "billing_address_id) "  // 7
+      "VALUES (?,?,?,?,?,?,?,?)"));
   for (const CreditCard& card : credit_cards) {
     DCHECK_EQ(CreditCard::MASKED_SERVER_CARD, card.record_type());
 
     masked_insert.BindString(0, card.server_id());
     masked_insert.BindString(1, card.type());
     masked_insert.BindString(2,
                              ServerStatusEnumToString(card.GetServerStatus()));
     masked_insert.BindString16(3, card.GetRawInfo(CREDIT_CARD_NAME_FULL));
     masked_insert.BindString16(4, card.LastFourDigits());
     masked_insert.BindString16(5, card.GetRawInfo(CREDIT_CARD_EXP_MONTH));
     masked_insert.BindString16(6,
                                card.GetRawInfo(CREDIT_CARD_EXP_4_DIGIT_YEAR));
+    masked_insert.BindString(7, card.billing_address_id());
 
     masked_insert.Run();
     masked_insert.Reset(true);
 
     // Save the use count and use date of the card.
-    UpdateServerCardMetadata(card);
+    UpdateServerCardUsageStats(card);
   }
 
   // Delete all items in the unmasked table that aren't in the new set.
   sql::Statement unmasked_delete(db_->GetUniqueStatement(
       "DELETE FROM unmasked_credit_cards WHERE id NOT IN "
           "(SELECT id FROM masked_credit_cards)"));
   unmasked_delete.Run();
   // Do the same for metadata.
   sql::Statement metadata_delete(db_->GetUniqueStatement(
       "DELETE FROM server_card_metadata WHERE id NOT IN "
@@ skipping 20 matching lines @@
   s.BindBlob(1, encrypted_data.data(),
              static_cast<int>(encrypted_data.length()));
   s.BindInt64(2, Time::Now().ToInternalValue());  // unmask_date
 
   s.Run();
 
   CreditCard unmasked = masked;
   unmasked.set_record_type(CreditCard::FULL_SERVER_CARD);
   unmasked.SetNumber(full_number);
   unmasked.RecordAndLogUse();
-  UpdateServerCardMetadata(unmasked);
+  UpdateServerCardUsageStats(unmasked);
 
   return db_->GetLastChangeCount() > 0;
 }
 
 bool AutofillTable::MaskServerCreditCard(const std::string& id) {
   sql::Statement s(db_->GetUniqueStatement(
       "DELETE FROM unmasked_credit_cards WHERE id = ?"));
   s.BindString(0, id);
   s.Run();
   return db_->GetLastChangeCount() > 0;
 }
 
-bool AutofillTable::UpdateServerCardMetadata(const CreditCard& credit_card) {
+bool AutofillTable::UpdateServerCardUsageStats(
+    const CreditCard& credit_card) {
   DCHECK_NE(CreditCard::LOCAL_CARD, credit_card.record_type());
   sql::Transaction transaction(db_);
   if (!transaction.Begin())
     return false;
 
   sql::Statement remove(db_->GetUniqueStatement(
       "DELETE FROM server_card_metadata WHERE id = ?"));
   remove.BindString(0, credit_card.server_id());
   remove.Run();
 
-  sql::Statement s(
-      db_->GetUniqueStatement("INSERT INTO server_card_metadata(use_count, "
-                              "use_date, billing_address_id, id)"
-                              "VALUES (?,?,?,?)"));
+  sql::Statement s(db_->GetUniqueStatement(
+      "INSERT INTO server_card_metadata(use_count, use_date, id)"
+      "VALUES (?,?,?)"));
   s.BindInt64(0, credit_card.use_count());
   s.BindInt64(1, credit_card.use_date().ToInternalValue());
-  s.BindString(2, credit_card.billing_address_id());
-  s.BindString(3, credit_card.server_id());
+  s.BindString(2, credit_card.server_id());
   s.Run();
 
   transaction.Commit();
 
   return db_->GetLastChangeCount() > 0;
 }
 
-// TODO(crbug.com/680182): Record the address conversion status when a server
-// address gets converted.
-bool AutofillTable::UpdateServerAddressMetadata(
+bool AutofillTable::UpdateServerAddressUsageStats(
     const AutofillProfile& profile) {
   DCHECK_EQ(AutofillProfile::SERVER_PROFILE, profile.record_type());
 
   sql::Transaction transaction(db_);
   if (!transaction.Begin())
     return false;
 
   sql::Statement remove(db_->GetUniqueStatement(
       "DELETE FROM server_address_metadata WHERE id = ?"));
   remove.BindString(0, profile.server_id());
   remove.Run();
 
-  sql::Statement s(
-      db_->GetUniqueStatement("INSERT INTO server_address_metadata(use_count, "
-                              "use_date, has_converted, id)"
-                              "VALUES (?,?,?,?)"));
+  sql::Statement s(db_->GetUniqueStatement(
+      "INSERT INTO server_address_metadata(use_count, use_date, id)"
+      "VALUES (?,?,?)"));
   s.BindInt64(0, profile.use_count());
   s.BindInt64(1, profile.use_date().ToInternalValue());
-  s.BindBool(2, false);
-  s.BindString(3, profile.server_id());
+  s.BindString(2, profile.server_id());
   s.Run();
 
   transaction.Commit();
 
   return db_->GetLastChangeCount() > 0;
 }
 
+bool AutofillTable::UpdateServerCardBillingAddress(
+    const CreditCard& credit_card) {
+  DCHECK_NE(CreditCard::LOCAL_CARD, credit_card.record_type());
+
+  sql::Statement update(db_->GetUniqueStatement(
+      "UPDATE masked_credit_cards SET billing_address_id = ? "
+      "WHERE id = ?"));
+  update.BindString(0, credit_card.billing_address_id());
+  update.BindString(1, credit_card.server_id());
+  if (!update.Run())
+    return false;
+
+  return db_->GetLastChangeCount() > 0;
+}
+
 bool AutofillTable::ClearAllServerData() {
   sql::Transaction transaction(db_);
   if (!transaction.Begin())
     return false;  // Some error, nothing was changed.
 
   sql::Statement masked(db_->GetUniqueStatement(
       "DELETE FROM masked_credit_cards"));
   masked.Run();
   bool changed = db_->GetLastChangeCount() > 0;
 
@@ skipping 480 matching lines @@
 
 bool AutofillTable::InitMaskedCreditCardsTable() {
   if (!db_->DoesTableExist("masked_credit_cards")) {
     if (!db_->Execute("CREATE TABLE masked_credit_cards ("
                       "id VARCHAR,"
                       "status VARCHAR,"
                       "name_on_card VARCHAR,"
                       "type VARCHAR,"
                       "last_four VARCHAR,"
                       "exp_month INTEGER DEFAULT 0,"
-                      "exp_year INTEGER DEFAULT 0)")) {
+                      "exp_year INTEGER DEFAULT 0, "
+                      "billing_address_id VARCHAR)")) {
       NOTREACHED();
       return false;
     }
   }
   return true;
 }
 
 bool AutofillTable::InitUnmaskedCreditCardsTable() {
   if (!db_->DoesTableExist("unmasked_credit_cards")) {
     if (!db_->Execute("CREATE TABLE unmasked_credit_cards ("
                       "id VARCHAR,"
                       "card_number_encrypted VARCHAR, "
                       "use_count INTEGER NOT NULL DEFAULT 0, "
                       "use_date INTEGER NOT NULL DEFAULT 0, "
                       "unmask_date INTEGER NOT NULL DEFAULT 0)")) {
       NOTREACHED();
       return false;
     }
   }
   return true;
 }
 
 bool AutofillTable::InitServerCardMetadataTable() {
   if (!db_->DoesTableExist("server_card_metadata")) {
     if (!db_->Execute("CREATE TABLE server_card_metadata ("
                       "id VARCHAR NOT NULL,"
                       "use_count INTEGER NOT NULL DEFAULT 0, "
-                      "use_date INTEGER NOT NULL DEFAULT 0, "
-                      "billing_address_id VARCHAR)")) {
+                      "use_date INTEGER NOT NULL DEFAULT 0)")) {
       NOTREACHED();
       return false;
     }
   }
   return true;
 }
 
 bool AutofillTable::InitServerAddressesTable() {
   if (!db_->DoesTableExist("server_addresses")) {
     // The space after language_code is necessary to match what sqlite does
@@ skipping 17 matching lines @@
     }
   }
   return true;
 }
 
 bool AutofillTable::InitServerAddressMetadataTable() {
   if (!db_->DoesTableExist("server_address_metadata")) {
     if (!db_->Execute("CREATE TABLE server_address_metadata ("
                       "id VARCHAR NOT NULL,"
                       "use_count INTEGER NOT NULL DEFAULT 0, "
-                      "use_date INTEGER NOT NULL DEFAULT 0, "
-                      "has_converted BOOL NOT NULL DEFAULT FALSE)")) {
+                      "use_date INTEGER NOT NULL DEFAULT 0)")) {
       NOTREACHED();
       return false;
     }
   }
   return true;
 }
 
 bool AutofillTable::InitAutofillSyncMetadataTable() {
   if (!db_->DoesTableExist("autofill_sync_metadata")) {
     if (!db_->Execute("CREATE TABLE autofill_sync_metadata ("
@@ skipping 457 matching lines @@
   if (!db_->Execute("CREATE TABLE autofill_sync_metadata ("
                     "storage_key VARCHAR PRIMARY KEY NOT NULL,"
                     "value BLOB)")) {
     return false;
   }
   return db_->Execute(
       "CREATE TABLE autofill_model_type_state (id INTEGER PRIMARY KEY, value "
       "BLOB)");
 }
 
-bool AutofillTable::
-    MigrateToVersion71AddHasConvertedAndBillingAddressIdMetadata() {
-  sql::Transaction transaction(db_);
-  if (!transaction.Begin())
-    return false;
-
-  // Add the new has_converted column to the server_address_metadata table.
-  if (!db_->DoesColumnExist("server_address_metadata", "has_converted") &&
-      !db_->Execute("ALTER TABLE server_address_metadata ADD COLUMN "
-                    "has_converted BOOL NOT NULL DEFAULT FALSE")) {
-    return false;
-  }
-
-  // Add the new billing_address_id column to the server_card_metadata table.
-  if (!db_->DoesColumnExist("server_card_metadata", "billing_address_id") &&
-      !db_->Execute("ALTER TABLE server_card_metadata ADD COLUMN "
-                    "billing_address_id VARCHAR")) {
-    return false;
-  }
-
-  // Copy over the billing_address_id from the masked_server_cards to
-  // server_card_metadata.
-  if (!db_->Execute("UPDATE server_card_metadata "
-                    "SET billing_address_id = "
-                    "(SELECT billing_address_id "
-                    "FROM masked_credit_cards "
-                    "WHERE id = server_card_metadata.id)")) {
-    return false;
-  }
-
-  // Remove the billing_address_id column from the masked_credit_cards table.
-  // Create a temporary table that is a copy of masked_credit_cards but without
-  // the billing_address_id column.
-  if (db_->DoesTableExist("masked_credit_cards_temp") ||
-      !db_->Execute("CREATE TABLE masked_credit_cards_temp ("
-                    "id VARCHAR,"
-                    "status VARCHAR,"
-                    "name_on_card VARCHAR,"
-                    "type VARCHAR,"
-                    "last_four VARCHAR,"
-                    "exp_month INTEGER DEFAULT 0,"
-                    "exp_year INTEGER DEFAULT 0)")) {
-    return false;
-  }
-  // Copy over the data from the original masked_credit_cards table.
-  if (!db_->Execute("INSERT INTO masked_credit_cards_temp "
-                    "SELECT id, status, name_on_card, type, last_four, "
-                    "exp_month, exp_year "
-                    "FROM masked_credit_cards")) {
-    return false;
-  }
-  // Delete the existing table and replace it with the contents of the
-  // temporary table.
-  if (!db_->Execute("DROP TABLE masked_credit_cards") ||
-      !db_->Execute("ALTER TABLE masked_credit_cards_temp "
-                    "RENAME TO masked_credit_cards")) {
-    return false;
-  }
-
-  return transaction.Commit();
-}
-
 }  // namespace autofill