Fix crash in UMA reporting of accelerated compositing in WebGL codepaths.

Fix crash in UMA reporting of accelerated compositing in WebGL codepaths.

BUG=129030
TEST=fast/canvas/webgl/ layouttests all pass

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=138160

[scherkus (not reviewing), 2012-05-21 19:56:28]

lgtm -- feel free to fix up as discussed offline if you feel it's necessary to
improve the accuracy of our counts

[commit-bot: I haz the power, 2012-05-21 19:58:13]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/fischman@chromium.org/10332275/1

[Peter Kasting, 2012-05-21 19:59:32]

https://chromiumcodereview.appspot.com/10332275/diff/1/webkit/media/webmediap...
File webkit/media/webmediaplayer_impl.cc (right):

https://chromiumcodereview.appspot.com/10332275/diff/1/webkit/media/webmediap...
webkit/media/webmediaplayer_impl.cc:576:
DCHECK_EQ(frame_->view()->isAcceleratedCompositingActive(), true);
Should this spot be changed too?

Also, nit: DCHECK(cond) rather than DCHECK_EQ(cond, true)

[Ami GONE FROM CHROMIUM, 2012-05-21 20:01:34]

https://chromiumcodereview.appspot.com/10332275/diff/1/webkit/media/webmediap...
File webkit/media/webmediaplayer_impl.cc (right):

https://chromiumcodereview.appspot.com/10332275/diff/1/webkit/media/webmediap...
webkit/media/webmediaplayer_impl.cc:576:
DCHECK_EQ(frame_->view()->isAcceleratedCompositingActive(), true);
On 2012/05/21 19:59:33, Peter Kasting wrote:
> Should this spot be changed too?

No; it shouldn't be possible to hit this codepath *without* accelerated
compositing being active.

> Also, nit: DCHECK(cond) rather than DCHECK_EQ(cond, true)

Hah.  Indeed.

[commit-bot: I haz the power, 2012-05-21 20:01:50]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/fischman@chromium.org/10332275/2002

[commit-bot: I haz the power, 2012-05-21 22:52:00]

Try job failure for 10332275-2002 (retry) on win_rel for step "sync_unit_tests".
It's a second try, previously, steps "base_unittests, sync_unit_tests" failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_rel&nu...