Use a ConditionVariable in place of WaitableEvent in BookmarkModelWorker.

This is to ensure future changes to the code don't introduce harmful races
that could arise in presence of compiler optimizations due to the lack of
a barrier permitting instruction re-ordering.  The theoretical race that
exists now isn't causing any real malfunction today, but is causing
ThreadSanitizer warnings.

BUG=25915
TEST=BookmarkModelWorkerTest

Review URL: http://codereview.chromium.org/488012

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@34558 0039d316-1c4b-4281-b951-d872f2087c98
2 files changed