MediaStreamRemoteVideoSource: Remove dependency to cricket::VideoFrame::GetElapsedTime()

MediaStreamRemoteVideoSource: Remove dependency to cricket::VideoFrame::GetElapsedTime()

The purpose with this CL is to remove the last use of cricket::VideoFrame::GetElapsedTime() so we can remove it, see: https://codereview.webrtc.org/1324263004/

Committed: https://crrev.com/f25569ff7c510367b0083ff54ba263c7b456ab87
Cr-Commit-Position: refs/heads/master@{#350814}

[magjed_chromium, 2015-09-25 07:12:29]

magjed@chromium.org changed reviewers:
+ tommi@chromium.org

[magjed_chromium, 2015-09-25 07:13:18]

tommi - Please take a look.

[tommi (sloooow) - chröme, 2015-09-25 07:53:50]

Lgtm. One question

https://codereview.chromium.org/1360023003/diff/1/content/renderer/media/webr...
File content/renderer/media/webrtc/media_stream_remote_video_source.cc (right):

https://codereview.chromium.org/1360023003/diff/1/content/renderer/media/webr...
content/renderer/media/webrtc/media_stream_remote_video_source.cc:65:
start_timestamp_(media::kNoTimestamp()),
Is kNoTimestamp a function?

[magjed_chromium, 2015-09-25 08:21:35]

https://codereview.chromium.org/1360023003/diff/1/content/renderer/media/webr...
File content/renderer/media/webrtc/media_stream_remote_video_source.cc (right):

https://codereview.chromium.org/1360023003/diff/1/content/renderer/media/webr...
content/renderer/media/webrtc/media_stream_remote_video_source.cc:65:
start_timestamp_(media::kNoTimestamp()),
On 2015/09/25 07:53:50, tommi wrote:
> Is kNoTimestamp a function? 
Yes, it looks like this:
MEDIA_EXPORT inline base::TimeDelta kNoTimestamp() {
  return base::TimeDelta::FromMicroseconds(kint64min);
}
I reused the code where we set the cricket::CapturedFrame.elapsed_time in the
first place:
https://code.google.com/p/chromium/codesearch#chromium/src/content/renderer/m...
That logic can be removed when this and the WebRTC CL lands btw.

I can add a bool flag |has_received_frame_| instead if you want. It is a bit
scary to reserve one of the timestamps with a special meaning. In theory,
kNoTimestamp() can be generated by a camera.

[magjed_chromium, 2015-09-25 12:21:14]

The CQ bit was checked by magjed@chromium.org

[magjed_chromium, 2015-09-25 12:21:14]

The patchset sent to the CQ was uploaded after l-g-t-m from tommi@chromium.org
Link to the patchset: https://codereview.chromium.org/1360023003/#ps20001
(title: "Check that incoming timestamp is never kNoTimestamp")

[commit-bot: I haz the power, 2015-09-25 12:21:35]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1360023003/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1360023003/20001

[commit-bot: I haz the power, 2015-09-25 13:28:25]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-09-25 13:30:50]

Patchset 2 (id:??) landed as
https://crrev.com/f25569ff7c510367b0083ff54ba263c7b456ab87
Cr-Commit-Position: refs/heads/master@{#350814}