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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: webrtc/test/fake_encoder.cc

Issue 1369923005: Remove kSkipFrame usage. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Created 5 years, 2 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/main/source/encoded_frame.cc ('k') | webrtc/video_frame.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webrtc/test/fake_encoder.cc
diff --git a/webrtc/test/fake_encoder.cc b/webrtc/test/fake_encoder.cc
index a840ddb68ac75c458d25f48495fb2b2a70143ffa..61b731a7b8daa28727c9c620abb4da07381f6cfd 100644
--- a/webrtc/test/fake_encoder.cc
+++ b/webrtc/test/fake_encoder.cc
@@ -101,8 +101,7 @@ int32_t FakeEncoder::Encode(const VideoFrame& input_image,
encoded._encodedHeight = config_.simulcastStream[i].height;
// Always encode something on the first frame.
if (min_stream_bits > bits_available && i > 0) {
stefan-webrtc 2015/10/02 13:43:08 Optionally remove {}
- encoded._length = 0;
- encoded._frameType = kSkipFrame;
+ continue;
}
assert(callback_ != NULL);
if (callback_->Encoded(encoded, &specifics, NULL) != 0)
« no previous file with comments | « webrtc/modules/video_coding/main/source/encoded_frame.cc ('k') | webrtc/video_frame.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698