Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(39)

Issue 507053: Differentiate between ADD and CHANGED when adding new autofill entries. (Closed)

Created:
11 years ago by skrul
Modified:
9 years, 7 months ago
Reviewers:
James Hawkins
CC:
chromium-reviews_googlegroups.com, brettw+cc_chromium.org, not_the_right_glider, ben+cc_chromium.org, John Grabowski, Erik does not do reviews, jam, dank, Aaron Boodman, stuartmorgan, pam+watch_chromium.org, Paweł Hajdan Jr., darin (slow to review), native-client-reviews_googlegroups.com, amit, Timur Iskhodzhanov, DaveMoore
Visibility:
Public.

Description

Differentiate between ADD and CHANGED when adding new autofill entries. Adding a new autofill entry can either result in an ADD (when the name/value par was never seen before) or with an UPDATE (when the name/value pair already exists). Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=35108

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+172 lines, -65 lines) Patch
M chrome/browser/webdata/web_data_service.cc View 2 chunks +8 lines, -11 lines 0 comments Download
M chrome/browser/webdata/web_data_service_unittest.cc View 9 chunks +69 lines, -30 lines 0 comments Download
M chrome/browser/webdata/web_database.h View 2 chunks +14 lines, -6 lines 0 comments Download
M chrome/browser/webdata/web_database.cc View 3 chunks +21 lines, -7 lines 0 comments Download
M chrome/browser/webdata/web_database_unittest.cc View 6 chunks +60 lines, -11 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
skrul
11 years ago (2009-12-21 21:03:28 UTC) #1
James Hawkins
11 years ago (2009-12-21 21:16:17 UTC) #2
LGTM

Powered by Google App Engine
This is Rietveld 408576698