Preliminary change for new rtc rendering algorithm

content/renderer/media/webrtc/media_stream_remote_video_source.cc

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 e81b37701de15df10019bdf517c105cdc50cffef..41ea5b1d7a6e8513f7374e42a29ce8c0406491b8 100644
--- a/content/renderer/media/webrtc/media_stream_remote_video_source.cc
+++ b/content/renderer/media/webrtc/media_stream_remote_video_source.cc
@@ -62,7 +62,7 @@ RemoteVideoSourceDelegate::~RemoteVideoSourceDelegate() {
 
 void MediaStreamRemoteVideoSource::RemoteVideoSourceDelegate::RenderFrame(
     const cricket::VideoFrame* incoming_frame) {
-  TRACE_EVENT0("webrtc", "RemoteVideoSourceDelegate::RenderFrame");
+  TRACE_EVENT_BEGIN0("webrtc", "RemoteVideoSourceDelegate::RenderFrame");
   base::TimeDelta timestamp = base::TimeDelta::FromMicroseconds(
       incoming_frame->GetElapsedTime() / rtc::kNumNanosecsPerMicrosec);
 
@@ -96,6 +96,13 @@ void MediaStreamRemoteVideoSource::RemoteVideoSourceDelegate::RenderFrame(
         base::Bind(&base::DeletePointer<cricket::VideoFrame>, frame->Copy()));
   }
 
+  video_frame->set_render_time(base::TimeTicks::FromInternalValue(

[DaleCurtis, 2015/07/30 05:55:24]

Using InternalValue like this isn't correct. Why do this instead of just setting
the timestamp field directly?

[qiangchen, 2015/07/30 17:40:26]

cricket::VideoFrame's timestamp is a number in unit nanosecond. And here I need
to convert it to a TimeTicks. What about
"base::TimeTicks()+base::TimeDelta::FromMicroseconds(...)"?

+      incoming_frame->GetTimeStamp() / 1000));
+
+  TRACE_EVENT_END1("webrtc", "RemoteVideoSourceDelegate::RenderFrame",
+                   "Ideal Render Instant",
+                   video_frame->render_time().ToInternalValue());
+
   io_task_runner_->PostTask(
       FROM_HERE, base::Bind(&RemoteVideoSourceDelegate::DoRenderFrameOnIOThread,
                             this, video_frame));