Fix leaks introduced by WDS change

Fix leaks introduced by WDS change

Make sure tests give WDS time to shutdown its DB.

TBR=atwilson@chromium.org
(for Signin changes)

R=joi@chromium.org
BUG=222363
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=189388

[Jói, 2013-03-20 16:18:42]

This LGTM, with a nit about a comment, as long as my assumption below is
correct.

https://codereview.chromium.org/12847010/diff/1/components/autofill/browser/p...
File components/autofill/browser/personal_data_manager_unittest.cc (right):

https://codereview.chromium.org/12847010/diff/1/components/autofill/browser/p...
components/autofill/browser/personal_data_manager_unittest.cc:82:
base::WaitableEvent done(false, false);
Is the point of these 4 lines just to make sure to wait while the DB thread
finishes processing any tasks posted to it before this one?  Perhaps a one or
two line comment is in order to explain.

[Cait (Slow), 2013-03-20 16:49:02]

https://codereview.chromium.org/12847010/diff/1/components/autofill/browser/p...
File components/autofill/browser/personal_data_manager_unittest.cc (right):

https://codereview.chromium.org/12847010/diff/1/components/autofill/browser/p...
components/autofill/browser/personal_data_manager_unittest.cc:82:
base::WaitableEvent done(false, false);
On 2013/03/20 16:18:42, Jói wrote:
> Is the point of these 4 lines just to make sure to wait while the DB thread
> finishes processing any tasks posted to it before this one?  Perhaps a one or
> two line comment is in order to explain.

Done.

[Jói, 2013-03-20 16:50:38]

LGTM

[commit-bot: I haz the power, 2013-03-20 17:02:18]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/caitkp@chromium.org/12847010/6001

[commit-bot: I haz the power, 2013-03-20 20:32:08]

Change committed as 189388