We forgot to call Associate in case of local changes.

We forgot to call Associate in case of local changes for autofill. 

When a local profile is created we create the corresponding sync node and we need to associate the sync node with profile. We forgot to make the call and that is fixed in this cl. 
BUG=68912, 69173
TEST= sync_integration_tests

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=71409

[lipalani1, 2011-01-13 20:27:21]

We forgot to put in a call to Associate(guid, id) when an item was created
locally. This fixes that.

[Raghu Simha, 2011-01-13 22:16:10]

Drive-by code review; no need to wait for an LGTM from me.

I suggest you add "TEST=sync_integration_tests" to your patch description to
indicate that it has been tested. Also, a more detailed description / title
would help, since right now, it isn't clear that this is about sync autofill.

http://codereview.chromium.org/6323001/diff/1/chrome/test/live_sync/two_clien...
File chrome/test/live_sync/two_client_live_autofill_sync_test.cc (right):

http://codereview.chromium.org/6323001/diff/1/chrome/test/live_sync/two_clien...
chrome/test/live_sync/two_client_live_autofill_sync_test.cc:98: //
TODO(lipalani): Enable after http://crbug.com/69173 is fixed.
Remove this comment.

http://codereview.chromium.org/6323001/diff/1/chrome/test/live_sync/two_clien...
chrome/test/live_sync/two_client_live_autofill_sync_test.cc:230: //
TODO(lipalani): Enable after http://crbug.com/69173 is fixed.
Remove this comment.

[tim (not reviewing), 2011-01-14 01:00:11]

LGTM