Fix reporting of Media.AcceleratedCompositingActive when false.

webkit/media/webmediaplayer_impl.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "webkit/media/webmediaplayer_impl.h"
 
 #include <limits>
 #include <string>
 
 #include "base/bind.h"
@@ skipping 483 matching lines @@
                                const WebKit::WebRect& rect) {
   paint(canvas, rect, 0xFF);
 }
 
 void WebMediaPlayerImpl::paint(WebCanvas* canvas,
                                const WebRect& rect,
                                uint8_t alpha) {
   DCHECK_EQ(main_loop_, MessageLoop::current());
   DCHECK(proxy_);
 
+  if (!accelerated_compositing_reported_) {
+    accelerated_compositing_reported_ = true;
+    UMA_HISTOGRAM_BOOLEAN("Media.AcceleratedCompositingActive",
+                          frame_->view()->isAcceleratedCompositingActive());

[Ami GONE FROM CHROMIUM, 2012/05/19 01:25:32]

Could also just use false here (and true at the other reporting site) instead of
frame_->view()->isAcceleratedCompositingActive() but figured it was better this
way.  Could be convinced.

[scherkus (not reviewing), 2012/05/19 02:09:31]

report false + comment w/ DCHECK to be safe and catch breakage should things
change?

[Ami GONE FROM CHROMIUM, 2012/05/19 02:48:24]

Done.

+  }
+
   proxy_->Paint(canvas, rect, alpha);
 }
 
 bool WebMediaPlayerImpl::hasSingleSecurityOrigin() const {
   if (proxy_)
     return proxy_->HasSingleOrigin();
   return true;
 }
 
 WebMediaPlayer::MovieLoadType WebMediaPlayerImpl::movieLoadType() const {
@@ skipping 501 matching lines @@
   return audio_source_provider_;
 }
 
 void WebMediaPlayerImpl::IncrementExternallyAllocatedMemory() {
   DCHECK_EQ(main_loop_, MessageLoop::current());
   incremented_externally_allocated_memory_ = true;
   v8::V8::AdjustAmountOfExternalAllocatedMemory(kPlayerExtraMemory);
 }
 
 }  // namespace webkit_media