Rename config structs in VideoProcessor tests.

Rename config structs in VideoProcessor tests.

This CL harmonizes and improves the naming of the config structs
in the VideoProcessor tests. It should have no functional implications.

CodecConfigPars -> CodecParams:
  This struct mainly contains codec settings.

QualityMetrics -> QualityThresholds:
  This struct contains thresholds against which the calculated
  PSNR and SSIM metrics are compared to.

RateControlMetrics -> RateControlThresholds:
  This struct contains thresholds against which the calculated
  rate control metrics are compared to.

BUG=webrtc:6634
Review-Url: https://codereview.webrtc.org/2703333004
Cr-Commit-Position: refs/heads/master@{#16794}
Committed: https://chromium.googlesource.com/external/webrtc/+/1e46434e7ef90da1aebaf28cb7ac0d674c857e7a

[brandtr, 2017-02-21 12:04:30]

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

[brandtr, 2017-02-21 12:04:31]

https://codereview.webrtc.org/2703333004/diff/1/webrtc/modules/video_coding/c...
File webrtc/modules/video_coding/codecs/test/videoprocessor_integrationtest.h
(left):

https://codereview.webrtc.org/2703333004/diff/1/webrtc/modules/video_coding/c...
webrtc/modules/video_coding/codecs/test/videoprocessor_integrationtest.h:83:
double minimum_avg_psnr;
'minimum' -> 'min' to harmonize with RateControlThreshold field naming.

https://codereview.webrtc.org/2703333004/diff/1/webrtc/modules/video_coding/c...
webrtc/modules/video_coding/codecs/test/videoprocessor_integrationtest.h:379:
void VerifyRateControl(int update_index,
To harmonize with the "UpdateRateControlMetrics" method.

[åsapersson, 2017-02-21 14:57:59]

lgtm

[brandtr, 2017-02-22 07:48:38]

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

[brandtr, 2017-02-22 07:48:38]

+sprang as owner

[brandtr, 2017-02-22 09:51:43]

Rebase.

[brandtr, 2017-02-22 12:16:02]

Rebase.

[sprang_webrtc, 2017-02-22 16:39:31]

lgtm with nit

https://codereview.webrtc.org/2703333004/diff/60001/webrtc/modules/video_codi...
File webrtc/modules/video_coding/codecs/test/videoprocessor_integrationtest.h
(right):

https://codereview.webrtc.org/2703333004/diff/60001/webrtc/modules/video_codi...
webrtc/modules/video_coding/codecs/test/videoprocessor_integrationtest.h:80:
float packet_loss;
packet_loss_perecent? or fraction?

[brandtr, 2017-02-23 08:16:45]

https://codereview.webrtc.org/2703333004/diff/60001/webrtc/modules/video_codi...
File webrtc/modules/video_coding/codecs/test/videoprocessor_integrationtest.h
(right):

https://codereview.webrtc.org/2703333004/diff/60001/webrtc/modules/video_codi...
webrtc/modules/video_coding/codecs/test/videoprocessor_integrationtest.h:80:
float packet_loss;
On 2017/02/22 16:39:31, språng wrote:
> packet_loss_perecent? or fraction?

Fixed :)

[brandtr, 2017-02-23 08:17:14]

The CQ bit was checked by brandtr@webrtc.org

[brandtr, 2017-02-23 08:17:14]

The patchset sent to the CQ was uploaded after l-g-t-m from
asapersson@webrtc.org, sprang@webrtc.org
Link to the patchset: https://codereview.webrtc.org/2703333004/#ps80001 (title:
"sprang comments 1.")

[commit-bot: I haz the power, 2017-02-23 08:17:20]

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-02-23 10:49:16]

CQ is committing da patch.

Bot data: {"patchset_id": 80001, "attempt_start_ts": 1487837834449050,
"parent_rev": "f166e1bcaba8b873205b59633bbdc99efe293f1c", "commit_rev":
"1e46434e7ef90da1aebaf28cb7ac0d674c857e7a"}

[commit-bot: I haz the power, 2017-02-23 10:49:19]

Description was changed from

==========
Rename config structs in VideoProcessor tests.

This CL harmonizes and improves the naming of the config structs
in the VideoProcessor tests. It should have no functional implications.

CodecConfigPars -> CodecParams:
  This struct mainly contains codec settings.

QualityMetrics -> QualityThresholds:
  This struct contains thresholds against which the calculated
  PSNR and SSIM metrics are compared to.

RateControlMetrics -> RateControlThresholds:
  This struct contains thresholds against which the calculated
  rate control metrics are compared to.

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

to

==========
Rename config structs in VideoProcessor tests.

This CL harmonizes and improves the naming of the config structs
in the VideoProcessor tests. It should have no functional implications.

CodecConfigPars -> CodecParams:
  This struct mainly contains codec settings.

QualityMetrics -> QualityThresholds:
  This struct contains thresholds against which the calculated
  PSNR and SSIM metrics are compared to.

RateControlMetrics -> RateControlThresholds:
  This struct contains thresholds against which the calculated
  rate control metrics are compared to.

BUG=webrtc:6634

Review-Url: https://codereview.webrtc.org/2703333004
Cr-Commit-Position: refs/heads/master@{#16794}
Committed:
https://chromium.googlesource.com/external/webrtc/+/1e46434e7ef90da1aebaf28cb...
==========

[commit-bot: I haz the power, 2017-02-23 10:49:20]

Committed patchset #5 (id:80001) as
https://chromium.googlesource.com/external/webrtc/+/1e46434e7ef90da1aebaf28cb...