Add interface implementations for WebVideoFrameImpl to give the exact video Format, textureId and t…

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 "base/memory/ref_counted.h"
 #include "third_party/WebKit/Source/Platform/chromium/public/WebVideoFrame.h"
 
 namespace media {
 class VideoFrame;
 }
 
 namespace webkit_media {
 
 class WebVideoFrameImpl : public WebKit::WebVideoFrame {
  public:
   WebVideoFrameImpl(scoped_refptr<media::VideoFrame> video_frame);
   virtual ~WebVideoFrameImpl();
-
+  virtual WebKit::WebVideoFrame::Format format() const;

[danakj, 2013/03/04 15:57:50]

Why can't you use the video_frame directly? This class is more or less slated
for deletion at some point.

[Ken Russell (switch to Gerrit), 2013/03/05 00:48:39]

Is your suggestion to instead pass the WebVideoFrame* into a new method on
WebMediaPlayer to do the work?

+  virtual unsigned textureId() const;
+  virtual unsigned textureTarget() const;
   scoped_refptr<media::VideoFrame> video_frame;
 
  private:
   DISALLOW_COPY_AND_ASSIGN(WebVideoFrameImpl);
 };
 
 }  // namespace webkit_media
 
 #endif  // WEBKIT_MEDIA_WEBVIDEOFRAME_IMPL_H_