New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.
Starred by 2 users

Issue metadata

Status: WontFix
Last visit > 30 days ago
Closed: Sep 2010
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 2
Type: Bug

  • Only users with EditIssue permission may comment.

Sign in to add a comment

Issue 51956: Sync: Autofill entries get overwritten between two clients

Reported by, Aug 12 2010 Project Member

Issue description

Google Chrome	6.0.490.0 (Official Build 55537) dev
WebKit	534.6
User Agent	Mozilla/5.0 (X11; U; Linux x86_64; en-US) AppleWebKit/534.6 (KHTML, like Gecko) Chrome/6.0.490.0 Safari/534.6
Command Line	 /opt/google/chrome/google-chrome --user-data-dir=./datadirs/6.0.490.0_dev_A5 --enable-log --log-level=0

0. Two Linux clients A and B, both online and signed into the same sync account;
1. Client A: In Wrench -> Options -> Personal Stuff -> [AutoFill options] add autofill entry name 1A;
2. Client B: In Wrench -> Options -> Personal Stuff -> [AutoFill options] add autofill entry name 1B.

After syncing both clients contain only one entry 1B.

Both 1A and 1B should be available between both clients.

This only happens between two Linux clients. 
No repro between a Mac-Linux.

Comment 1 by, Aug 12 2010

Labels: Feature-Sync

Comment 2 by, Aug 12 2010

Labels: -Area-Undefined -OS-Linux Area-UI OS-All
Status: Available
Summary: Sync: Autofill entries get overwritten between two clients
Update: I'm seeing this in all OSs when I run the sync integration tests. The test case that fails is TwoClientLiveAutofillSyncTest.Client1HasData.

Comment 3 by, Aug 12 2010

Labels: Tests-Fails

Comment 4 by, Aug 12 2010

The following revision refers to this bug: 

r55948 | | 2010-08-12 14:20:34 -0700 (Thu, 12 Aug 2010) | 11 lines
Changed paths:

Marking Client1HasData test as failed due to  bug 51956 .

The TwoClientLiveAutofillSyncTest.Client1HasData test fails due to bug
51956. Marking it as FAILS until the bug is fixed.

BUG= 51956 

Review URL:

Comment 5 by, Aug 17 2010

Labels: Mstone-7

Comment 6 by, Aug 18 2010

The following revision refers to this bug: 

r56469 | | 2010-08-17 17:45:26 -0700 (Tue, 17 Aug 2010) | 15 lines
Changed paths:

Fix failing sync integration tests.

TwoClientLivePreferencesSyncTest.Security was failing due to a couple of
EXPECT_NEs that ought to have been EXPECT_EQs.

TwoClientLiveAutofillSyncTest.Client1HasData was failing due to a change
in autofill behavior introduced in r55781 to fix security bug .

This changelist also includes a clarification in the logic used by

BUG= 51727 , 51956 

Review URL:

Comment 7 by, Aug 18 2010

@rsimha are you planning to merge these fixes to 472?  I expect tests will be failing there if not.

Comment 8 by, Aug 26 2010

Labels: MrBotRecommends
This an automated note...
Recommendations based on labels Feature-Sync and os-all from activity in the last 60 days

Comment 9 by, Aug 26 2010

Status: Assigned

Comment 10 by, Aug 27 2010

Status: Available
My checkin (in the bug comments) did not fix the bug -- it was meant to accommodate a change in design in how autofill manages duplicate entries.

This bug still needs to be investigated by a sync developer.

Comment 11 by, Aug 27 2010

Status: Assigned

Comment 12 by, Sep 29 2010

Status: WontFix
Anna, Dudley says to talk to him about expected behavior.  Also, Tim is reworking this for M8.

Comment 13 by, Oct 12 2012

Project Member
Labels: Restrict-AddIssueComment-Commit
This issue has been closed for some time. No one will pay attention to new comments.
If you are seeing this bug or have new data, please click New Issue to start a new bug.

Comment 14 by, Mar 10 2013

Project Member
Labels: -Area-UI -Feature-Sync -Tests-Fails -Mstone-7 M-7 Cr-Services-Sync Cr-UI Cr-Tests-Fails

Comment 15 by, Mar 13 2013

Project Member
Labels: -Restrict-AddIssueComment-Commit Restrict-AddIssueComment-EditIssue

Sign in to add a comment