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

Unified Diff: remoting/protocol/video_frame_pump.cc

Issue 2334293002: Two minor cleanups in remoting protocol. (Closed)
Patch Set: . Created 4 years, 3 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: remoting/protocol/video_frame_pump.cc
diff --git a/remoting/protocol/video_frame_pump.cc b/remoting/protocol/video_frame_pump.cc
index 5e24d40fb0d5ec1550720d7f42ab3b603b729f44..505ac32ed378c1b06ff66aeba2af313eb6f2d7f1 100644
--- a/remoting/protocol/video_frame_pump.cc
+++ b/remoting/protocol/video_frame_pump.cc
@@ -30,8 +30,6 @@ namespace protocol {
// must be smaller than the minimum RTO used in PseudoTCP, which is 250ms.
static const int kKeepAlivePacketIntervalMs = 200;
-static bool g_enable_timestamps = false;
-
VideoFramePump::FrameTimestamps::FrameTimestamps() {}
VideoFramePump::FrameTimestamps::~FrameTimestamps() {}
@@ -42,11 +40,6 @@ VideoFramePump::PacketWithTimestamps::PacketWithTimestamps(
VideoFramePump::PacketWithTimestamps::~PacketWithTimestamps() {}
-// static
-void VideoFramePump::EnableTimestampsForTests() {
- g_enable_timestamps = true;
-}
-
VideoFramePump::VideoFramePump(
scoped_refptr<base::SingleThreadTaskRunner> encode_task_runner,
std::unique_ptr<webrtc::DesktopCapturer> capturer,
@@ -222,10 +215,6 @@ void VideoFramePump::SendPacket(std::unique_ptr<PacketWithTimestamps> packet) {
void VideoFramePump::UpdateFrameTimers(VideoPacket* packet,
FrameTimestamps* timestamps) {
- if (g_enable_timestamps)
- packet->set_timestamp(timestamps->capture_ended_time.ToInternalValue());
-
-
if (!timestamps->input_event_received_time.is_null()) {
packet->set_capture_pending_time_ms((timestamps->capture_started_time -
timestamps->input_event_received_time)

Powered by Google App Engine
This is Rietveld 408576698