Minor improvements to VideoProcessorIntegrationTest.

Minor improvements to VideoProcessorIntegrationTest.

* Create all encoders/decoders using factories.
* Add ::Release() method, to mirror the existing ::Init().
* Remove unnecessary ::test prefixes.
* Reorganize constants and members.
* Remove extraneous packet loss rate assignments.
* Remove members |start_frame_rate_| and |num_temporal_layers_|.
* Explicitly give ::SetUpObjects(.) access to initial rates.
* Change visualization output file names.

BUG=webrtc:6634
Review-Url: https://codereview.webrtc.org/2999613002
Cr-Commit-Position: refs/heads/master@{#19326}
Committed: https://chromium.googlesource.com/external/webrtc/+/77920a415bdbf71946125bc4bbdbe5b23e02e61a

[brandtr, 2017-08-08 08:26:40]

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

[brandtr, 2017-08-08 08:26:41]

... and this followup, which is more interesting.

[brandtr, 2017-08-08 10:43:31]

Reorder constants and remove unnecessary packet loss assignments.

[brandtr, 2017-08-08 10:43:56]

Description was changed from

==========
Minor improvements to VideoProcessorIntegrationTest.

* Create all encoders/decoders using factories.
* Remove unnecessary ::test prefixes.
* Reorganize constants and members.

BUG=webrtc:6634
==========

to

==========
Minor improvements to VideoProcessorIntegrationTest.

* Create all encoders/decoders using factories.
* Remove unnecessary ::test prefixes.
* Reorganize constants and members.
* Remove extraneous packet loss rate assignments.

BUG=webrtc:6634
==========

[brandtr, 2017-08-08 15:37:05]

Rebase.

[brandtr, 2017-08-09 10:51:59]

Rebase.

[brandtr, 2017-08-09 11:14:47]

Description was changed from

==========
Minor improvements to VideoProcessorIntegrationTest.

* Create all encoders/decoders using factories.
* Remove unnecessary ::test prefixes.
* Reorganize constants and members.
* Remove extraneous packet loss rate assignments.

BUG=webrtc:6634
==========

to

==========
Minor improvements to VideoProcessorIntegrationTest.

* Create all encoders/decoders using factories.
* Remove unnecessary ::test prefixes.
* Reorganize constants and members.
* Remove extraneous packet loss rate assignments.
* Remove members |start_frame_rate_| and |num_temporal_layers_|.
* Explicitly give ::SetUpObjects(.) access to initial rates.
* Change visualization output file names.

BUG=webrtc:6634
==========

[brandtr, 2017-08-09 11:15:48]

Description was changed from

==========
Minor improvements to VideoProcessorIntegrationTest.

* Create all encoders/decoders using factories.
* Remove unnecessary ::test prefixes.
* Reorganize constants and members.
* Remove extraneous packet loss rate assignments.
* Remove members |start_frame_rate_| and |num_temporal_layers_|.
* Explicitly give ::SetUpObjects(.) access to initial rates.
* Change visualization output file names.

BUG=webrtc:6634
==========

to

==========
Minor improvements to VideoProcessorIntegrationTest.

* Create all encoders/decoders using factories.
* Add ::Release() method, to mirror the existing ::Init().
* Remove unnecessary ::test prefixes.
* Reorganize constants and members.
* Remove extraneous packet loss rate assignments.
* Remove members |start_frame_rate_| and |num_temporal_layers_|.
* Explicitly give ::SetUpObjects(.) access to initial rates.
* Change visualization output file names.

BUG=webrtc:6634
==========

[brandtr, 2017-08-09 11:18:24]

Added some more cleanups.

[brandtr, 2017-08-09 11:57:11]

Set startBitrate, to fix tests.

[brandtr, 2017-08-10 08:11:02]

I just verified that the output of the VideoProcessorIntegrationTest is
identical before and after this change, except for the encoding/decoding times
that vary a bit.

[åsapersson, 2017-08-11 13:13:01]

lgtm

[brandtr, 2017-08-11 13:24:39]

The CQ bit was checked by brandtr@webrtc.org

[commit-bot: I haz the power, 2017-08-11 13:24:43]

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, 2017-08-11 13:27:52]

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

[commit-bot: I haz the power, 2017-08-11 13:27:53]

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

[brandtr, 2017-08-11 13:45:12]

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

[commit-bot: I haz the power, 2017-08-11 13:45:22]

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, 2017-08-11 13:48:52]

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

[commit-bot: I haz the power, 2017-08-11 13:48:53]

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

[brandtr, 2017-08-11 13:53:14]

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

[commit-bot: I haz the power, 2017-08-11 13:53:22]

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, 2017-08-11 14:36:52]

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

[commit-bot: I haz the power, 2017-08-11 14:36:53]

Dry run: This issue passed the CQ dry run.

[brandtr, 2017-08-11 14:45:22]

The CQ bit was checked by brandtr@webrtc.org

[brandtr, 2017-08-11 14:45:24]

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/2999613002/#ps140001 (title:
"Fix iOS more.")

[commit-bot: I haz the power, 2017-08-11 14:45:37]

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, 2017-08-11 14:48:16]

CQ is committing da patch.

Bot data: {"patchset_id": 140001, "attempt_start_ts": 1502462722864230,
"parent_rev": "8dc0ff813d8b8658cb51f1cca244f1dca11ff423", "commit_rev":
"77920a415bdbf71946125bc4bbdbe5b23e02e61a"}

[commit-bot: I haz the power, 2017-08-11 14:48:21]

Description was changed from

==========
Minor improvements to VideoProcessorIntegrationTest.

* Create all encoders/decoders using factories.
* Add ::Release() method, to mirror the existing ::Init().
* Remove unnecessary ::test prefixes.
* Reorganize constants and members.
* Remove extraneous packet loss rate assignments.
* Remove members |start_frame_rate_| and |num_temporal_layers_|.
* Explicitly give ::SetUpObjects(.) access to initial rates.
* Change visualization output file names.

BUG=webrtc:6634
==========

to

==========
Minor improvements to VideoProcessorIntegrationTest.

* Create all encoders/decoders using factories.
* Add ::Release() method, to mirror the existing ::Init().
* Remove unnecessary ::test prefixes.
* Reorganize constants and members.
* Remove extraneous packet loss rate assignments.
* Remove members |start_frame_rate_| and |num_temporal_layers_|.
* Explicitly give ::SetUpObjects(.) access to initial rates.
* Change visualization output file names.

BUG=webrtc:6634

Review-Url: https://codereview.webrtc.org/2999613002
Cr-Commit-Position: refs/heads/master@{#19326}
Committed:
https://chromium.googlesource.com/external/webrtc/+/77920a415bdbf71946125bc4b...
==========

[commit-bot: I haz the power, 2017-08-11 14:48:22]

Committed patchset #8 (id:140001) as
https://chromium.googlesource.com/external/webrtc/+/77920a415bdbf71946125bc4b...