Reorder member functions in RtpFecTest. (pt. 4b)

Reorder member functions in RtpFecTest.

Place member functions before tests. No changes to the functionality.

BUG=webrtc:5654
R=danilchap@webrtc.org

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

[brandtr, 2016-08-30 08:33:22]

Patchset #1 (id:1) has been deleted

[brandtr, 2016-08-30 09:19:08]

Rebase.

[brandtr, 2016-08-30 09:23:47]

brandtr@webrtc.org changed reviewers:
+ danilchap@webrtc.org

[brandtr, 2016-08-30 09:23:47]

This is just a cleanup CL, where some member functions have
been moved around.

I placed it "in between" pt. 4 and pt. 5. No particular reason for that
placement, except that I found the order annoying when I added a unit test to pt
5.

[danilchap, 2016-08-30 12:45:17]

On 2016/08/30 09:23:47, brandtr wrote:
> This is just a cleanup CL, where some member functions have
> been moved around.
> 
> I placed it "in between" pt. 4 and pt. 5. No particular reason for that
> placement, except that I found the order annoying when I added a unit test to
pt
> 5.

better to depend on as little other patches as possible. So as long as you do
not need anything from pt. 5, place before it.
If it is possible to depend directly on master (I guess not in this case) - that
would be the perfect dependency set.

lgtm

[brandtr, 2016-08-30 14:28:10]

Rebase.

[brandtr, 2016-08-30 14:31:02]

Description was changed from

==========
Reorder member functions in RtpFecTest.

Place member functions before tests. No changes to the functionality.

BUG=webrtc:5654
==========

to

==========
Reorder member functions in RtpFecTest.

Place member functions before tests. No changes to the functionality.

BUG=webrtc:5654
==========

[brandtr, 2016-09-01 10:12:02]

Rebase.

[brandtr, 2016-09-02 07:24:36]

Rebase.

[brandtr, 2016-09-02 08:27:02]

Rebase.

[brandtr, 2016-09-02 11:07:37]

Rebase.

[brandtr, 2016-09-06 14:35:50]

Rebase.

[brandtr, 2016-09-13 11:38:40]

Rebase.

[brandtr, 2016-09-13 11:43:35]

Rebase.

[brandtr, 2016-09-14 11:37:33]

Rebase.

[brandtr, 2016-09-20 08:51:08]

Rebase.

[brandtr, 2016-09-20 09:31:13]

Rebase after gyp deprecation.

[brandtr, 2016-09-20 11:07:13]

Rebase.

[brandtr, 2016-09-21 06:18:44]

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

[commit-bot: I haz the power, 2016-09-21 06:18:54]

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-21 06:29:24]

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

[commit-bot: I haz the power, 2016-09-21 06:29:25]

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

[brandtr, 2016-09-21 09:18:58]

The CQ bit was checked by brandtr@webrtc.org

[brandtr, 2016-09-21 09:18:59]

The patchset sent to the CQ was uploaded after l-g-t-m from danilchap@webrtc.org
Link to the patchset: https://codereview.webrtc.org/2297533002/#ps280001 (title:
"Rebase.")

[commit-bot: I haz the power, 2016-09-21 09:19:09]

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-21 09:48:32]

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

[commit-bot: I haz the power, 2016-09-21 09:48:33]

Try jobs failed on following builders:
  android_dbg on master.tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/android_dbg/builds/16940)

[brandtr, 2016-09-21 10:01:33]

Description was changed from

==========
Reorder member functions in RtpFecTest.

Place member functions before tests. No changes to the functionality.

BUG=webrtc:5654
==========

to

==========
Reorder member functions in RtpFecTest.

Place member functions before tests. No changes to the functionality.

BUG=webrtc:5654
R=danilchap@webrtc.org

Committed:
https://chromium.googlesource.com/external/webrtc/+/ea7beb9741da38043432aefb6...
==========

[brandtr, 2016-09-21 10:01:35]

Committed patchset #14 (id:280001) manually as
ea7beb9741da38043432aefb695731c49ff7c413 (presubmit successful).

[commit-bot: I haz the power, 2016-09-21 10:01:38]

Description was changed from

==========
Reorder member functions in RtpFecTest.

Place member functions before tests. No changes to the functionality.

BUG=webrtc:5654
R=danilchap@webrtc.org

Committed:
https://chromium.googlesource.com/external/webrtc/+/ea7beb9741da38043432aefb6...
==========

to

==========
Reorder member functions in RtpFecTest.

Place member functions before tests. No changes to the functionality.

BUG=webrtc:5654
R=danilchap@webrtc.org

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