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

Side by Side Diff: content/common/gpu/media/v4l2_video_decode_accelerator.h

Issue 317083003: Merge VideoDecodeAcceleratorImpl with VideoDecodeAccelerator (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: add logging include Created 6 years, 6 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 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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 // This file contains an implementation of VideoDecodeAccelerator 5 // This file contains an implementation of VideoDecodeAccelerator
6 // that utilizes hardware video decoders, which expose Video4Linux 2 API 6 // that utilizes hardware video decoders, which expose Video4Linux 2 API
7 // (http://linuxtv.org/downloads/v4l-dvb-apis/). 7 // (http://linuxtv.org/downloads/v4l-dvb-apis/).
8 8
9 #ifndef CONTENT_COMMON_GPU_MEDIA_V4L2_VIDEO_DECODE_ACCELERATOR_H_ 9 #ifndef CONTENT_COMMON_GPU_MEDIA_V4L2_VIDEO_DECODE_ACCELERATOR_H_
10 #define CONTENT_COMMON_GPU_MEDIA_V4L2_VIDEO_DECODE_ACCELERATOR_H_ 10 #define CONTENT_COMMON_GPU_MEDIA_V4L2_VIDEO_DECODE_ACCELERATOR_H_
11 11
12 #include <queue> 12 #include <queue>
13 #include <vector> 13 #include <vector>
14 14
15 #include "base/callback_forward.h" 15 #include "base/callback_forward.h"
16 #include "base/memory/linked_ptr.h" 16 #include "base/memory/linked_ptr.h"
17 #include "base/memory/scoped_ptr.h" 17 #include "base/memory/scoped_ptr.h"
18 #include "base/synchronization/waitable_event.h" 18 #include "base/synchronization/waitable_event.h"
19 #include "base/threading/thread.h" 19 #include "base/threading/thread.h"
20 #include "content/common/content_export.h" 20 #include "content/common/content_export.h"
21 #include "content/common/gpu/media/v4l2_video_device.h" 21 #include "content/common/gpu/media/v4l2_video_device.h"
22 #include "content/common/gpu/media/video_decode_accelerator_impl.h"
23 #include "media/base/limits.h" 22 #include "media/base/limits.h"
24 #include "media/base/video_decoder_config.h" 23 #include "media/base/video_decoder_config.h"
25 #include "media/video/picture.h" 24 #include "media/video/picture.h"
26 #include "ui/gfx/size.h" 25 #include "ui/gfx/size.h"
27 #include "ui/gl/gl_bindings.h" 26 #include "ui/gl/gl_bindings.h"
28 27
29 namespace base { 28 namespace base {
30 class MessageLoopProxy; 29 class MessageLoopProxy;
31 } // namespace base 30 } // namespace base
32 31
(...skipping 33 matching lines...) Expand 10 before | Expand all | Expand 10 after
66 // Buffer creation is a two-step process that is serviced partially on the 65 // Buffer creation is a two-step process that is serviced partially on the
67 // Child thread, because we need to wait for the client to provide textures 66 // Child thread, because we need to wait for the client to provide textures
68 // for the buffers we allocate. We cannot keep the decoder thread running while 67 // for the buffers we allocate. We cannot keep the decoder thread running while
69 // the client allocates Pictures for us, because we need to REQBUFS first to get 68 // the client allocates Pictures for us, because we need to REQBUFS first to get
70 // the required number of output buffers from the device and that cannot be done 69 // the required number of output buffers from the device and that cannot be done
71 // unless we free the previous set of buffers, leaving the decoding in a 70 // unless we free the previous set of buffers, leaving the decoding in a
72 // inoperable state for the duration of the wait for Pictures. So to prevent 71 // inoperable state for the duration of the wait for Pictures. So to prevent
73 // subtle races (esp. if we get Reset() in the meantime), we block the decoder 72 // subtle races (esp. if we get Reset() in the meantime), we block the decoder
74 // thread while we wait for AssignPictureBuffers from the client. 73 // thread while we wait for AssignPictureBuffers from the client.
75 class CONTENT_EXPORT V4L2VideoDecodeAccelerator 74 class CONTENT_EXPORT V4L2VideoDecodeAccelerator
76 : public VideoDecodeAcceleratorImpl { 75 : public media::VideoDecodeAccelerator {
77 public: 76 public:
78 V4L2VideoDecodeAccelerator( 77 V4L2VideoDecodeAccelerator(
79 EGLDisplay egl_display, 78 EGLDisplay egl_display,
80 EGLContext egl_context, 79 EGLContext egl_context,
81 const base::WeakPtr<Client>& io_client_, 80 const base::WeakPtr<Client>& io_client_,
82 const base::Callback<bool(void)>& make_context_current, 81 const base::Callback<bool(void)>& make_context_current,
83 scoped_ptr<V4L2Device> device, 82 scoped_ptr<V4L2Device> device,
84 const scoped_refptr<base::MessageLoopProxy>& io_message_loop_proxy); 83 const scoped_refptr<base::MessageLoopProxy>& io_message_loop_proxy);
85 virtual ~V4L2VideoDecodeAccelerator(); 84 virtual ~V4L2VideoDecodeAccelerator();
86 85
87 // media::VideoDecodeAccelerator implementation. 86 // media::VideoDecodeAccelerator implementation.
88 // Note: Initialize() and Destroy() are synchronous. 87 // Note: Initialize() and Destroy() are synchronous.
89 virtual bool Initialize(media::VideoCodecProfile profile, 88 virtual bool Initialize(media::VideoCodecProfile profile,
90 Client* client) OVERRIDE; 89 Client* client) OVERRIDE;
91 virtual void Decode(const media::BitstreamBuffer& bitstream_buffer) OVERRIDE; 90 virtual void Decode(const media::BitstreamBuffer& bitstream_buffer) OVERRIDE;
92 virtual void AssignPictureBuffers( 91 virtual void AssignPictureBuffers(
93 const std::vector<media::PictureBuffer>& buffers) OVERRIDE; 92 const std::vector<media::PictureBuffer>& buffers) OVERRIDE;
94 virtual void ReusePictureBuffer(int32 picture_buffer_id) OVERRIDE; 93 virtual void ReusePictureBuffer(int32 picture_buffer_id) OVERRIDE;
95 virtual void Flush() OVERRIDE; 94 virtual void Flush() OVERRIDE;
96 virtual void Reset() OVERRIDE; 95 virtual void Reset() OVERRIDE;
97 virtual void Destroy() OVERRIDE; 96 virtual void Destroy() OVERRIDE;
98
99 // VideoDecodeAcceleratorImpl implementation.
100 virtual bool CanDecodeOnIOThread() OVERRIDE; 97 virtual bool CanDecodeOnIOThread() OVERRIDE;
101 98
102 private: 99 private:
103 // These are rather subjectively tuned. 100 // These are rather subjectively tuned.
104 enum { 101 enum {
105 kInputBufferCount = 8, 102 kInputBufferCount = 8,
106 // TODO(posciak): determine input buffer size based on level limits. 103 // TODO(posciak): determine input buffer size based on level limits.
107 // See http://crbug.com/255116. 104 // See http://crbug.com/255116.
108 // Input bitstream buffer size for up to 1080p streams. 105 // Input bitstream buffer size for up to 1080p streams.
109 kInputBufferMaxSizeFor1080p = 1024 * 1024, 106 kInputBufferMaxSizeFor1080p = 1024 * 1024,
(...skipping 329 matching lines...) Expand 10 before | Expand all | Expand 10 after
439 436
440 // The WeakPtrFactory for |weak_this_|. 437 // The WeakPtrFactory for |weak_this_|.
441 base::WeakPtrFactory<V4L2VideoDecodeAccelerator> weak_this_factory_; 438 base::WeakPtrFactory<V4L2VideoDecodeAccelerator> weak_this_factory_;
442 439
443 DISALLOW_COPY_AND_ASSIGN(V4L2VideoDecodeAccelerator); 440 DISALLOW_COPY_AND_ASSIGN(V4L2VideoDecodeAccelerator);
444 }; 441 };
445 442
446 } // namespace content 443 } // namespace content
447 444
448 #endif // CONTENT_COMMON_GPU_MEDIA_V4L2_VIDEO_DECODE_ACCELERATOR_H_ 445 #endif // CONTENT_COMMON_GPU_MEDIA_V4L2_VIDEO_DECODE_ACCELERATOR_H_
OLDNEW
« no previous file with comments | « content/common/gpu/media/gpu_video_decode_accelerator.h ('k') | content/common/gpu/media/vaapi_video_decode_accelerator.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698