Added support for YUV videos to the software compositor.

webkit/media/webvideoframe_impl.h

 // 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.
 
 #ifndef WEBKIT_MEDIA_WEBVIDEOFRAME_IMPL_H_
 #define WEBKIT_MEDIA_WEBVIDEOFRAME_IMPL_H_
 
 #include "base/compiler_specific.h"
 #include "media/base/video_frame.h"
 #include "third_party/WebKit/Source/WebKit/chromium/public/WebVideoFrame.h"
@@ skipping 15 matching lines @@
   virtual int stride(unsigned plane) const;
   virtual const void* data(unsigned plane) const;
   virtual unsigned textureId() const;
   virtual unsigned textureTarget() const;
 
  private:
   scoped_refptr<media::VideoFrame> video_frame_;
   DISALLOW_COPY_AND_ASSIGN(WebVideoFrameImpl);
 };
 
+// TODO(skaslev) This is temporarily inline pending the removal of
+// WebKit::WebVideoFrame and WebVideoFrameImpl which are currently unused.

[Ami GONE FROM CHROMIUM, 2012/10/29 16:57:25]

I'm not sure why you say they are unused.
They are used by WebVideoFrameProvider which is used by
WebMediaPlayerClientImpl.
Is your plan to GTFO WMPCI out of webkit and into chromium to avoid the need for
WVF?

[Ami GONE FROM CHROMIUM, 2012/10/29 17:10:17]

Clarified in IM that "currently unused" refers to WVFI's non-static methods: as
of this CL nobody uses them anymore and then will be deleted in a follow-up CL.

LGTM++

+inline media::VideoFrame* WebVideoFrameImpl::toVideoFrame(
+    WebKit::WebVideoFrame* web_video_frame) {
+  WebVideoFrameImpl* wrapped_frame =
+      static_cast<WebVideoFrameImpl*>(web_video_frame);
+  if (wrapped_frame)
+    return wrapped_frame->video_frame_.get();
+  return NULL;
+}
+
 }  // namespace webkit_media
 
 #endif  // WEBKIT_MEDIA_WEBVIDEOFRAME_IMPL_H_