| Index: webrtc/modules/video_coding/codecs/vp8/simulcast_unittest.h
|
| diff --git a/webrtc/modules/video_coding/codecs/vp8/simulcast_unittest.h b/webrtc/modules/video_coding/codecs/vp8/simulcast_unittest.h
|
| index b277ad2ee485b73417761d7d95413add5459e787..e2bd71e546b739b31f804c6a88f4aa7fe4e6633e 100644
|
| --- a/webrtc/modules/video_coding/codecs/vp8/simulcast_unittest.h
|
| +++ b/webrtc/modules/video_coding/codecs/vp8/simulcast_unittest.h
|
| @@ -61,9 +61,9 @@ class Vp8TestEncodedImageCallback : public EncodedImageCallback {
|
| delete[] encoded_frame_._buffer;
|
| }
|
|
|
| - virtual int32_t Encoded(const EncodedImage& encoded_image,
|
| - const CodecSpecificInfo* codec_specific_info,
|
| - const RTPFragmentationHeader* fragmentation) {
|
| + virtual Result OnEncodedImage(const EncodedImage& encoded_image,
|
| + const CodecSpecificInfo* codec_specific_info,
|
| + const RTPFragmentationHeader* fragmentation) {
|
| // Only store the base layer.
|
| if (codec_specific_info->codecSpecific.VP8.simulcastIdx == 0) {
|
| if (encoded_image._frameType == kVideoFrameKey) {
|
| @@ -89,7 +89,7 @@ class Vp8TestEncodedImageCallback : public EncodedImageCallback {
|
| codec_specific_info->codecSpecific.VP8.layerSync;
|
| temporal_layer_[codec_specific_info->codecSpecific.VP8.simulcastIdx] =
|
| codec_specific_info->codecSpecific.VP8.temporalIdx;
|
| - return 0;
|
| + return Result(Result::OK, encoded_image._timeStamp);
|
| }
|
| void GetLastEncodedFrameInfo(int* picture_id,
|
| int* temporal_layer,
|
| @@ -338,34 +338,38 @@ class TestVp8Simulcast : public ::testing::Test {
|
| if (expected_video_streams >= 1) {
|
| EXPECT_CALL(
|
| encoder_callback_,
|
| - Encoded(
|
| + OnEncodedImage(
|
| AllOf(Field(&EncodedImage::_frameType, frame_type),
|
| Field(&EncodedImage::_encodedWidth, kDefaultWidth / 4),
|
| Field(&EncodedImage::_encodedHeight, kDefaultHeight / 4)),
|
| _, _))
|
| .Times(1)
|
| - .WillRepeatedly(Return(0));
|
| + .WillRepeatedly(Return(EncodedImageCallback::Result(
|
| + EncodedImageCallback::Result::OK, 0)));
|
| }
|
| if (expected_video_streams >= 2) {
|
| EXPECT_CALL(
|
| encoder_callback_,
|
| - Encoded(
|
| + OnEncodedImage(
|
| AllOf(Field(&EncodedImage::_frameType, frame_type),
|
| Field(&EncodedImage::_encodedWidth, kDefaultWidth / 2),
|
| Field(&EncodedImage::_encodedHeight, kDefaultHeight / 2)),
|
| _, _))
|
| .Times(1)
|
| - .WillRepeatedly(Return(0));
|
| + .WillRepeatedly(Return(EncodedImageCallback::Result(
|
| + EncodedImageCallback::Result::OK, 0)));
|
| }
|
| if (expected_video_streams >= 3) {
|
| EXPECT_CALL(
|
| encoder_callback_,
|
| - Encoded(AllOf(Field(&EncodedImage::_frameType, frame_type),
|
| - Field(&EncodedImage::_encodedWidth, kDefaultWidth),
|
| - Field(&EncodedImage::_encodedHeight, kDefaultHeight)),
|
| - _, _))
|
| + OnEncodedImage(
|
| + AllOf(Field(&EncodedImage::_frameType, frame_type),
|
| + Field(&EncodedImage::_encodedWidth, kDefaultWidth),
|
| + Field(&EncodedImage::_encodedHeight, kDefaultHeight)),
|
| + _, _))
|
| .Times(1)
|
| - .WillRepeatedly(Return(0));
|
| + .WillRepeatedly(Return(EncodedImageCallback::Result(
|
| + EncodedImageCallback::Result::OK, 0)));
|
| }
|
| }
|
|
|
| @@ -590,13 +594,15 @@ class TestVp8Simulcast : public ::testing::Test {
|
| encoder_->SetRates(kMaxBitrates[0] + kMaxBitrates[1], 30);
|
| std::vector<FrameType> frame_types(kNumberOfSimulcastStreams,
|
| kVideoFrameDelta);
|
| - EXPECT_CALL(encoder_callback_,
|
| - Encoded(AllOf(Field(&EncodedImage::_frameType, kVideoFrameKey),
|
| - Field(&EncodedImage::_encodedWidth, width),
|
| - Field(&EncodedImage::_encodedHeight, height)),
|
| - _, _))
|
| + EXPECT_CALL(
|
| + encoder_callback_,
|
| + OnEncodedImage(AllOf(Field(&EncodedImage::_frameType, kVideoFrameKey),
|
| + Field(&EncodedImage::_encodedWidth, width),
|
| + Field(&EncodedImage::_encodedHeight, height)),
|
| + _, _))
|
| .Times(1)
|
| - .WillRepeatedly(Return(0));
|
| + .WillRepeatedly(Return(
|
| + EncodedImageCallback::Result(EncodedImageCallback::Result::OK, 0)));
|
| EXPECT_EQ(0, encoder_->Encode(input_frame_, NULL, &frame_types));
|
|
|
| // Switch back.
|
|
|