[Cast] Clean-up: Merge RtpReceiver+AudioReceiver+VideoReceiver-->FrameReceiver.

[Cast] Clean-up: Merge RtpReceiver+AudioReceiver+VideoReceiver-->FrameReceiver.

Creates a new media/cast/receiver directory, and consolidates all code
in media/cast/audio_receiver and media/cast/video_receiver, and some
modules from media/cast/rtp_receiver into one place.

Removed FrameReceiver proxy interface in cast_receiver.h, since there
are no special threading/lifecycle concerns.  Clients will directly call
methods in the CastReceiver interface to request frames on the MAIN
thread.

BUG=378568
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=274464

[miu, 2014-05-31 02:24:38]

hubbe or hclam: PTAL.  This is *much much* smaller than the delta LOC suggests,
for obvious reasons.  I managed to get the "diff similarity" to detect all the
file moves, so it should be pretty easy to review.

[Alpha Left Google, 2014-06-02 20:16:59]

https://codereview.chromium.org/308043006/diff/20001/media/cast/receiver/cast...
File media/cast/receiver/cast_receiver_impl.cc (right):

https://codereview.chromium.org/308043006/diff/20001/media/cast/receiver/cast...
media/cast/receiver/cast_receiver_impl.cc:80: return
base::Bind(&CastReceiverImpl::DispatchReceivedPacket,
I think we need a TODO here.

https://codereview.chromium.org/308043006/diff/20001/media/cast/receiver/fram...
File media/cast/receiver/frame_receiver.h (right):

https://codereview.chromium.org/308043006/diff/20001/media/cast/receiver/fram...
media/cast/receiver/frame_receiver.h:61: void RequestEncodedFrame(const
FrameEncodedCallback& callback);
nit: The type FrameEncodedCallback is misleading because it is called when an
encoded frame is fully received and prased. But it sounds like it is called when
a frame is encoded. Maybe rename ReceiveEncodedFrameCallback?

[miu, 2014-06-02 22:36:07]

Done and done.  LGTY?

https://codereview.chromium.org/308043006/diff/20001/media/cast/receiver/cast...
File media/cast/receiver/cast_receiver_impl.cc (right):

https://codereview.chromium.org/308043006/diff/20001/media/cast/receiver/cast...
media/cast/receiver/cast_receiver_impl.cc:80: return
base::Bind(&CastReceiverImpl::DispatchReceivedPacket,
On 2014/06/02 20:16:59, Alpha wrote:
> I think we need a TODO here.

Done.

https://codereview.chromium.org/308043006/diff/20001/media/cast/receiver/fram...
File media/cast/receiver/frame_receiver.h (right):

https://codereview.chromium.org/308043006/diff/20001/media/cast/receiver/fram...
media/cast/receiver/frame_receiver.h:61: void RequestEncodedFrame(const
FrameEncodedCallback& callback);
On 2014/06/02 20:16:59, Alpha wrote:
> nit: The type FrameEncodedCallback is misleading because it is called when an
> encoded frame is fully received and prased. But it sounds like it is called
when
> a frame is encoded. Maybe rename ReceiveEncodedFrameCallback?

Done.  Good call.

[Alpha Left Google, 2014-06-02 23:00:32]

lgtm

[miu, 2014-06-02 23:28:03]

The CQ bit was checked by miu@chromium.org

[commit-bot: I haz the power, 2014-06-02 23:29:14]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/miu@chromium.org/308043006/40001

[commit-bot: I haz the power, 2014-06-03 10:45:17]

Change committed as 274464