Clean up Stats class used in VideoProcessor tests.

Clean up Stats class used in VideoProcessor tests.

* Reorder and rename members of FrameStatistic.
* Rename corresponding temp variables used in the summary calculations.
* Make Stats class less structy.
* Try to enforce consistent frame numbers at all times.
* Remove VideoProcessor::FrameInfo struct and use the Stats class
  for keeping track of encode/decode start times, etc.

BUG=webrtc:6634
Review-Url: https://codereview.webrtc.org/3011923002
Cr-Commit-Position: refs/heads/master@{#19703}
Committed: https://chromium.googlesource.com/external/webrtc/+/8935d97f169d0c85ea04d2f370964f283783cd79

[brandtr, 2017-09-05 08:05:12]

Description was changed from

==========
Clean up Stats class used in VideoProcessor tests.

* Reorder and rename members of FrameStatistic.
* Rename corresponding temp variables used in the summary calculations.
* Make Stats less structy.
* Try to enforce consistent frame numbers at all times.
* Remove VideoProcessor::FrameInfo struct, and use the Stats class
  for keeping track of encode/decode start times, etc.

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

to

==========
Clean up Stats class used in VideoProcessor tests.

* Reorder and rename members of FrameStatistic.
* Rename corresponding temp variables used in the summary calculations.
* Make Stats class less structy.
* Try to enforce consistent frame numbers at all times.
* Remove VideoProcessor::FrameInfo struct and use the Stats class
  for keeping track of encode/decode start times, etc.

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

[brandtr, 2017-09-05 08:05:30]

Patchset #1 (id:1) has been deleted

[brandtr, 2017-09-05 08:12:15]

Patchset #1 (id:20001) has been deleted

[brandtr, 2017-09-05 08:17:59]

Patchset #1 (id:40001) has been deleted

[brandtr, 2017-09-05 08:18:31]

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

[brandtr, 2017-09-05 08:18:32]

Please take a look :)

[srte, 2017-09-05 11:34:16]

lgtm

[srte, 2017-09-05 11:34:16]

lgtm

[brandtr, 2017-09-05 13:35:03]

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

[commit-bot: I haz the power, 2017-09-05 13:35:09]

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-09-05 13:43:18]

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

[commit-bot: I haz the power, 2017-09-05 13:43:20]

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

[brandtr, 2017-09-06 07:49:34]

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

[commit-bot: I haz the power, 2017-09-06 07:49:39]

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-09-06 08:44:29]

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

[commit-bot: I haz the power, 2017-09-06 08:44:30]

Dry run: This issue passed the CQ dry run.

[brandtr, 2017-09-06 08:50:37]

The CQ bit was checked by brandtr@webrtc.org

[brandtr, 2017-09-06 08:50:38]

The patchset sent to the CQ was uploaded after l-g-t-m from srte@webrtc.org
Link to the patchset: https://codereview.webrtc.org/3011923002/#ps80001 (title:
"Fix VS compile.")

[commit-bot: I haz the power, 2017-09-06 08:50:41]

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-09-06 08:53:23]

CQ is committing da patch.

Bot data: {"patchset_id": 80001, "attempt_start_ts": 1504687837357240,
"parent_rev": "42a70e31d6b08355e5803f66433b043d5825b56f", "commit_rev":
"8935d97f169d0c85ea04d2f370964f283783cd79"}

[commit-bot: I haz the power, 2017-09-06 08:53:28]

Description was changed from

==========
Clean up Stats class used in VideoProcessor tests.

* Reorder and rename members of FrameStatistic.
* Rename corresponding temp variables used in the summary calculations.
* Make Stats class less structy.
* Try to enforce consistent frame numbers at all times.
* Remove VideoProcessor::FrameInfo struct and use the Stats class
  for keeping track of encode/decode start times, etc.

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

to

==========
Clean up Stats class used in VideoProcessor tests.

* Reorder and rename members of FrameStatistic.
* Rename corresponding temp variables used in the summary calculations.
* Make Stats class less structy.
* Try to enforce consistent frame numbers at all times.
* Remove VideoProcessor::FrameInfo struct and use the Stats class
  for keeping track of encode/decode start times, etc.

BUG=webrtc:6634

Review-Url: https://codereview.webrtc.org/3011923002
Cr-Commit-Position: refs/heads/master@{#19703}
Committed:
https://chromium.googlesource.com/external/webrtc/+/8935d97f169d0c85ea04d2f37...
==========

[commit-bot: I haz the power, 2017-09-06 08:53:29]

Committed patchset #2 (id:80001) as
https://chromium.googlesource.com/external/webrtc/+/8935d97f169d0c85ea04d2f37...