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

Unified Diff: content/renderer/media/webrtc/media_stream_remote_video_source.cc

Issue 2158923004: Convert media constants to constexpr. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 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
Index: content/renderer/media/webrtc/media_stream_remote_video_source.cc
diff --git a/content/renderer/media/webrtc/media_stream_remote_video_source.cc b/content/renderer/media/webrtc/media_stream_remote_video_source.cc
index 57e8f94994baee9fa3fab185d9fc1a2098e22b7d..30c25baa3dbf659b70c4316d13b9aecdd8333b1c 100644
--- a/content/renderer/media/webrtc/media_stream_remote_video_source.cc
+++ b/content/renderer/media/webrtc/media_stream_remote_video_source.cc
@@ -80,7 +80,7 @@ MediaStreamRemoteVideoSource::RemoteVideoSourceDelegate::
const VideoCaptureDeliverFrameCB& new_frame_callback)
: io_task_runner_(io_task_runner),
frame_callback_(new_frame_callback),
- start_timestamp_(media::kNoTimestamp()),
+ start_timestamp_(media::kNoTimestamp),
// TODO(qiangchen): There can be two differences between clocks: 1)
// the offset, 2) the rate (i.e., one clock runs faster than the other).
// See http://crbug/516700
@@ -105,8 +105,8 @@ void MediaStreamRemoteVideoSource::RemoteVideoSourceDelegate::OnFrame(
TRACE_EVENT1("webrtc", "RemoteVideoSourceDelegate::RenderFrame",
"Ideal Render Instant", render_time.ToInternalValue());
- CHECK_NE(media::kNoTimestamp(), incoming_timestamp);
- if (start_timestamp_ == media::kNoTimestamp())
+ CHECK_NE(media::kNoTimestamp, incoming_timestamp);
+ if (start_timestamp_ == media::kNoTimestamp)
start_timestamp_ = incoming_timestamp;
const base::TimeDelta elapsed_timestamp =
incoming_timestamp - start_timestamp_;

Powered by Google App Engine
This is Rietveld 408576698