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

Side by Side Diff: content/renderer/pepper_platform_video_decoder_impl.h

Issue 7065010: Add initialization callback support for Video Decoder PPAPI. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix CR comments Created 9 years, 7 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef CONTENT_RENDERER_PEPPER_PLATFORM_VIDEO_DECODER_IMPL_H_ 5 #ifndef CONTENT_RENDERER_PEPPER_PLATFORM_VIDEO_DECODER_IMPL_H_
6 #define CONTENT_RENDERER_PEPPER_PLATFORM_VIDEO_DECODER_IMPL_H_ 6 #define CONTENT_RENDERER_PEPPER_PLATFORM_VIDEO_DECODER_IMPL_H_
7 7
8 #include <vector> 8 #include <vector>
9 9
10 #include "base/scoped_ptr.h" 10 #include "base/scoped_ptr.h"
(...skipping 27 matching lines...) Expand all
38 virtual bool Flush() OVERRIDE; 38 virtual bool Flush() OVERRIDE;
39 virtual bool Abort() OVERRIDE; 39 virtual bool Abort() OVERRIDE;
40 40
41 // VideoDecodeAccelerator::Client implementation. 41 // VideoDecodeAccelerator::Client implementation.
42 virtual void ProvidePictureBuffers( 42 virtual void ProvidePictureBuffers(
43 uint32 requested_num_of_buffers, 43 uint32 requested_num_of_buffers,
44 const gfx::Size& dimensions, 44 const gfx::Size& dimensions,
45 media::VideoDecodeAccelerator::MemoryType type) OVERRIDE; 45 media::VideoDecodeAccelerator::MemoryType type) OVERRIDE;
46 virtual void PictureReady(const media::Picture& picture) OVERRIDE; 46 virtual void PictureReady(const media::Picture& picture) OVERRIDE;
47 virtual void DismissPictureBuffer(int32 picture_buffer_id) OVERRIDE; 47 virtual void DismissPictureBuffer(int32 picture_buffer_id) OVERRIDE;
48 virtual void NotifyInitializeDone() OVERRIDE;
48 virtual void NotifyEndOfStream() OVERRIDE; 49 virtual void NotifyEndOfStream() OVERRIDE;
49 virtual void NotifyError( 50 virtual void NotifyError(
50 media::VideoDecodeAccelerator::Error error) OVERRIDE; 51 media::VideoDecodeAccelerator::Error error) OVERRIDE;
51 virtual void NotifyEndOfBitstreamBuffer(int32 bitstream_buffer_id) OVERRIDE; 52 virtual void NotifyEndOfBitstreamBuffer(int32 bitstream_buffer_id) OVERRIDE;
52 virtual void NotifyFlushDone() OVERRIDE; 53 virtual void NotifyFlushDone() OVERRIDE;
53 virtual void NotifyAbortDone() OVERRIDE; 54 virtual void NotifyAbortDone() OVERRIDE;
54 55
55 private: 56 private:
56 void InitializeDecoder(const std::vector<uint32>& configs); 57 void InitializeDecoder(const std::vector<uint32>& configs);
57 58
(...skipping 11 matching lines...) Expand all
69 70
70 // PlatformVideoDecoderImpl must extend RefCountedThreadSafe in order to post 71 // PlatformVideoDecoderImpl must extend RefCountedThreadSafe in order to post
71 // tasks on the IO loop. However, it is not actually ref counted: 72 // tasks on the IO loop. However, it is not actually ref counted:
72 // PPB_VideoDecode_Impl is the only thing that holds reference to 73 // PPB_VideoDecode_Impl is the only thing that holds reference to
73 // PlatformVideoDecoderImpl, so ref counting is unnecessary. 74 // PlatformVideoDecoderImpl, so ref counting is unnecessary.
74 // 75 //
75 // TODO(vrk): Not sure if this is the right thing to do. Talk with fischman. 76 // TODO(vrk): Not sure if this is the right thing to do. Talk with fischman.
76 DISABLE_RUNNABLE_METHOD_REFCOUNT(PlatformVideoDecoderImpl); 77 DISABLE_RUNNABLE_METHOD_REFCOUNT(PlatformVideoDecoderImpl);
77 78
78 #endif // CONTENT_RENDERER_PEPPER_PLATFORM_VIDEO_DECODER_IMPL_H_ 79 #endif // CONTENT_RENDERER_PEPPER_PLATFORM_VIDEO_DECODER_IMPL_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698