Style fixes: VideoProcessor and corresponding integration test.

Style fixes: VideoProcessor and corresponding integration test.

This CL has no intended functional changes.

BUG=webrtc:6634
Review-Url: https://codereview.webrtc.org/2697583002
Cr-Commit-Position: refs/heads/master@{#16628}
Committed: https://chromium.googlesource.com/external/webrtc/+/8bc9385fcbeea589cbac577d2c15e7e1632c57b8

[brandtr, 2017-02-13 13:30:45]

Patchset #1 (id:1) has been deleted

[brandtr, 2017-02-13 13:32:36]

Description was changed from

==========
Style fixes: VideoProcessor and corresponding integration test.

No functional changes are intended with this CL.

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

to

==========
Style fixes: VideoProcessor and corresponding integration test.

This CL is 95% style fixes, but has one intended functional change:
the encode time calculation is made tighter, but not not including
the I/O time for writing frames to disk.

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

[brandtr, 2017-02-13 15:39:51]

Description was changed from

==========
Style fixes: VideoProcessor and corresponding integration test.

This CL is 95% style fixes, but has one intended functional change:
the encode time calculation is made tighter, but not not including
the I/O time for writing frames to disk.

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

to

==========
Style fixes: VideoProcessor and corresponding integration test.

This CL has no intended functional changes.

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

[brandtr, 2017-02-13 15:39:56]

Patchset #1 (id:20001) has been deleted

[brandtr, 2017-02-13 16:25:13]

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

[åsapersson, 2017-02-15 08:51:35]

lgtm

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

https://codereview.webrtc.org/2697583002/diff/60001/webrtc/modules/video_codi...
webrtc/modules/video_coding/codecs/test/videoprocessor_integrationtest.h:196:
RTC_DCHECK(frame_reader_->Init());
maybe RTC_CHECK

https://codereview.webrtc.org/2697583002/diff/60001/webrtc/modules/video_codi...
webrtc/modules/video_coding/codecs/test/videoprocessor_integrationtest.h:204:
RTC_DCHECK(processor_->Init());
ditto

[brandtr, 2017-02-15 08:57:13]

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

https://codereview.webrtc.org/2697583002/diff/60001/webrtc/modules/video_codi...
webrtc/modules/video_coding/codecs/test/videoprocessor_integrationtest.h:196:
RTC_DCHECK(frame_reader_->Init());
On 2017/02/15 08:51:34, åsapersson wrote:
> maybe RTC_CHECK

Done.

https://codereview.webrtc.org/2697583002/diff/60001/webrtc/modules/video_codi...
webrtc/modules/video_coding/codecs/test/videoprocessor_integrationtest.h:204:
RTC_DCHECK(processor_->Init());
On 2017/02/15 08:51:34, åsapersson wrote:
> ditto

Done.

[sprang_webrtc, 2017-02-15 10:53:02]

lgtm

[brandtr, 2017-02-15 12:05:47]

The CQ bit was checked by brandtr@webrtc.org

[brandtr, 2017-02-15 12:05: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/2697583002/#ps80001 (title:
"asapersson comments 1.")

[commit-bot: I haz the power, 2017-02-15 12:05:52]

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-15 13:19:51]

CQ is committing da patch.

Bot data: {"patchset_id": 80001, "attempt_start_ts": 1487160347466200,
"parent_rev": "280eb224e2e0630ec94a6072777d58354357ee67", "commit_rev":
"8bc9385fcbeea589cbac577d2c15e7e1632c57b8"}

[commit-bot: I haz the power, 2017-02-15 13:19:55]

Description was changed from

==========
Style fixes: VideoProcessor and corresponding integration test.

This CL has no intended functional changes.

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

to

==========
Style fixes: VideoProcessor and corresponding integration test.

This CL has no intended functional changes.

BUG=webrtc:6634

Review-Url: https://codereview.webrtc.org/2697583002
Cr-Commit-Position: refs/heads/master@{#16628}
Committed:
https://chromium.googlesource.com/external/webrtc/+/8bc9385fcbeea589cbac577d2...
==========

[commit-bot: I haz the power, 2017-02-15 13:19:57]

Committed patchset #3 (id:80001) as
https://chromium.googlesource.com/external/webrtc/+/8bc9385fcbeea589cbac577d2...