[Sync] Enable ProfileSyncServiceAutofillTest.ServerChangeRace

[Sync] Enable ProfileSyncServiceAutofillTest.ServerChangeRace

Test was disabled because number of calls to
PersonalDataManager::Refresh didn't match expectation. I think this
happened because calls to Refresh were posted to UI thread, but test
didn't run message loop at the end of the test.

BUG=57884
R=zea@chromium.org

Committed: https://crrev.com/14e9197c285a1edc95dc7e2f314f814fcc0a853f
Cr-Commit-Position: refs/heads/master@{#364439}

[Nicolas Zea, 2015-12-10 18:58:11]

lgtm

[pavely, 2015-12-10 19:07:11]

The CQ bit was checked by pavely@chromium.org

[commit-bot: I haz the power, 2015-12-10 19:08:01]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1511333003/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1511333003/1

[commit-bot: I haz the power, 2015-12-10 19:51:49]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-12-10 19:52:36]

Description was changed from

==========
[Sync] Enable ProfileSyncServiceAutofillTest.ServerChangeRace

Test was disabled because number of calls to
PersonalDataManager::Refresh didn't match expectation. I think this
happened because calls to Refresh were posted to UI thread, but test
didn't run message loop at the end of the test.

BUG=57884
R=zea@chromium.org
==========

to

==========
[Sync] Enable ProfileSyncServiceAutofillTest.ServerChangeRace

Test was disabled because number of calls to
PersonalDataManager::Refresh didn't match expectation. I think this
happened because calls to Refresh were posted to UI thread, but test
didn't run message loop at the end of the test.

BUG=57884
R=zea@chromium.org

Committed: https://crrev.com/14e9197c285a1edc95dc7e2f314f814fcc0a853f
Cr-Commit-Position: refs/heads/master@{#364439}
==========

[commit-bot: I haz the power, 2015-12-10 19:52:37]

Patchset 1 (id:??) landed as
https://crrev.com/14e9197c285a1edc95dc7e2f314f814fcc0a853f
Cr-Commit-Position: refs/heads/master@{#364439}

[Jeffrey Yasskin, 2015-12-14 21:17:16]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/1523083002/ by jyasskin@chromium.org.

The reason for reverting is: This is broken on the DrMemory bot:
http://build.chromium.org/p/chromium.memory.fyi/builders/Windows%20Unit%20%28...

[ RUN      ] ProfileSyncServiceAutofillTest.ServerChangeRace

GMOCK WARNING:
Uninteresting mock function call - returning directly.
    Function call: RegisterDataTypes(@05A1F340 8-byte object <C0-B7 D2-11 0E-BB
12-00>)
NOTE: You can safely ignore the above warning unless this call should not
happen.  Do not suppress it by blindly adding an EXPECT_CALL() if you don't mean
to enforce the call.  See
http://code.google.com/p/googlemock/wiki/CookBook#Knowing_When_to_Expect for
details.
[3560:2356:1210/173205:14452494:WARNING:sync_backend_registrar.cc(95)] No
history worker -- removing TYPED_URLS
[3560:2356:1210/173205:14452494:WARNING:sync_backend_registrar.cc(99)] No
password worker -- removing PASSWORDS
[3560:2356:1210/173205:14452494:WARNING:sync_encryption_handler_impl.cc(989)]
Nigori had empty encryption keybag.
[3560:2436:1210/173205:14452697:WARNING:sync_backend_registrar.cc(95)] No
history worker -- removing TYPED_URLS
[3560:2436:1210/173205:14452697:WARNING:sync_backend_registrar.cc(99)] No
password worker -- removing PASSWORDS
[3560:2436:1210/173205:14452697:WARNING:sync_backend_registrar.cc(95)] No
history worker -- removing TYPED_URLS
[3560:2436:1210/173205:14452697:WARNING:sync_backend_registrar.cc(99)] No
password worker -- removing PASSWORDS
c:\b\build\slave\drm-cr\build\src\chrome\browser\sync\profile_sync_service_autofill_unittest.cc(1318):
error: Value of: sync_entries.size()
  Actual: 2
Expected: 3U
Which is: 3
c:\b\build\slave\drm-cr\build\src\chrome\browser\sync\profile_sync_service_autofill_unittest.cc(1284):
error: Actual function call count doesn't match
EXPECT_CALL(*personal_data_manager_, Refresh())...
         Expected: to be called 3 times
           Actual: called twice - unsatisfied and active
[  FAILED  ] ProfileSyncServiceAutofillTest.ServerChangeRace (356 ms).