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

Side by Side Diff: media/cast/video_sender/external_video_encoder_unittest.cc

Issue 252923007: Cast: Fix two video freezing problems (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: hide max outstanding frames from cast library users Created 6 years, 7 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 unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include <vector> 5 #include <vector>
6 6
7 #include "base/bind.h" 7 #include "base/bind.h"
8 #include "base/memory/ref_counted.h" 8 #include "base/memory/ref_counted.h"
9 #include "base/memory/scoped_ptr.h" 9 #include "base/memory/scoped_ptr.h"
10 #include "media/base/video_frame.h" 10 #include "media/base/video_frame.h"
(...skipping 66 matching lines...) Expand 10 before | Expand all | Expand 10 after
77 base::TimeTicks expected_capture_time_; 77 base::TimeTicks expected_capture_time_;
78 78
79 DISALLOW_COPY_AND_ASSIGN(TestVideoEncoderCallback); 79 DISALLOW_COPY_AND_ASSIGN(TestVideoEncoderCallback);
80 }; 80 };
81 } // namespace 81 } // namespace
82 82
83 class ExternalVideoEncoderTest : public ::testing::Test { 83 class ExternalVideoEncoderTest : public ::testing::Test {
84 protected: 84 protected:
85 ExternalVideoEncoderTest() 85 ExternalVideoEncoderTest()
86 : test_video_encoder_callback_(new TestVideoEncoderCallback()) { 86 : test_video_encoder_callback_(new TestVideoEncoderCallback()) {
87 video_config_.sender_ssrc = 1; 87 video_config_.rtp_config.ssrc = 1;
88 video_config_.incoming_feedback_ssrc = 2; 88 video_config_.incoming_feedback_ssrc = 2;
89 video_config_.rtp_config.payload_type = 127; 89 video_config_.rtp_config.payload_type = 127;
90 video_config_.use_external_encoder = true; 90 video_config_.use_external_encoder = true;
91 video_config_.width = 320; 91 video_config_.width = 320;
92 video_config_.height = 240; 92 video_config_.height = 240;
93 video_config_.max_bitrate = 5000000; 93 video_config_.max_bitrate = 5000000;
94 video_config_.min_bitrate = 1000000; 94 video_config_.min_bitrate = 1000000;
95 video_config_.start_bitrate = 2000000; 95 video_config_.start_bitrate = 2000000;
96 video_config_.max_qp = 56; 96 video_config_.max_qp = 56;
97 video_config_.min_qp = 0; 97 video_config_.min_qp = 0;
(...skipping 119 matching lines...) Expand 10 before | Expand all | Expand 10 after
217 video_frame_, capture_time, frame_encoded_callback)); 217 video_frame_, capture_time, frame_encoded_callback));
218 task_runner_->RunTasks(); 218 task_runner_->RunTasks();
219 219
220 // We need to run the task to cleanup the GPU instance. 220 // We need to run the task to cleanup the GPU instance.
221 video_encoder_.reset(NULL); 221 video_encoder_.reset(NULL);
222 task_runner_->RunTasks(); 222 task_runner_->RunTasks();
223 } 223 }
224 224
225 } // namespace cast 225 } // namespace cast
226 } // namespace media 226 } // namespace media
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698