Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2398)

Unified Diff: cc/video_layer.h

Issue 11754003: Move VideoFrameProvider to, and remove all usage of WebVideoFrame from cc/ (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 12 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: cc/video_layer.h
diff --git a/cc/video_layer.h b/cc/video_layer.h
index 25250792540ee758ae54eac2b9d786fc06c4bf41..3624f772270e57263754a0a61a94eaf256d8c882 100644
--- a/cc/video_layer.h
+++ b/cc/video_layer.h
@@ -9,36 +9,28 @@
#include "cc/cc_export.h"
#include "cc/layer.h"
-namespace WebKit {
-class WebVideoFrame;
-class WebVideoFrameProvider;
-}
-
namespace media {
class VideoFrame;
}
namespace cc {
+class VideoFrameProvider;
class VideoLayerImpl;
// A Layer that contains a Video element.
class CC_EXPORT VideoLayer : public Layer {
public:
- typedef base::Callback<media::VideoFrame* (WebKit::WebVideoFrame*)> FrameUnwrapper;
-
- static scoped_refptr<VideoLayer> create(WebKit::WebVideoFrameProvider*,
- const FrameUnwrapper&);
+ static scoped_refptr<VideoLayer> create(VideoFrameProvider*);
virtual scoped_ptr<LayerImpl> createLayerImpl(LayerTreeImpl* treeImpl) OVERRIDE;
private:
- VideoLayer(WebKit::WebVideoFrameProvider*, const FrameUnwrapper&);
+ VideoLayer(VideoFrameProvider*);
jamesr 2013/01/03 23:35:55 explicit
danakj 2013/01/04 14:01:29 Done.
virtual ~VideoLayer();
// This pointer is only for passing to VideoLayerImpl's constructor. It should never be dereferenced by this class.
- WebKit::WebVideoFrameProvider* m_provider;
- FrameUnwrapper m_unwrapper;
+ VideoFrameProvider* m_provider;
};
} // namespace cc

Powered by Google App Engine
This is Rietveld 408576698