webrtc/modules/video_coding/include/mock/mock_video_codec_interface.h - Issue 2649133005: Add QP sum stats for received streams.

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

Unified Diff: webrtc/modules/video_coding/include/mock/mock_video_codec_interface.h

Issue 2649133005: Add QP sum stats for received streams. (Closed)

Patch Set: Fix lint warning. Created 3 years, 11 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

« no previous file with comments | « webrtc/modules/video_coding/generic_decoder.cc ('k') | webrtc/modules/video_coding/include/video_coding_defines.h » ('j') | webrtc/video/video_stream_decoder.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

Index: webrtc/modules/video_coding/include/mock/mock_video_codec_interface.h

diff --git a/webrtc/modules/video_coding/include/mock/mock_video_codec_interface.h b/webrtc/modules/video_coding/include/mock/mock_video_codec_interface.h

index 7e2076ccdc1cff805dcf087cfbdb805c6a8079f8..d1d7bffada79c6c063cda46134d7cc398d33bc85 100644

--- a/webrtc/modules/video_coding/include/mock/mock_video_codec_interface.h

+++ b/webrtc/modules/video_coding/include/mock/mock_video_codec_interface.h

@@ -57,6 +57,10 @@ class MockDecodedImageCallback : public DecodedImageCallback {

MOCK_METHOD2(Decoded,

int32_t(VideoFrame& decodedImage, // NOLINT

int64_t decode_time_ms));

+ MOCK_METHOD3(Decoded,

+ int32_t(VideoFrame& decodedImage, // NOLINT

+ rtc::Optional<int32_t> decode_time_ms,

+ rtc::Optional<uint8_t> qp));

hbos 2017/01/26 15:04:45 Is NOLINT necessary?

sakal 2017/01/26 15:46:52 Yes: Is this a non-const reference? If so, make co

MOCK_METHOD1(ReceivedDecodedReferenceFrame,

int32_t(const uint64_t pictureId));

MOCK_METHOD1(ReceivedDecodedFrame, int32_t(const uint64_t pictureId));