Debug flaky gpu pixel tests by printing event trace on failure.

Debug flaky gpu pixel tests by printing event trace on failure.

R=zmo


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=126187

[Zhenyao Mo, 2012-03-10 01:12:46]

On 2012/03/10 00:18:11, jbates wrote:

LGTM

[Zhenyao Mo, 2012-03-10 01:12:51]

https://chromiumcodereview.appspot.com/9663033/diff/1/chrome/test/gpu/gpu_pix...
File chrome/test/gpu/gpu_pixel_browsertest.cc (right):

https://chromiumcodereview.appspot.com/9663033/diff/1/chrome/test/gpu/gpu_pix...
chrome/test/gpu/gpu_pixel_browsertest.cc:163: fprintf(stderr, "\n\nTRACE
JSON:\n\n%s\n\n", trace_events.c_str());
It's worth adding a comment that this is for temporary debugging purpose. 
Otherwise such kind of code will freak people out :)

[jbates, 2012-03-12 17:32:11]

https://chromiumcodereview.appspot.com/9663033/diff/1/chrome/test/gpu/gpu_pix...
File chrome/test/gpu/gpu_pixel_browsertest.cc (right):

https://chromiumcodereview.appspot.com/9663033/diff/1/chrome/test/gpu/gpu_pix...
chrome/test/gpu/gpu_pixel_browsertest.cc:163: fprintf(stderr, "\n\nTRACE
JSON:\n\n%s\n\n", trace_events.c_str());
On 2012/03/10 01:12:51, Zhenyao Mo wrote:
> It's worth adding a comment that this is for temporary debugging purpose. 
> Otherwise such kind of code will freak people out :)

Done.

[commit-bot: I haz the power, 2012-03-12 17:33:13]

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

[commit-bot: I haz the power, 2012-03-12 19:23:33]

Change committed as 126187