Move RTP for synchroninzation and rename classes, files and variables.

Move RTP for synchroninzation and rename classes, files and variables.

This CL removes (almost) the last RTP references in VideoReceiveStream.
There are still references to RTPFragmentationHeader and SSRCs, which
will be dealt with later.

There are also new GUARDED_BY and thred checker added to the
synchronization class.

When there are othre transports than RTP, there will instead be an
interface + inheritance for RtpStreamReceiver and
RtpStreamSynchronizattion in VideoReceiveStream. This work will be done
when we actually know how we want to make thee transport interface.

BUG=webrtc:5838
Committed: https://crrev.com/4cd2790f170028b2864b7fa66d5e7400631b4273
Cr-Commit-Position: refs/heads/master@{#13655}

[mflodman, 2016-08-04 08:24:05]

mflodman@webrtc.org changed reviewers:
+ asapersson@webrtc.org

[mflodman, 2016-08-04 08:24:05]

Åsa,
Would you mind taking a look at this CL?

https://codereview.webrtc.org/2216533002/diff/1/webrtc/video/rtp_streams_sync...
File webrtc/video/rtp_streams_synchronizer.h (right):

https://codereview.webrtc.org/2216533002/diff/1/webrtc/video/rtp_streams_sync...
webrtc/video/rtp_streams_synchronizer.h:35: class RtpStreamsSynchronizer :
public Module {
This is not really a new file, rather vie_sync_module.h being renamed and with
name and signature changes.

[åsapersson, 2016-08-05 09:11:47]

lgtm

https://codereview.webrtc.org/2216533002/diff/1/webrtc/video/rtp_streams_sync...
File webrtc/video/rtp_streams_synchronizer.cc (right):

https://codereview.webrtc.org/2216533002/diff/1/webrtc/video/rtp_streams_sync...
webrtc/video/rtp_streams_synchronizer.cc:92: voe_channel_id_));
nit: check indentation

[mflodman, 2016-08-05 12:33:42]

https://codereview.webrtc.org/2216533002/diff/1/webrtc/video/rtp_streams_sync...
File webrtc/video/rtp_streams_synchronizer.cc (right):

https://codereview.webrtc.org/2216533002/diff/1/webrtc/video/rtp_streams_sync...
webrtc/video/rtp_streams_synchronizer.cc:92: voe_channel_id_));
On 2016/08/05 09:11:47, åsapersson wrote:
> nit: check indentation

Done.

[mflodman, 2016-08-05 12:33:47]

The CQ bit was checked by mflodman@webrtc.org

[mflodman, 2016-08-05 12:33:48]

The patchset sent to the CQ was uploaded after l-g-t-m from
asapersson@webrtc.org
Link to the patchset: https://codereview.webrtc.org/2216533002/#ps20001 (title:
"Indentation + rebase.")

[commit-bot: I haz the power, 2016-08-05 12:33:59]

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-08-05 13:28:50]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-08-05 13:28:59]

Description was changed from

==========
Move RTP for synchroninzation and rename classes, files and variables.

This CL removes (almost) the last RTP references in VideoReceiveStream.
There are still references to RTPFragmentationHeader and SSRCs, which
will be dealt with later.

There are also new GUARDED_BY and thred checker added to the
synchronization class.

When there are othre transports than RTP, there will instead be an
interface + inheritance for RtpStreamReceiver and
RtpStreamSynchronizattion in VideoReceiveStream. This work will be done
when we actually know how we want to make thee transport interface.

BUG=webrtc:5838
==========

to

==========
Move RTP for synchroninzation and rename classes, files and variables.

This CL removes (almost) the last RTP references in VideoReceiveStream.
There are still references to RTPFragmentationHeader and SSRCs, which
will be dealt with later.

There are also new GUARDED_BY and thred checker added to the
synchronization class.

When there are othre transports than RTP, there will instead be an
interface + inheritance for RtpStreamReceiver and
RtpStreamSynchronizattion in VideoReceiveStream. This work will be done
when we actually know how we want to make thee transport interface.

BUG=webrtc:5838

Committed: https://crrev.com/4cd2790f170028b2864b7fa66d5e7400631b4273
Cr-Commit-Position: refs/heads/master@{#13655}
==========

[commit-bot: I haz the power, 2016-08-05 13:29:00]

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