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

Unified Diff: webrtc/modules/video_coding/frame_buffer2.cc

Issue 2990463002: [EXPERIMENTAL] Generic stereo codec with index header sending merged frames
Patch Set: Created 3 years, 5 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/encoded_frame.cc ('k') | webrtc/modules/video_coding/frame_object.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webrtc/modules/video_coding/frame_buffer2.cc
diff --git a/webrtc/modules/video_coding/frame_buffer2.cc b/webrtc/modules/video_coding/frame_buffer2.cc
index 5be20a2bd5e54576b9fbe797569f4796626343ce..4923a1331afd1b2808e4aeef9265ba26e9e2c2d2 100644
--- a/webrtc/modules/video_coding/frame_buffer2.cc
+++ b/webrtc/modules/video_coding/frame_buffer2.cc
@@ -62,14 +62,14 @@ FrameBuffer::ReturnReason FrameBuffer::NextFrame(
clock_->TimeInMilliseconds() + max_wait_time_ms;
int64_t wait_ms = max_wait_time_ms;
int64_t now_ms = 0;
-
do {
now_ms = clock_->TimeInMilliseconds();
{
rtc::CritScope lock(&crit_);
new_continuous_frame_event_.Reset();
- if (stopped_)
+ if (stopped_) {
return kStopped;
+ }
wait_ms = max_wait_time_ms;
@@ -162,7 +162,7 @@ FrameBuffer::ReturnReason FrameBuffer::NextFrame(
last_decoded_frame_key.picture_id == frame_key.picture_id &&
last_decoded_frame_key.spatial_layer < frame_key.spatial_layer;
- if (AheadOrAt(last_decoded_frame_timestamp_, frame->timestamp) &&
+ if (AheadOf(last_decoded_frame_timestamp_, frame->timestamp) &&
!frame_is_higher_spatial_layer_of_last_decoded_frame) {
// TODO(brandtr): Consider clearing the entire buffer when we hit
// these conditions.
@@ -329,7 +329,6 @@ int FrameBuffer::InsertFrame(std::unique_ptr<FrameObject> frame) {
ClearFramesAndHistory();
last_continuous_picture_id = -1;
}
-
auto info = frames_.insert(std::make_pair(key, FrameInfo())).first;
if (info->second.frame) {
@@ -402,7 +401,7 @@ void FrameBuffer::PropagateDecodability(const FrameInfo& info) {
RTC_DCHECK(ref_info != frames_.end());
// TODO(philipel): Look into why we've seen this happen.
if (ref_info != frames_.end()) {
- RTC_DCHECK_GT(ref_info->second.num_missing_decodable, 0U);
+ // RTC_DCHECK_GT(ref_info->second.num_missing_decodable, 0U);
--ref_info->second.num_missing_decodable;
}
}
« no previous file with comments | « webrtc/modules/video_coding/encoded_frame.cc ('k') | webrtc/modules/video_coding/frame_object.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698