Improve autofill unit test

Improve autofill unit test
With chron's help, I was able to get rid of the whole TestModelAssocator thing and use the low level syncapi directly to create the server tagged autofill node before model association starts.  Because of this, I was able to revert to the original AutofillModelAssociator::AssociateModels() code that zork had and it now works fine in a test.

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

[tim (not reviewing), 2010-03-02 23:33:56]

http://codereview.chromium.org/661410/diff/1/4
File chrome/browser/sync/profile_sync_service.cc (right):

http://codereview.chromium.org/661410/diff/1/4#newcode269
chrome/browser/sync/profile_sync_service.cc:269: // TODO(timsteele): Is it ok to
do this after the notification?
Hm, it looks like the IfReady() part of the below method doesn't apply anymore?
I can't see anything conditional about it, and except for the unrecoverable
error case we always fire a notification now when the data type manager start
callback is run, so I think as long as we fire that as well it's fine to do this
after the notification here.