Add an HD resolution perf test.

Add an HD resolution perf test.

Also update existing perf tests to use send side bwe.

BUG=webrtc:4604, chromium:522001
Committed: https://crrev.com/b17976763d58e9fdf5b25747ec45f86344225627
Cr-Commit-Position: refs/heads/master@{#13726}

[stefan-webrtc, 2016-08-09 13:54:16]

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

[stefan-webrtc, 2016-08-09 13:54:34]

Description was changed from

==========
Add an HD resolution perf test.

BUG=webrtc:4604
==========

to

==========
Add an HD resolution perf test.

Also update existing perf tests to use send side bwe.

BUG=webrtc:4604
==========

[stefan-webrtc, 2016-08-09 14:07:10]

Description was changed from

==========
Add an HD resolution perf test.

Also update existing perf tests to use send side bwe.

BUG=webrtc:4604
==========

to

==========
Add an HD resolution perf test.

Also update existing perf tests to use send side bwe.

BUG=webrtc:4604, chromium:522001
==========

[sprang_webrtc, 2016-08-09 14:19:00]

lgtm

[stefan-webrtc, 2016-08-09 14:23:32]

The CQ bit was checked by stefan@webrtc.org

[commit-bot: I haz the power, 2016-08-09 14:23:45]

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-08-09 14:44:21]

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

[commit-bot: I haz the power, 2016-08-09 14:44:22]

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

[stefan-webrtc, 2016-08-10 08:57:23]

stefan@webrtc.org changed reviewers:
+ mflodman@webrtc.org

[stefan-webrtc, 2016-08-10 08:57:45]

Magnus, could you approve the change to webrtc_perf_tests.isolate?

[sprang_webrtc, 2016-08-11 10:43:50]

https://codereview.webrtc.org/2227733004/diff/40001/webrtc/video/video_qualit...
File webrtc/video/video_quality_test.cc (right):

https://codereview.webrtc.org/2227733004/diff/40001/webrtc/video/video_qualit...
webrtc/video/video_quality_test.cc:417: }
else {
  reference_copy.set_ntp_time_ms(reference.ntp_time_ms());
  render_copy.set_ntp_time_ms(render.ntp_time_ms());
}


Otherwise some time deltas will be bogus.

[stefan-webrtc, 2016-08-11 10:49:23]

Add a new test which still runs with recv-side bwe.

[stefan-webrtc, 2016-08-11 10:49:47]

PTAL

https://codereview.webrtc.org/2227733004/diff/40001/webrtc/video/video_qualit...
File webrtc/video/video_quality_test.cc (right):

https://codereview.webrtc.org/2227733004/diff/40001/webrtc/video/video_qualit...
webrtc/video/video_quality_test.cc:417: }
On 2016/08/11 10:43:50, språng wrote:
> else {
>   reference_copy.set_ntp_time_ms(reference.ntp_time_ms());
>   render_copy.set_ntp_time_ms(render.ntp_time_ms());
> }
> 
> 
> Otherwise some time deltas will be bogus.

Done.

[mflodman, 2016-08-11 11:29:14]

isolate LGTM

[sprang_webrtc, 2016-08-11 11:32:47]

lgtm

[sprang_webrtc, 2016-08-11 11:33:00]

The CQ bit was checked by sprang@webrtc.org

[commit-bot: I haz the power, 2016-08-11 11:33:08]

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-08-11 14:00:59]

Description was changed from

==========
Add an HD resolution perf test.

Also update existing perf tests to use send side bwe.

BUG=webrtc:4604, chromium:522001
==========

to

==========
Add an HD resolution perf test.

Also update existing perf tests to use send side bwe.

BUG=webrtc:4604, chromium:522001
==========

[commit-bot: I haz the power, 2016-08-11 14:01:00]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-08-11 14:01:10]

Description was changed from

==========
Add an HD resolution perf test.

Also update existing perf tests to use send side bwe.

BUG=webrtc:4604, chromium:522001
==========

to

==========
Add an HD resolution perf test.

Also update existing perf tests to use send side bwe.

BUG=webrtc:4604, chromium:522001

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

[commit-bot: I haz the power, 2016-08-11 14:01:11]

Patchset 4 (id:??) landed as
https://crrev.com/b17976763d58e9fdf5b25747ec45f86344225627
Cr-Commit-Position: refs/heads/master@{#13726}