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

Unified Diff: content/renderer/pepper/video_encoder_shim.h

Issue 956893002: content: pepper: VideoEncoder: add software encoder support (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Update after bbudge's review Created 5 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
Index: content/renderer/pepper/video_encoder_shim.h
diff --git a/content/renderer/pepper/video_encoder_shim.h b/content/renderer/pepper/video_encoder_shim.h
new file mode 100644
index 0000000000000000000000000000000000000000..2a6ffff682a1afb1443ffa5b1132c642ba4da923
--- /dev/null
+++ b/content/renderer/pepper/video_encoder_shim.h
@@ -0,0 +1,77 @@
+// Copyright 2015 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 CONTENT_RENDERER_PEPPER_VIDEO_ENCODER_SHIM_H_
+#define CONTENT_RENDERER_PEPPER_VIDEO_ENCODER_SHIM_H_
+
+#include <vector>
+
+#include "base/basictypes.h"
+#include "base/memory/scoped_ptr.h"
+#include "base/memory/weak_ptr.h"
+#include "media/video/video_encode_accelerator.h"
+
+namespace base {
+class SingleThreadTaskRunner;
+}
+
+namespace gfx {
+class Size;
+}
+
+namespace content {
+
+class PepperVideoEncoderHost;
+
+// This class is a shim to wrap a media::cast::SoftwareVideoEncoder so that it
+// can be used by PepperVideoEncoderHost in place of a
+// media::VideoEncodeAccelerator. This class should be constructed, used, and
+// destructed on the main (render) thread.
+class VideoEncoderShim : public media::VideoEncodeAccelerator {
+ public:
+ explicit VideoEncoderShim(PepperVideoEncoderHost* host);
+ ~VideoEncoderShim() override;
+
+ // media::VideoEncodeAccelerator implementation.
+ std::vector<media::VideoEncodeAccelerator::SupportedProfile>
+ GetSupportedProfiles() override;
bbudge 2015/03/02 20:01:07 nit: indent, or is this how the formatter handles
+ bool Initialize(media::VideoFrame::Format input_format,
+ const gfx::Size& input_visible_size,
+ media::VideoCodecProfile output_profile,
+ uint32 initial_bitrate,
+ media::VideoEncodeAccelerator::Client* client) override;
+ void Encode(const scoped_refptr<media::VideoFrame>& frame,
+ bool force_keyframe) override;
+ void UseOutputBitstreamBuffer(const media::BitstreamBuffer& buffer) override;
+ void RequestEncodingParametersChange(uint32 bitrate,
+ uint32 framerate) override;
+ void Destroy() override;
+
+ private:
+ class EncoderImpl;
+
+ void OnRequireBitstreamBuffers(unsigned int input_count,
+ const gfx::Size& input_coded_size,
+ size_t output_buffer_size);
+ void OnBitstreamBufferReady(scoped_refptr<media::VideoFrame> frame,
+ int32 bitstream_buffer_id,
+ size_t payload_size,
+ bool key_frame);
+ void OnNotifyError(media::VideoEncodeAccelerator::Error error);
+
+ scoped_ptr<EncoderImpl> encoder_impl_;
+
+ PepperVideoEncoderHost* host_;
+
+ // Task doing the encoding.
bbudge 2015/03/02 20:01:06 nit: s/Task/TaskRunner or Task runner
+ scoped_refptr<base::SingleThreadTaskRunner> media_task_runner_;
+
+ base::WeakPtrFactory<VideoEncoderShim> weak_ptr_factory_;
+
+ DISALLOW_COPY_AND_ASSIGN(VideoEncoderShim);
+};
+
+} // namespace content
+
+#endif // CONTENT_RENDERER_PEPPER_VIDEO_ENCODER_SHIM_H_

Powered by Google App Engine
This is Rietveld 408576698