Replace unused output parameter in VCMReceiver::FrameForDecoding().

webrtc/modules/video_coding/receiver_unittest.cc

 /*  Copyright (c) 2013 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.
  */
 
 #include <string.h>
@@ skipping 55 matching lines @@
     if (!complete) {
       // Drop the second packet.
       VCMPacket packet;
       stream_generator_->PopPacket(&packet, 0);
     }
     clock_->AdvanceTimeMilliseconds(kDefaultFramePeriodMs);
     return ret;
   }
 
   bool DecodeNextFrame() {
-    int64_t render_time_ms = 0;
-    VCMEncodedFrame* frame =
-        receiver_.FrameForDecoding(0, &render_time_ms, false);
+    VCMEncodedFrame* frame = receiver_.FrameForDecoding(0, false);
     if (!frame)
       return false;
     receiver_.ReleaseFrame(frame);
     return true;
   }
 
   std::unique_ptr<SimulatedClock> clock_;
   VCMTiming timing_;
   NullEventFactory event_factory_;
   VCMReceiver receiver_;
@@ skipping 313 matching lines @@
 // Test whether VCMReceiver::FrameForDecoding handles parameter
 // |max_wait_time_ms| correctly:
 // 1. The function execution should never take more than |max_wait_time_ms|.
 // 2. If the function exit before now + |max_wait_time_ms|, a frame must be
 //    returned.
 TEST_F(VCMReceiverTimingTest, FrameForDecoding) {
   const size_t kNumFrames = 100;
   const int kFramePeriod = 40;
   int64_t arrive_timestamps[kNumFrames];
   int64_t render_timestamps[kNumFrames];
-  int64_t next_render_time;
 
   // Construct test samples.
   // render_timestamps are the timestamps stored in the Frame;
   // arrive_timestamps controls when the Frame packet got received.
   for (size_t i = 0; i < kNumFrames; i++) {
     // Preset frame rate to 25Hz.
     // But we add a reasonable deviation to arrive_timestamps to mimic Internet
     // fluctuation.
     arrive_timestamps[i] =
         (i + 1) * kFramePeriod + (i % 10) * ((i % 2) ? 1 : -1);
     render_timestamps[i] = (i + 1) * kFramePeriod;
   }
 
   clock_.SetFrames(arrive_timestamps, render_timestamps, kNumFrames);
 
   // Record how many frames we finally get out of the receiver.
   size_t num_frames_return = 0;
 
   const int64_t kMaxWaitTime = 30;
 
   // Ideally, we should get all frames that we input in InitializeFrames.
   // In the case that FrameForDecoding kills frames by error, we rely on the
   // build bot to kill the test.
   while (num_frames_return < kNumFrames) {
     int64_t start_time = clock_.TimeInMilliseconds();
-    VCMEncodedFrame* frame =
-        receiver_.FrameForDecoding(kMaxWaitTime, &next_render_time, false);
+    VCMEncodedFrame* frame = receiver_.FrameForDecoding(kMaxWaitTime, false);
     int64_t end_time = clock_.TimeInMilliseconds();
 
     // In any case the FrameForDecoding should not wait longer than
     // max_wait_time.
     // In the case that we did not get a frame, it should have been waiting for
     // exactly max_wait_time. (By the testing samples we constructed above, we
     // are sure there is no timing error, so the only case it returns with NULL
     // is that it runs out of time.)
     if (frame) {
       receiver_.ReleaseFrame(frame);
@@ skipping 10 matching lines @@
 // 1. The function execution should never take more than |max_wait_time_ms|.
 // 2. If the function exit before now + |max_wait_time_ms|, a frame must be
 //    returned and the end time must be equal to the render timestamp - delay
 //    for decoding and rendering.
 TEST_F(VCMReceiverTimingTest, FrameForDecodingPreferLateDecoding) {
   const size_t kNumFrames = 100;
   const int kFramePeriod = 40;
 
   int64_t arrive_timestamps[kNumFrames];
   int64_t render_timestamps[kNumFrames];
-  int64_t next_render_time;
 
   int render_delay_ms;
   int max_decode_ms;
   int dummy;
   timing_.GetTimings(&dummy, &max_decode_ms, &dummy, &dummy, &dummy, &dummy,
                      &render_delay_ms);
 
   // Construct test samples.
   // render_timestamps are the timestamps stored in the Frame;
   // arrive_timestamps controls when the Frame packet got received.
   for (size_t i = 0; i < kNumFrames; i++) {
     // Preset frame rate to 25Hz.
     // But we add a reasonable deviation to arrive_timestamps to mimic Internet
     // fluctuation.
     arrive_timestamps[i] =
         (i + 1) * kFramePeriod + (i % 10) * ((i % 2) ? 1 : -1);
     render_timestamps[i] = (i + 1) * kFramePeriod;
   }
 
   clock_.SetFrames(arrive_timestamps, render_timestamps, kNumFrames);
 
   // Record how many frames we finally get out of the receiver.
   size_t num_frames_return = 0;
   const int64_t kMaxWaitTime = 30;
   bool prefer_late_decoding = true;
   while (num_frames_return < kNumFrames) {
     int64_t start_time = clock_.TimeInMilliseconds();
 
-    VCMEncodedFrame* frame = receiver_.FrameForDecoding(
-        kMaxWaitTime, &next_render_time, prefer_late_decoding);
+    VCMEncodedFrame* frame =
+        receiver_.FrameForDecoding(kMaxWaitTime, prefer_late_decoding);
     int64_t end_time = clock_.TimeInMilliseconds();
     if (frame) {
       EXPECT_EQ(frame->RenderTimeMs() - max_decode_ms - render_delay_ms,
                 end_time);
       receiver_.ReleaseFrame(frame);
       ++num_frames_return;
     } else {
       EXPECT_EQ(kMaxWaitTime, end_time - start_time);
     }
   }
 }
 
 }  // namespace webrtc