Differentiate between ADD and CHANGED when adding new autofill entries.

chrome/browser/webdata/web_database.cc

 // Copyright (c) 2009 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/web_database.h"
 
 #include <limits>
 
 #include "app/gfx/codec/png_codec.h"
 #include "app/l10n_util.h"
@@ skipping 786 matching lines @@
 
   while (s.Step()) {
     PasswordForm* new_form = new PasswordForm();
     InitPasswordFormFromStatement(new_form, &s);
 
     forms->push_back(new_form);
   }
   return s.Succeeded();
 }
 
-bool WebDatabase::AddFormFieldValues(const std::vector<FormField>& elements) {
-  return AddFormFieldValuesTime(elements, Time::Now());
+bool WebDatabase::AddFormFieldValues(const std::vector<FormField>& elements,
+                                     std::vector<AutofillChange>* changes) {
+  return AddFormFieldValuesTime(elements, changes, Time::Now());
 }
 
 bool WebDatabase::AddFormFieldValuesTime(const std::vector<FormField>& elements,
+                                         std::vector<AutofillChange>* changes,
                                          base::Time time) {
   bool result = true;
   for (std::vector<FormField>::const_iterator
        itr = elements.begin();
        itr != elements.end();
        itr++) {
-    result = result && AddFormFieldValueTime(*itr, time);
+    result = result && AddFormFieldValueTime(*itr, changes, time);
   }
   return result;
 }
 
 bool WebDatabase::ClearAutofillEmptyValueElements() {
   sql::Statement s(db_.GetUniqueStatement(
       "SELECT pair_id FROM autofill WHERE TRIM(value)= \"\""));
   if (!s) {
     NOTREACHED() << "Statement prepare failed";
     return false;
@@ skipping 109 matching lines @@
   s.BindInt(0, count);
   s.BindInt64(1, pair_id);
   if (!s.Run()) {
     NOTREACHED();
     return false;
   }
 
   return true;
 }
 
-bool WebDatabase::AddFormFieldValue(const FormField& element) {
-  return AddFormFieldValueTime(element, base::Time::Now());
+bool WebDatabase::AddFormFieldValue(const FormField& element,
+                                    std::vector<AutofillChange>* changes) {
+  return AddFormFieldValueTime(element, changes, base::Time::Now());
 }
 
 bool WebDatabase::AddFormFieldValueTime(const FormField& element,
+                                        std::vector<AutofillChange>* changes,
                                         base::Time time) {
   int count = 0;
   int64 pair_id;
 
   if (!GetIDAndCountOfFormElement(element, &pair_id, &count))
     return false;
 
   if (count == 0 && !InsertFormElement(element, &pair_id))
     return false;
 
-  return SetCountOfFormElement(pair_id, count + 1) &&
-      InsertPairIDAndDate(pair_id, time);
+  if (!SetCountOfFormElement(pair_id, count + 1))
+    return false;
+
+  if (!InsertPairIDAndDate(pair_id, time))
+    return false;
+
+  AutofillChange::Type change_type =
+      count == 0 ? AutofillChange::ADD : AutofillChange::UPDATE;
+  changes->push_back(
+      AutofillChange(change_type,
+                     AutofillKey(element.name(), element.value())));
+  return true;
 }
 
 bool WebDatabase::GetFormValuesForElementName(const string16& name,
                                               const string16& prefix,
                                               std::vector<string16>* values,
                                               int limit) {
   DCHECK(values);
   sql::Statement s;
 
   if (prefix.empty()) {
@@ skipping 204 matching lines @@
 
     // Add successive versions here.  Each should set the version number and
     // compatible version number as appropriate, then fall through to the next
     // case.
 
     case kCurrentVersionNumber:
       // No migration needed.
       return;
   }
 }