Revert of New rtc::VideoSinkInterface.

Revert of New rtc::VideoSinkInterface. (patchset #7 id:120001 of https://codereview.webrtc.org/1594973006/ )

Reason for revert:
Broke chrome build. Investigating.

First error relating to AddSink method in mock_peer_connection_dependency_factory.h

Original issue's description:
> New rtc::VideoSinkInterface.
>
> The plan is that this interface should be used by all classes which receive a stream of video frames, and replace the two generic classes webrtc::VideoRendererInterface and cricket::VideoRenderer.
>
> And the list goes on, there's a dozen of different classes which act as video frame sinks.
>
> At some point, we will likely add some method to handle sink properties like, e.g, maximum useful width and height. But hopefully this can be done while keeping the interface very simple.
>
> BUG=webrtc:5426
> R=perkj@webrtc.org, pthatcher@webrtc.org
>
> Committed: https://crrev.com/a862d4563fbc26e52bed442de784094ae1dfe5ee
> Cr-Commit-Position: refs/heads/master@{#11396}

TBR=pthatcher@webrtc.org,pbos@webrtc.org,perkj@webrtc.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=webrtc:5426
Committed: https://crrev.com/2098fca39a2ebc22d6fc4a9db329668e6b31f6cb
Cr-Commit-Position: refs/heads/master@{#11397}

[nisse-webrtc, 2016-01-27 14:12:29]

Created Revert of New rtc::VideoSinkInterface.

[commit-bot: I haz the power, 2016-01-27 14:12:41]

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

[commit-bot: I haz the power, 2016-01-27 14:12:53]

Description was changed from

==========
Revert of New rtc::VideoSinkInterface. (patchset #7 id:120001 of
https://codereview.webrtc.org/1594973006/ )

Reason for revert:
Broke chrome build. Investigating.

First error relating to AddSink method in
mock_peer_connection_dependency_factory.h

Original issue's description:
> New rtc::VideoSinkInterface.
> 
> The plan is that this interface should be used by all classes which receive a
stream of video frames, and replace the two generic classes
webrtc::VideoRendererInterface and cricket::VideoRenderer.
> 
> And the list goes on, there's a dozen of different classes which act as video
frame sinks.
> 
> At some point, we will likely add some method to handle sink properties like,
e.g, maximum useful width and height. But hopefully this can be done while
keeping the interface very simple.
> 
> BUG=webrtc:5426
> R=perkj@webrtc.org, pthatcher@webrtc.org
> 
> Committed: https://crrev.com/a862d4563fbc26e52bed442de784094ae1dfe5ee
> Cr-Commit-Position: refs/heads/master@{#11396}

TBR=pthatcher@webrtc.org,pbos@webrtc.org,perkj@webrtc.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=webrtc:5426
==========

to

==========
Revert of New rtc::VideoSinkInterface. (patchset #7 id:120001 of
https://codereview.webrtc.org/1594973006/ )

Reason for revert:
Broke chrome build. Investigating.

First error relating to AddSink method in
mock_peer_connection_dependency_factory.h

Original issue's description:
> New rtc::VideoSinkInterface.
> 
> The plan is that this interface should be used by all classes which receive a
stream of video frames, and replace the two generic classes
webrtc::VideoRendererInterface and cricket::VideoRenderer.
> 
> And the list goes on, there's a dozen of different classes which act as video
frame sinks.
> 
> At some point, we will likely add some method to handle sink properties like,
e.g, maximum useful width and height. But hopefully this can be done while
keeping the interface very simple.
> 
> BUG=webrtc:5426
> R=perkj@webrtc.org, pthatcher@webrtc.org
> 
> Committed: https://crrev.com/a862d4563fbc26e52bed442de784094ae1dfe5ee
> Cr-Commit-Position: refs/heads/master@{#11396}

TBR=pthatcher@webrtc.org,pbos@webrtc.org,perkj@webrtc.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=webrtc:5426
==========

[commit-bot: I haz the power, 2016-01-27 14:12:54]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-01-27 14:13:05]

Description was changed from

==========
Revert of New rtc::VideoSinkInterface. (patchset #7 id:120001 of
https://codereview.webrtc.org/1594973006/ )

Reason for revert:
Broke chrome build. Investigating.

First error relating to AddSink method in
mock_peer_connection_dependency_factory.h

Original issue's description:
> New rtc::VideoSinkInterface.
> 
> The plan is that this interface should be used by all classes which receive a
stream of video frames, and replace the two generic classes
webrtc::VideoRendererInterface and cricket::VideoRenderer.
> 
> And the list goes on, there's a dozen of different classes which act as video
frame sinks.
> 
> At some point, we will likely add some method to handle sink properties like,
e.g, maximum useful width and height. But hopefully this can be done while
keeping the interface very simple.
> 
> BUG=webrtc:5426
> R=perkj@webrtc.org, pthatcher@webrtc.org
> 
> Committed: https://crrev.com/a862d4563fbc26e52bed442de784094ae1dfe5ee
> Cr-Commit-Position: refs/heads/master@{#11396}

TBR=pthatcher@webrtc.org,pbos@webrtc.org,perkj@webrtc.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=webrtc:5426
==========

to

==========
Revert of New rtc::VideoSinkInterface. (patchset #7 id:120001 of
https://codereview.webrtc.org/1594973006/ )

Reason for revert:
Broke chrome build. Investigating.

First error relating to AddSink method in
mock_peer_connection_dependency_factory.h

Original issue's description:
> New rtc::VideoSinkInterface.
>
> The plan is that this interface should be used by all classes which receive a
stream of video frames, and replace the two generic classes
webrtc::VideoRendererInterface and cricket::VideoRenderer.
>
> And the list goes on, there's a dozen of different classes which act as video
frame sinks.
>
> At some point, we will likely add some method to handle sink properties like,
e.g, maximum useful width and height. But hopefully this can be done while
keeping the interface very simple.
>
> BUG=webrtc:5426
> R=perkj@webrtc.org, pthatcher@webrtc.org
>
> Committed: https://crrev.com/a862d4563fbc26e52bed442de784094ae1dfe5ee
> Cr-Commit-Position: refs/heads/master@{#11396}

TBR=pthatcher@webrtc.org,pbos@webrtc.org,perkj@webrtc.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=webrtc:5426

Committed: https://crrev.com/2098fca39a2ebc22d6fc4a9db329668e6b31f6cb
Cr-Commit-Position: refs/heads/master@{#11397}
==========

[commit-bot: I haz the power, 2016-01-27 14:13:06]

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