Remove requirement of compositing a video frame for dropped frame count.

content/renderer/media/video_frame_compositor.h

Index: content/renderer/media/video_frame_compositor.h
diff --git a/content/renderer/media/video_frame_compositor.h b/content/renderer/media/video_frame_compositor.h
index 47e6ad19fe69ef1e5c349219f7b811fb1943ed6e..237c6806fd88f9c85d25f70981f18a9a4f768535 100644
--- a/content/renderer/media/video_frame_compositor.h
+++ b/content/renderer/media/video_frame_compositor.h
@@ -52,15 +52,14 @@ class CONTENT_EXPORT VideoFrameCompositor {
   void UpdateCurrentFrame(const scoped_refptr<media::VideoFrame>& frame);
 
   // Retrieves the last frame set via UpdateCurrentFrame() for non-compositing
-  // purposes (e.g., painting to a canvas). Frames retrieved in this manner will
-  // not be counted as being composited for frame drop counting.
+  // purposes (e.g., painting to a canvas).
   //
   // Note that the compositor retrieves frames via the cc::VideoFrameProvider
   // interface instead of using this method.
   scoped_refptr<media::VideoFrame> GetCurrentFrame();
 
-  // Returns the number of frames dropped before the compositor was notified as
-  // well as being able to composite the previous frame.
+  // Returns the number of frames dropped before the compositor was notified
+  // of a new frame.
   uint32 GetFramesDroppedBeforeComposite();

[xhwang, 2014/03/27 21:28:59]

The name is not accurate now. "BeforeComposite" should really be
"BeforeCompositorIsNotified". Previously a frame is counted as dropped if it's
updated before the frame is composited. Now we don't care whether it's
composited or not.

That being said, I don't have a good naming suggestion yet... Probably just drop
the "BeforeComposite" part?

[scherkus (not reviewing), 2014/03/29 00:34:14]

let's go with the extremely long but accurate name

I feel it's OK for something like this where it's extremely important to
understand exactly what we're measuring

 
   void SetFramesDroppedBeforeCompositeForTesting(uint32 dropped_frames);