[Autofill] Convert wallet addresses to local autofill profiles.

components/autofill/core/browser/personal_data_manager.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/personal_data_manager.h"
 
 #include <stddef.h>
 
 #include <algorithm>
 #include <list>
@@ skipping 356 matching lines @@
 
   switch (result->GetType()) {
     case AUTOFILL_PROFILES_RESULT:
       if (h == pending_profiles_query_) {
         ReceiveLoadedDbValues(h, result.get(), &pending_profiles_query_,
                               &web_profiles_);
         LogProfileCount();  // This only logs local profiles.
       } else {
         ReceiveLoadedDbValues(h, result.get(), &pending_server_profiles_query_,
                               &server_profiles_);
-
-        if (!server_profiles_.empty()) {
-          std::string account_id = signin_manager_->GetAuthenticatedAccountId();
-          base::string16 email =
-              base::UTF8ToUTF16(
-                  account_tracker_->GetAccountInfo(account_id).email);
-
-          // User may have signed out during the fulfillment of the web data
-          // request, in which case there is no point updating
-          // |server_profiles_| as it will be cleared.
-          if (!email.empty()) {
-            for (auto& profile : server_profiles_)
-              profile->SetRawInfo(EMAIL_ADDRESS, email);
-          }
-        }
       }
       break;
     case AUTOFILL_CREDITCARDS_RESULT:
       if (h == pending_creditcards_query_) {
         ReceiveLoadedDbValues(h, result.get(), &pending_creditcards_query_,
                               &local_credit_cards_);
         LogLocalCreditCardCount();
       } else {
         ReceiveLoadedDbValues(h, result.get(),
                               &pending_server_creditcards_query_,
@@ skipping 15 matching lines @@
   if (pending_profiles_query_ == 0 &&
       pending_creditcards_query_ == 0 &&
       pending_server_profiles_query_ == 0 &&
       pending_server_creditcards_query_ == 0) {
     is_data_loaded_ = true;
     NotifyPersonalDataChanged();
   }
 }
 
 void PersonalDataManager::AutofillMultipleChanged() {
+  has_synced_new_data = true;
   Refresh();
 }
 
 void PersonalDataManager::SyncStarted(syncer::ModelType model_type) {
   if (model_type == syncer::AUTOFILL_PROFILE &&
       is_autofill_profile_cleanup_pending_) {
     // This runs as a one-time fix, tracked in syncable prefs. If it has already
     // run, it is a NOP (other than checking the pref).
     ApplyProfileUseDatesFix();
 
@@ skipping 309 matching lines @@
   return GetProfiles(false);
 }
 
 std::vector<AutofillProfile*> PersonalDataManager::web_profiles() const {
   std::vector<AutofillProfile*> result;
   for (const auto& profile : web_profiles_)
     result.push_back(profile.get());
   return result;
 }
 
+std::vector<AutofillProfile*> PersonalDataManager::server_profiles() const {
+  std::vector<AutofillProfile*> result;
+  for (const auto& profile : server_profiles_)
+    result.push_back(profile.get());
+  return result;
+}
+
 std::vector<CreditCard*> PersonalDataManager::GetLocalCreditCards() const {
   std::vector<CreditCard*> result;
   for (const auto& card : local_credit_cards_)
     result.push_back(card.get());
   return result;
 }
 
 const std::vector<CreditCard*>& PersonalDataManager::GetCreditCards() const {
   credit_cards_.clear();
   for (const auto& card : local_credit_cards_)
     credit_cards_.push_back(card.get());
   if (pref_service_->GetBoolean(prefs::kAutofillWalletImportEnabled)) {
     for (const auto& card : server_credit_cards_)
       credit_cards_.push_back(card.get());
   }
   return credit_cards_;
 }
 
 bool PersonalDataManager::HasServerData() const {

[Mathieu, 2017/02/02 01:22:52]

This seems unused. Delete?

[sebsg, 2017/02/02 15:18:51]

Done.

   return !server_credit_cards_.empty() || !server_profiles_.empty();
 }
 
 void PersonalDataManager::Refresh() {
   LoadProfiles();
   LoadCreditCards();
 }
 
 const std::vector<AutofillProfile*> PersonalDataManager::GetProfilesToSuggest()
     const {
@@ skipping 448 matching lines @@
     database_->CancelRequest(*handle);
   }
   *handle = 0;
 }
 
 std::string PersonalDataManager::SaveImportedProfile(
     const AutofillProfile& imported_profile) {
   if (is_off_the_record_)
     return std::string();
 
-  // Don't save a web profile if the data in the profile is a subset of a
-  // server profile, but do record the fact that it was used.
-  for (const auto& profile : server_profiles_) {
-    if (imported_profile.IsSubsetOf(*profile, app_locale_)) {
-      RecordUseOf(*profile);
-      return profile->guid();
-    }
-  }
-
   std::vector<AutofillProfile> profiles;
   std::string guid =
       MergeProfile(imported_profile, &web_profiles_, app_locale_, &profiles);
   SetProfiles(&profiles);
   return guid;
 }
 
 void PersonalDataManager::NotifyPersonalDataChanged() {
   for (PersonalDataManagerObserver& observer : observers_)
     observer.OnPersonalDataChanged();
+
+  // If new data was synced, try to convert new server profiles.
+  if (has_synced_new_data) {
+    has_synced_new_data = false;
+    ConvertWalletAddressesToLocalProfiles();
+  }
 }
 
 std::string PersonalDataManager::SaveImportedCreditCard(
     const CreditCard& imported_card) {
   DCHECK(!imported_card.number().empty());
   if (is_off_the_record_)
     return std::string();
 
   // Set to true if |imported_card| is merged into the credit card list.
   bool merged = false;
@@ skipping 297 matching lines @@
 
   imported_credit_card->reset(new CreditCard(candidate_credit_card));
   return true;
 }
 
 const std::vector<AutofillProfile*>& PersonalDataManager::GetProfiles(
     bool record_metrics) const {
   profiles_.clear();
   for (const auto& profile : web_profiles_)
     profiles_.push_back(profile.get());
-  if (pref_service_->GetBoolean(prefs::kAutofillWalletImportEnabled)) {
-    for (const auto& profile : server_profiles_)
-      profiles_.push_back(profile.get());
-  }
   return profiles_;
 }
 
 std::vector<Suggestion> PersonalDataManager::GetSuggestionsForCards(
     const AutofillType& type,
     const base::string16& field_contents,
     const std::vector<CreditCard*>& cards_to_suggest) const {
   std::vector<Suggestion> suggestions;
   base::string16 field_contents_lower = base::i18n::ToLower(field_contents);
   for (const CreditCard* credit_card : cards_to_suggest) {
@@ skipping 257 matching lines @@
     // If the card was modified, apply the changes to the database.
     if (was_modified) {
       if (credit_card->record_type() == CreditCard::LOCAL_CARD)
         database_->UpdateCreditCard(*credit_card);
       else
         database_->UpdateServerCardMetadata(*credit_card);
     }
   }
 }
 
+void PersonalDataManager::ConvertWalletAddressesToLocalProfiles() {
+  // Put the local profiles into a vector<AutofillProfile>. Theses are the

[Mathieu, 2017/02/02 01:22:52]

Put -> Copy

[sebsg, 2017/02/02 15:18:51]

Done.

+  // existing profiles. Get them sorted in decreasing order of frecency, so the
+  // "best" profiles are checked first. Put the verified profiles last so the
+  // server addresses have a chance to merge into the non-verified local
+  // profiles.
+  std::vector<AutofillProfile> local_profiles;
+  for (auto* existing_profile : GetProfilesToSuggest()) {

[Mathieu, 2017/02/02 01:22:52]

auto -> AutofillProfile?

[sebsg, 2017/02/02 15:18:51]

Done.

+    local_profiles.push_back(*existing_profile);
+  }
+
+  // Create the map used to update credit card's billing addresses after the
+  // convertion/merge.
+  std::unordered_map<std::string, std::string> guids_merge_map;
+
+  bool has_converted_addresses = false;
+  for (auto& wallet_address : server_profiles_) {

[Mathieu, 2017/02/02 01:22:52]

avoid using auto if the type is straightforward

[sebsg, 2017/02/02 15:18:51]

Done.

+    // If the address has not been converted yet, convert it.
+    if (!wallet_address->has_converted()) {
+      // Try to merge the server address into a similar local profile, or create
+      // a new local profile if no similar profile is found.
+      std::string address_guid = MergeServerAddressesIntoLocalProfiles(

[Mathieu, 2017/02/02 01:22:52]

MergeServerAddressIntoProfiles?

[sebsg, 2017/02/02 15:18:51]

Done.

+          *wallet_address, &local_profiles);
+
+      // Update the map to transfer the billing address relationship from the
+      // server address to the converted/merged local profile.
+      guids_merge_map.insert(std::pair<std::string, std::string>(
+          wallet_address->server_id(), address_guid));
+
+      // Update the wallet addresses metadata to record the conversion.
+      wallet_address->set_has_converted(true);
+      database_->UpdateServerAddressMetadata(*wallet_address);
+
+      has_converted_addresses = true;
+    }
+  }
+
+  if (has_converted_addresses) {
+    // Save the local profiles to the DB.
+    SetProfiles(&local_profiles);
+
+    // Update the credit cards billing address relationship.
+    UpdateCardsBillingAddressReference(guids_merge_map);
+
+    // Force a reload of the profiles and cards.
+    Refresh();
+  }
+}
+
+std::string PersonalDataManager::MergeServerAddressesIntoLocalProfiles(
+    const AutofillProfile& converted_server_address,

[Mathieu, 2017/02/02 01:22:52]

It's not actually converted yet. |server_address|?

[sebsg, 2017/02/02 15:18:51]

Done.

+    std::vector<AutofillProfile>* local_profiles) {

[Mathieu, 2017/02/02 01:22:52]

I actually like |existing_profiles|

[sebsg, 2017/02/02 15:18:51]

Done.

+  // Set to true if |existing_profiles| already contains an equivalent profile.
+  bool matching_profile_found = false;
+  std::string guid = converted_server_address.guid();
+
+  // If there is already a local profile that is very similar, merge in any
+  // missing values. Only merge with the first match.
+  AutofillProfileComparator comparator(app_locale_);

[Mathieu, 2017/02/02 01:22:52]

It looks like you took this from
https://cs.chromium.org/chromium/src/components/autofill/core/browser/persona...

Any chance of reusing the code?

[sebsg, 2017/02/02 15:18:51]

Yeah I thought about it for a while but the original caller will need to be
modified a bit, so I prefer to do it in a different CL. I'll add a todo

+  for (auto& local_profile : *local_profiles) {
+    if (!matching_profile_found &&
+        comparator.AreMergeable(converted_server_address, local_profile) &&
+        local_profile.SaveAdditionalInfo(converted_server_address,
+                                         app_locale_)) {
+      matching_profile_found = true;
+      local_profile.set_modification_date(base::Time::Now());
+      guid = local_profile.guid();
+      AutofillMetrics::LogWalletAddressConversionType(
+          AutofillMetrics::CONVERTED_ADDRESS_MERGED);
+    }
+  }
+
+  // If the server address was not merged with a local profile, add it to the
+  // list.
+  if (!matching_profile_found) {
+    local_profiles->push_back(converted_server_address);
+    // Set the profile as being local.
+    local_profiles->back().set_record_type(AutofillProfile::LOCAL_PROFILE);
+    local_profiles->back().set_modification_date(base::Time::Now());

[Mathieu, 2017/02/02 01:22:52]

Are we advising people to use AutofillClock now?

[sebsg, 2017/02/02 15:18:51]

It just became available with the Revase (Patch2) so yeah! Thanks ;)

+    AutofillMetrics::LogWalletAddressConversionType(
+        AutofillMetrics::CONVERTED_ADDRESS_ADDED);
+  }
+
+  return guid;
+}
+
 }  // namespace autofill