Delete cricket::VideoFrame::GetTimeStamp.

Delete cricket::VideoFrame::GetTimeStamp.

TBR=tkchin@webrtc.org  # Trivial change to VideoRendererAdapter
BUG=webrtc:5682
Committed: https://crrev.com/fd6c99e43137d01fa6c120f7160f7c2999d1d8a3
Cr-Commit-Position: refs/heads/master@{#14037}

[nisse-webrtc, 2016-09-01 11:30:07]

The CQ bit was checked by nisse@webrtc.org to run a CQ dry run

[commit-bot: I haz the power, 2016-09-01 11:30:12]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...

[commit-bot: I haz the power, 2016-09-01 11:31:26]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-09-01 11:31:30]

Dry run: Try jobs failed on following builders:
  mac_gyp_rel on master.tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/mac_gyp_rel/builds/177)

[nisse-webrtc, 2016-09-01 13:03:41]

nisse@webrtc.org changed reviewers:
+ perkj@webrtc.org

[nisse-webrtc, 2016-09-01 13:03:41]

Please have a look. I don't think it's used downstream.

[perkj_webrtc, 2016-09-01 13:44:45]

lgtm

https://codereview.webrtc.org/2305623002/diff/1/webrtc/sdk/objc/Framework/Cla...
File webrtc/sdk/objc/Framework/Classes/RTCVideoRendererAdapter.mm (right):

https://codereview.webrtc.org/2305623002/diff/1/webrtc/sdk/objc/Framework/Cla...
webrtc/sdk/objc/Framework/Classes/RTCVideoRendererAdapter.mm:33:
timeStampNs:nativeVideoFrame.timestamp_us() *
is RTCVideoFrame using ns?

[perkj_webrtc, 2016-09-01 13:44:55]

lgtm

[perkj_webrtc, 2016-09-01 13:44:56]

lgtm

[nisse-webrtc, 2016-09-01 13:50:52]

https://codereview.webrtc.org/2305623002/diff/1/webrtc/sdk/objc/Framework/Cla...
File webrtc/sdk/objc/Framework/Classes/RTCVideoRendererAdapter.mm (right):

https://codereview.webrtc.org/2305623002/diff/1/webrtc/sdk/objc/Framework/Cla...
webrtc/sdk/objc/Framework/Classes/RTCVideoRendererAdapter.mm:33:
timeStampNs:nativeVideoFrame.timestamp_us() *
On 2016/09/01 13:44:44, perkj_webrtc wrote:
> is RTCVideoFrame using ns?

I think so, see
https://cs.chromium.org/chromium/src/third_party/webrtc/sdk/objc/Framework/He...

It doesn't wrap any of the C++ VideoFrame classes, but it does about the same
thing (pointer to VideoFrameBuffer, timestamp, rotation).

[nisse-webrtc, 2016-09-01 13:51:32]

The CQ bit was checked by nisse@webrtc.org

[commit-bot: I haz the power, 2016-09-01 13:51:43]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...

[nisse-webrtc, 2016-09-01 14:25:36]

Description was changed from

==========
Delete cricket::VideoFrame::GetTimeStamp.

BUG=webrtc:5682
==========

to

==========
Delete cricket::VideoFrame::GetTimeStamp.

TBR=tkchin@webrtc.org  # Trivial change to VideoRendererAdapter
BUG=webrtc:5682
==========

[commit-bot: I haz the power, 2016-09-01 15:52:14]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-09-01 15:52:15]

Try jobs failed on following builders:
  android_arm64_rel on master.tryserver.webrtc (JOB_TIMED_OUT, no build URL)

[nisse-webrtc, 2016-09-02 08:01:18]

The CQ bit was checked by nisse@webrtc.org

[commit-bot: I haz the power, 2016-09-02 08:01:23]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...

[commit-bot: I haz the power, 2016-09-02 08:33:57]

Description was changed from

==========
Delete cricket::VideoFrame::GetTimeStamp.

TBR=tkchin@webrtc.org  # Trivial change to VideoRendererAdapter
BUG=webrtc:5682
==========

to

==========
Delete cricket::VideoFrame::GetTimeStamp.

TBR=tkchin@webrtc.org  # Trivial change to VideoRendererAdapter
BUG=webrtc:5682
==========

[commit-bot: I haz the power, 2016-09-02 08:33:58]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-09-02 08:34:09]

Description was changed from

==========
Delete cricket::VideoFrame::GetTimeStamp.

TBR=tkchin@webrtc.org  # Trivial change to VideoRendererAdapter
BUG=webrtc:5682
==========

to

==========
Delete cricket::VideoFrame::GetTimeStamp.

TBR=tkchin@webrtc.org  # Trivial change to VideoRendererAdapter
BUG=webrtc:5682

Committed: https://crrev.com/fd6c99e43137d01fa6c120f7160f7c2999d1d8a3
Cr-Commit-Position: refs/heads/master@{#14037}
==========

[commit-bot: I haz the power, 2016-09-02 08:34:11]

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

[nisse-webrtc, 2016-09-02 09:06:49]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.webrtc.org/2306953002/ by nisse@webrtc.org.

The reason for reverting is: Broke downstream project..