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

Unified Diff: webrtc/modules/video_coding/generic_encoder.h

Issue 1853813002: Add support for writing raw encoder output to .ivf files. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Added checks Created 4 years, 8 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: webrtc/modules/video_coding/generic_encoder.h
diff --git a/webrtc/modules/video_coding/generic_encoder.h b/webrtc/modules/video_coding/generic_encoder.h
index e96d99578150f7b531445e14cb35f46995070801..431148f853d3600cd56dc55697b4213c0e26b74c 100644
--- a/webrtc/modules/video_coding/generic_encoder.h
+++ b/webrtc/modules/video_coding/generic_encoder.h
@@ -1,12 +1,12 @@
/*
- * Copyright (c) 2012 The WebRTC project authors. All Rights Reserved.
- *
- * Use of this source code is governed by a BSD-style license
- * that can be found in the LICENSE file in the root of the source
- * tree. An additional intellectual property rights grant can be found
- * in the file PATENTS. All contributing project authors may
- * be found in the AUTHORS file in the root of the source tree.
- */
+* Copyright (c) 2012 The WebRTC project authors. All Rights Reserved.
pbos-webrtc 2016/04/07 16:16:04 Comment formatting broken.
sprang_webrtc 2016/04/11 15:57:28 Done.
+*
+* Use of this source code is governed by a BSD-style license
+* that can be found in the LICENSE file in the root of the source
+* tree. An additional intellectual property rights grant can be found
+* in the file PATENTS. All contributing project authors may
+* be found in the AUTHORS file in the root of the source tree.
+*/
#ifndef WEBRTC_MODULES_VIDEO_CODING_GENERIC_ENCODER_H_
#define WEBRTC_MODULES_VIDEO_CODING_GENERIC_ENCODER_H_
@@ -33,60 +33,35 @@ struct EncoderParameters {
uint32_t input_frame_rate;
};
-/*************************************/
-/* VCMEncodeFrameCallback class */
-/***********************************/
class VCMEncodedFrameCallback : public EncodedImageCallback {
public:
- explicit VCMEncodedFrameCallback(
- EncodedImageCallback* post_encode_callback);
- virtual ~VCMEncodedFrameCallback();
-
- /*
- * Callback implementation - codec encode complete
- */
- int32_t Encoded(
- const EncodedImage& encodedImage,
- const CodecSpecificInfo* codecSpecificInfo = NULL,
- const RTPFragmentationHeader* fragmentationHeader = NULL);
- /*
- * Callback implementation - generic encoder encode complete
- */
- int32_t SetTransportCallback(VCMPacketizationCallback* transport);
- /**
- * Set media Optimization
- */
- void SetMediaOpt(media_optimization::MediaOptimization* mediaOpt);
-
- void SetPayloadType(uint8_t payloadType) {
- _payloadType = payloadType;
- }
-
- void SetInternalSource(bool internalSource) {
- _internalSource = internalSource;
- }
-
- void SetRotation(VideoRotation rotation) { _rotation = rotation; }
- void SignalLastEncoderImplementationUsed(
- const char* encoder_implementation_name);
+ explicit VCMEncodedFrameCallback(EncodedImageCallback* post_encode_callback);
+ virtual ~VCMEncodedFrameCallback();
+
+ // Implements EncodedImageCallback.
+ int32_t Encoded(const EncodedImage& encoded_image,
+ const CodecSpecificInfo* codec_specific,
+ const RTPFragmentationHeader* fragmentation_header) override;
+ int32_t SetTransportCallback(VCMPacketizationCallback* transport);
+ void SetMediaOpt(media_optimization::MediaOptimization* media_opt);
+ void SetPayloadType(uint8_t payload_type) { payload_type_ = payload_type; }
+ void SetInternalSource(bool internal_source) {
+ internal_source_ = internal_source;
+ }
+ void SetRotation(VideoRotation rotation) { rotation_ = rotation; }
+ void SignalLastEncoderImplementationUsed(
+ const char* encoder_implementation_name);
private:
- VCMPacketizationCallback* send_callback_;
- media_optimization::MediaOptimization* _mediaOpt;
- uint8_t _payloadType;
- bool _internalSource;
- VideoRotation _rotation;
-
- EncodedImageCallback* post_encode_callback_;
-
-#ifdef DEBUG_ENCODER_BIT_STREAM
- FILE* _bitStreamAfterEncoder;
-#endif
-}; // end of VCMEncodeFrameCallback class
-
-/******************************/
-/* VCMGenericEncoder class */
-/******************************/
+ VCMPacketizationCallback* send_callback_;
+ media_optimization::MediaOptimization* media_opt_;
+ uint8_t payload_type_;
+ bool internal_source_;
+ VideoRotation rotation_;
+
+ EncodedImageCallback* post_encode_callback_;
+}; // VCMEncodedFrameCallback
pbos-webrtc 2016/04/07 16:16:04 Remove comment
sprang_webrtc 2016/04/11 15:57:28 Done.
+
class VCMGenericEncoder {
friend class VCMCodecDataBase;
@@ -94,42 +69,24 @@ class VCMGenericEncoder {
VCMGenericEncoder(VideoEncoder* encoder,
VideoEncoderRateObserver* rate_observer,
VCMEncodedFrameCallback* encoded_frame_callback,
- bool internalSource);
+ bool internal_source);
~VCMGenericEncoder();
- /**
- * Free encoder memory
- */
- int32_t Release();
- /**
- * Initialize the encoder with the information from the VideoCodec
- */
+ int32_t Release(); // Free encoder memory.
pbos-webrtc 2016/04/07 16:16:04 Remove comment
sprang_webrtc 2016/04/11 15:57:28 Done.
int32_t InitEncode(const VideoCodec* settings,
- int32_t numberOfCores,
- size_t maxPayloadSize);
- /**
- * Encode raw image
- * inputFrame : Frame containing raw image
- * codecSpecificInfo : Specific codec data
- * cameraFrameRate : Request or information from the remote side
- * frameType : The requested frame type to encode
- */
- int32_t Encode(const VideoFrame& inputFrame,
- const CodecSpecificInfo* codecSpecificInfo,
- const std::vector<FrameType>& frameTypes);
+ int32_t number_of_cores,
+ size_t max_payload_size);
+ int32_t Encode(const VideoFrame& frame,
+ const CodecSpecificInfo* codec_specific,
+ const std::vector<FrameType>& frame_types);
void SetEncoderParameters(const EncoderParameters& params);
EncoderParameters GetEncoderParameters() const;
int32_t SetPeriodicKeyFrames(bool enable);
-
int32_t RequestFrame(const std::vector<FrameType>& frame_types);
-
bool InternalSource() const;
-
void OnDroppedFrame();
-
bool SupportsNativeHandle() const;
-
int GetTargetFramerate();
private:
@@ -141,7 +98,7 @@ class VCMGenericEncoder {
EncoderParameters encoder_params_ GUARDED_BY(params_lock_);
VideoRotation rotation_;
bool is_screenshare_;
-}; // end of VCMGenericEncoder class
+}; // VCMGenericEncoder
} // namespace webrtc

Powered by Google App Engine
This is Rietveld 408576698