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

Unified Diff: remoting/base/decoder_vp8.h

Issue 9331003: Improving the decoder pipeline. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Integer ScaleRect Created 8 years, 10 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
« no previous file with comments | « remoting/base/decoder_row_based.cc ('k') | remoting/base/decoder_vp8.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: remoting/base/decoder_vp8.h
diff --git a/remoting/base/decoder_vp8.h b/remoting/base/decoder_vp8.h
index 0549f03183002acbefef700bf6c4035863415b82..c316ba88e5009c394aca40430a1a3900226bdbca 100644
--- a/remoting/base/decoder_vp8.h
+++ b/remoting/base/decoder_vp8.h
@@ -18,15 +18,17 @@ class DecoderVp8 : public Decoder {
virtual ~DecoderVp8();
// Decoder implementations.
- virtual void Initialize(scoped_refptr<media::VideoFrame> frame) OVERRIDE;
+ virtual void Initialize(const SkISize& screen_size) OVERRIDE;
virtual DecodeResult DecodePacket(const VideoPacket* packet) OVERRIDE;
- virtual void GetUpdatedRegion(SkRegion* region) OVERRIDE;
virtual bool IsReadyForData() OVERRIDE;
- virtual void Reset() OVERRIDE;
virtual VideoPacketFormat::Encoding Encoding() OVERRIDE;
- virtual void SetOutputSize(const SkISize& size) OVERRIDE;
- virtual void SetClipRect(const SkIRect& clip_rect) OVERRIDE;
- virtual void RefreshRegion(const SkRegion& region) OVERRIDE;
+ virtual void Invalidate(const SkISize& view_size,
+ const SkRegion& region) OVERRIDE;
+ virtual void RenderFrame(const SkISize& view_size,
+ const SkIRect& clip_area,
+ uint8* image_buffer,
+ int image_stride,
+ SkRegion* output_region) OVERRIDE;
private:
enum State {
@@ -38,22 +40,16 @@ class DecoderVp8 : public Decoder {
// The internal state of the decoder.
State state_;
- // The video frame to write to.
- scoped_refptr<media::VideoFrame> frame_;
-
vpx_codec_ctx_t* codec_;
// Pointer to the last decoded image.
vpx_image_t* last_image_;
- // The region updated by the most recent decode.
+ // The region updated that hasn't been copied to the screen yet.
SkRegion updated_region_;
- // Clipping rect for the output of the decoder.
- SkIRect clip_rect_;
-
// Output dimensions.
- SkISize output_size_;
+ SkISize screen_size_;
DISALLOW_COPY_AND_ASSIGN(DecoderVp8);
};
« no previous file with comments | « remoting/base/decoder_row_based.cc ('k') | remoting/base/decoder_vp8.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698