[Autofill] Sync Autofill profiles' origins.

chrome/browser/webdata/autofill_profile_syncable_service.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/webdata/autofill_profile_syncable_service.h"
 
 #include "base/guid.h"
 #include "base/location.h"
 #include "base/logging.h"
 #include "base/strings/utf_string_conversions.h"
@@ skipping 301 matching lines @@
   }
   return success;
 }
 
 // static
 bool AutofillProfileSyncableService::OverwriteProfileWithServerData(
     const sync_pb::AutofillProfileSpecifics& specifics,
     AutofillProfile* profile,
     const std::string& app_locale) {
   bool diff = false;
+  if (profile->origin() != specifics.origin()) {
+    bool was_verified = profile->IsVerified();
+    profile->set_origin(specifics.origin());
+    diff = true;
+
+    // Verified profiles should never be overwritten by unverified ones.
+    DCHECK(!was_verified || profile->IsVerified());
+  }
+
   diff = UpdateMultivaluedField(autofill::NAME_FIRST,
                                 specifics.name_first(), profile) || diff;
   diff = UpdateMultivaluedField(autofill::NAME_MIDDLE,
                                 specifics.name_middle(), profile) || diff;
   diff = UpdateMultivaluedField(autofill::NAME_LAST,
                                 specifics.name_last(), profile) || diff;
   diff = UpdateField(autofill::ADDRESS_HOME_LINE1,
                      specifics.address_home_line1(), profile) || diff;
   diff = UpdateField(autofill::ADDRESS_HOME_LINE2,
                      specifics.address_home_line2(), profile) || diff;
@@ skipping 29 matching lines @@
   DCHECK(base::IsValidGUID(profile.guid()));
 
   // Reset all multi-valued fields in the protobuf.
   specifics->clear_name_first();
   specifics->clear_name_middle();
   specifics->clear_name_last();
   specifics->clear_email_address();
   specifics->clear_phone_home_whole_number();
 
   specifics->set_guid(profile.guid());
+  specifics->set_origin(profile.origin());
+
   std::vector<string16> values;
   profile.GetRawMultiInfo(autofill::NAME_FIRST, &values);
   for (size_t i = 0; i < values.size(); ++i) {
     specifics->add_name_first(LimitData(UTF16ToUTF8(values[i])));
   }
 
   profile.GetRawMultiInfo(autofill::NAME_MIDDLE, &values);
   for (size_t i = 0; i < values.size(); ++i) {
     specifics->add_name_middle(LimitData(UTF16ToUTF8(values[i])));
   }
@@ skipping 57 matching lines @@
   GUIDToProfileMap::iterator it = profile_map->find(
         autofill_specifics.guid());
   if (it != profile_map->end()) {
     // Some profile that already present is synced.
     if (OverwriteProfileWithServerData(
             autofill_specifics, it->second, app_locale_)) {
       bundle->profiles_to_update.push_back(it->second);
     }
   } else {
     // New profile synced.
-    // TODO(isherman): Read the origin from |autofill_specifics|.
-    AutofillProfile* new_profile(
-        new AutofillProfile(autofill_specifics.guid(), std::string()));
+    AutofillProfile* new_profile = new AutofillProfile(
+        autofill_specifics.guid(), autofill_specifics.origin());
     OverwriteProfileWithServerData(
         autofill_specifics, new_profile, app_locale_);
 
     // Check if profile appears under a different guid.
+    // Unverified profiles should never overwrite verified ones.
     for (GUIDToProfileMap::iterator i = profile_map->begin();
          i != profile_map->end(); ++i) {

[Evan Stade, 2013/06/12 22:12:24]

nit: more readable if you create a local variable for |i->second|

[Ilya Sherman, 2013/06/13 04:19:32]

Done.

       if (i->second->Compare(*new_profile) == 0) {
+        // Ensure that a verified profile can never revert back to an unverified
+        // one.
+        if (i->second->IsVerified() && !new_profile->IsVerified()) {
+          new_profile->set_origin(i->second->origin());
+          bundle->profiles_to_sync_back.push_back(new_profile);
+        }
+
         bundle->profiles_to_delete.push_back(i->second->guid());
         DVLOG(2) << "[AUTOFILL SYNC]"
                  << "Found in sync db but with a different guid: "
                  << UTF16ToUTF8(i->second->GetRawInfo(autofill::NAME_FIRST))
                  << UTF16ToUTF8(i->second->GetRawInfo(autofill::NAME_LAST))
                  << "New guid " << new_profile->guid()
                  << ". Profile to be deleted " << i->second->guid();
         profile_map->erase(i);
         break;
-      } else if (!i->second->PrimaryValue().empty() &&
+      } else if (!i->second->IsVerified() &&
+                 !new_profile->IsVerified() &&
+                 !i->second->PrimaryValue().empty() &&
                  i->second->PrimaryValue() == new_profile->PrimaryValue()) {
         // Add it to candidates for merge - if there is no profile with this
         // guid we will merge them.
         bundle->candidates_to_merge.insert(std::make_pair(i->second->guid(),
                                                           new_profile));
       }
     }
     profiles_.push_back(new_profile);
     it = profile_map->insert(std::make_pair(new_profile->guid(),
                                             new_profile)).first;
@@ skipping 96 matching lines @@
   }
   if (changed)
     autofill_profile->SetRawMultiInfo(field_type, values);
   return changed;
 }
 
 bool AutofillProfileSyncableService::MergeProfile(
     const AutofillProfile& merge_from,
     AutofillProfile* merge_into,
     const std::string& app_locale) {
+  DCHECK(!merge_into->IsVerified() || merge_from.IsVerified());
+  merge_into->set_origin(merge_from.origin());
   merge_into->OverwriteWithOrAddTo(merge_from, app_locale);
-  return (merge_into->Compare(merge_from) != 0);
+  return (merge_into->Compare(merge_from) != 0 ||
+          merge_into->origin() != merge_from.origin());
 }
 
 AutofillTable* AutofillProfileSyncableService::GetAutofillTable() const {
   return AutofillTable::FromWebDatabase(webdata_backend_->GetDatabase());
 }
 
 void AutofillProfileSyncableService::InjectStartSyncFlare(
     const syncer::SyncableService::StartSyncFlare& flare) {
   flare_ = flare;
 }
 
 AutofillProfileSyncableService::DataBundle::DataBundle() {}
 
 AutofillProfileSyncableService::DataBundle::~DataBundle() {}