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

Unified Diff: media/cast/transport/transport_video_sender.cc

Issue 268983002: Revert of Cast: Fix two video freezing problems (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 8 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
Index: media/cast/transport/transport_video_sender.cc
diff --git a/media/cast/transport/transport_video_sender.cc b/media/cast/transport/transport_video_sender.cc
index ef120bcb0547802e88a91ab28bb1224669e9d4a4..18624585f29f56877f1ff8a294a3644095745450 100644
--- a/media/cast/transport/transport_video_sender.cc
+++ b/media/cast/transport/transport_video_sender.cc
@@ -3,6 +3,8 @@
// found in the LICENSE file.
#include "media/cast/transport/transport_video_sender.h"
+
+#include <list>
#include "base/bind.h"
#include "base/logging.h"
@@ -19,10 +21,13 @@
base::TickClock* clock,
const scoped_refptr<base::SingleThreadTaskRunner>& transport_task_runner,
PacedSender* const paced_packet_sender)
- : rtp_sender_(clock, transport_task_runner, paced_packet_sender) {
- initialized_ = rtp_sender_.InitializeVideo(config) &&
- encryptor_.Initialize(config.rtp.config.aes_key,
- config.rtp.config.aes_iv_mask);
+ : rtp_max_delay_(base::TimeDelta::FromMilliseconds(
+ config.base.rtp_config.max_delay_ms)),
+ encryptor_(),
+ rtp_sender_(clock, transport_task_runner, paced_packet_sender) {
+ rtp_sender_.InitializeVideo(config);
+ initialized_ =
+ encryptor_.Initialize(config.base.aes_key, config.base.aes_iv_mask);
}
TransportVideoSender::~TransportVideoSender() {}
@@ -30,9 +35,6 @@
void TransportVideoSender::InsertCodedVideoFrame(
const EncodedVideoFrame* coded_frame,
const base::TimeTicks& capture_time) {
- if (!initialized_) {
- return;
- }
if (encryptor_.initialized()) {
EncodedVideoFrame encrypted_video_frame;
@@ -52,9 +54,6 @@
bool TransportVideoSender::EncryptVideoFrame(
const EncodedVideoFrame& video_frame,
EncodedVideoFrame* encrypted_frame) {
- if (!initialized_) {
- return false;
- }
if (!encryptor_.Encrypt(
video_frame.frame_id, video_frame.data, &(encrypted_frame->data)))
return false;
@@ -69,9 +68,6 @@
void TransportVideoSender::ResendPackets(
const MissingFramesAndPacketsMap& missing_frames_and_packets) {
- if (!initialized_) {
- return;
- }
rtp_sender_.ResendPackets(missing_frames_and_packets);
}
« no previous file with comments | « media/cast/transport/transport_audio_sender.cc ('k') | media/cast/video_sender/external_video_encoder_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698