Replace RunLoop with WaitableEvent in ConditionalCacheDeletionHelperBrowserTest

Replace RunLoop with WaitableEvent in ConditionalCacheDeletionHelperBrowserTest

...to synchronize the UI and IO threads. WaitableEvent correctly handles
the situation where the waited IO thread tasks finishes first (i.e. Signal()
is called first, Wait() later) which was not the case in the previous
implementation with RunLoop().

BUG=651505
Committed: https://crrev.com/5263a0fd408925936e119912e6a7c318413861d9
Cr-Commit-Position: refs/heads/master@{#424136}

[msramek, 2016-10-10 11:38:53]

The CQ bit was checked by msramek@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-10-10 11:39:02]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-10 12:09:16]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-10-10 12:09:17]

Dry run: This issue passed the CQ dry run.

[msramek, 2016-10-10 12:49:06]

msramek@chromium.org changed reviewers:
+ vabr@chromium.org

[msramek, 2016-10-10 12:49:06]

Hi Václav,

Could you please have a quick look? This is hopefully relatively context-free. I
would send it to gab@ who reported it, but he's OOO.

Thanks,
Martin

[vabr (Chromium), 2016-10-10 13:00:31]

LGTM, nice!
Vaclav

[msramek, 2016-10-10 13:03:30]

The CQ bit was checked by msramek@chromium.org

[commit-bot: I haz the power, 2016-10-10 13:03:43]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-10 13:08:48]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-10-10 13:11:09]

Description was changed from

==========
Replace RunLoop with WaitableEvent in ConditionalCacheDeletionHelperBrowserTest

...to synchronize the UI and IO threads. WaitableEvent correctly handles
the situation where the waited IO thread tasks finishes first (i.e. Signal()
is called first, Wait() later) which was not the case in the previous
implementation with RunLoop().

BUG=651505
==========

to

==========
Replace RunLoop with WaitableEvent in ConditionalCacheDeletionHelperBrowserTest

...to synchronize the UI and IO threads. WaitableEvent correctly handles
the situation where the waited IO thread tasks finishes first (i.e. Signal()
is called first, Wait() later) which was not the case in the previous
implementation with RunLoop().

BUG=651505

Committed: https://crrev.com/5263a0fd408925936e119912e6a7c318413861d9
Cr-Commit-Position: refs/heads/master@{#424136}
==========

[commit-bot: I haz the power, 2016-10-10 13:11:10]

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