Fix TraceEventTestFixture flakiness

Fix TraceEventTestFixture flakiness
BUG=142804
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=151851

[jar (doing other things), 2012-08-16 01:23:35]

LGTM

https://chromiumcodereview.appspot.com/10828326/diff/5001/base/debug/trace_ev...
File base/debug/trace_event_unittest.cc (right):

https://chromiumcodereview.appspot.com/10828326/diff/5001/base/debug/trace_ev...
base/debug/trace_event_unittest.cc:642:
base::PlatformThread::Sleep(base::TimeDelta::FromMilliseconds(1));
As per our discussion... it is very alarming that ::Sleep() can return a full
10ms too soon... and you had to try to play this game....

It will be very interesting to see if this really cures the flakyness that would
otherwise appear around line 856 of this (modified) file.

[jbates, 2012-08-16 01:26:18]

On 2012/08/16 01:23:35, jar wrote:
> LGTM
> 
>
https://chromiumcodereview.appspot.com/10828326/diff/5001/base/debug/trace_ev...
> File base/debug/trace_event_unittest.cc (right):
> 
>
https://chromiumcodereview.appspot.com/10828326/diff/5001/base/debug/trace_ev...
> base/debug/trace_event_unittest.cc:642:
> base::PlatformThread::Sleep(base::TimeDelta::FromMilliseconds(1));
> As per our discussion... it is very alarming that ::Sleep() can return a full
> 10ms too soon... and you had to try to play this game....
>
> It will be very interesting to see if this really cures the flakyness that
would
> otherwise appear around line 856 of this (modified) file.

Yeah, I'll keep an eye on it.

[commit-bot: I haz the power, 2012-08-16 01:27:21]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/jbates@chromium.org/10828326/5001

[commit-bot: I haz the power, 2012-08-16 04:32:51]

Change committed as 151851