Use initial bitrates for software VP8.

Use initial bitrates for software VP8.

Makes the software encoder start at VGA as well, since ~300k isn't good
enough to produce a good HD stream.

BUG=webrtc:5678
R=glaznev@webrtc.org, stefan@webrtc.org

Committed: https://crrev.com/e1da27e543bdb1983638118172a4efd599ca51b5
Cr-Commit-Position: refs/heads/master@{#12428}

[pbos-webrtc, 2016-04-18 11:40:12]

PTAL, starting at VGA also reduces blocky low-framerate start conditions.

[AlexG, 2016-04-18 22:39:00]

Description was changed from

==========
Use initial bitrates for software VP8.

Makes the software encoder start at VGA as well, since ~300k isn't good
enough to produce a good HD stream.

BUG=webrtc:5678
R=stefan@webrtc.org
==========

to

==========
Use initial bitrates for software VP8.

Makes the software encoder start at VGA as well, since ~300k isn't good
enough to produce a good HD stream.

BUG=webrtc:5678
R=stefan@webrtc.org
==========

[AlexG, 2016-04-18 22:39:00]

glaznev@webrtc.org changed reviewers:
+ glaznev@webrtc.org

[AlexG, 2016-04-18 22:39:12]

lgtm

[pbos-webrtc, 2016-04-18 22:39:39]

Description was changed from

==========
Use initial bitrates for software VP8.

Makes the software encoder start at VGA as well, since ~300k isn't good
enough to produce a good HD stream.

BUG=webrtc:5678
R=stefan@webrtc.org
==========

to

==========
Use initial bitrates for software VP8.

Makes the software encoder start at VGA as well, since ~300k isn't good
enough to produce a good HD stream.

BUG=webrtc:5678
R=glaznev@webrtc.org
==========

[pbos-webrtc, 2016-04-18 22:56:11]

fix test codec bitrates to not hit initial downscale

[pbos-webrtc, 2016-04-18 22:58:59]

The CQ bit was checked by pbos@webrtc.org

[pbos-webrtc, 2016-04-18 22:58:59]

The patchset sent to the CQ was uploaded after l-g-t-m from glaznev@webrtc.org
Link to the patchset: https://codereview.webrtc.org/1893313002/#ps20001 (title:
"fix test codec bitrates to not hit initial downscale")

[commit-bot: I haz the power, 2016-04-18 22:59:09]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1893313002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1893313002/20001

[commit-bot: I haz the power, 2016-04-19 00:59:39]

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

[commit-bot: I haz the power, 2016-04-19 00:59:40]

Try jobs failed on following builders:
  android_arm64_rel on tryserver.webrtc (JOB_TIMED_OUT, no build URL)

[stefan-webrtc, 2016-04-19 06:46:37]

Have you looked at how this affects appr.tc on a desktop? Just to make sure it
doesn't worsen the quality when probing actually succeeds?

https://codereview.webrtc.org/1893313002/diff/20001/webrtc/media/engine/webrt...
File webrtc/media/engine/webrtcvideoengine2_unittest.cc (right):

https://codereview.webrtc.org/1893313002/diff/20001/webrtc/media/engine/webrt...
webrtc/media/engine/webrtcvideoengine2_unittest.cc:872:
Base::TwoStreamsSendAndReceive(codec);
Shouldn't we instead change TwoStreamsSendAndReceive to verify QVGA?

[pbos-webrtc, 2016-04-19 13:50:28]

verify that two streams start sending QVGA

[pbos-webrtc, 2016-04-19 13:50:52]

desktop appr.tc looks fine, and is up to HD in ~2 seconds in loopback

[pbos-webrtc, 2016-04-19 13:51:50]

Description was changed from

==========
Use initial bitrates for software VP8.

Makes the software encoder start at VGA as well, since ~300k isn't good
enough to produce a good HD stream.

BUG=webrtc:5678
R=glaznev@webrtc.org
==========

to

==========
Use initial bitrates for software VP8.

Makes the software encoder start at VGA as well, since ~300k isn't good
enough to produce a good HD stream.

BUG=webrtc:5678
R=glaznev@webrtc.org, stefan@webrtc.org
==========

[stefan-webrtc, 2016-04-19 13:52:14]

lgtm

[pbos-webrtc, 2016-04-19 13:53:25]

Description was changed from

==========
Use initial bitrates for software VP8.

Makes the software encoder start at VGA as well, since ~300k isn't good
enough to produce a good HD stream.

BUG=webrtc:5678
R=glaznev@webrtc.org, stefan@webrtc.org
==========

to

==========
Use initial bitrates for software VP8.

Makes the software encoder start at VGA as well, since ~300k isn't good
enough to produce a good HD stream.

BUG=webrtc:5678
R=glaznev@webrtc.org, stefan@webrtc.org

Committed:
https://chromium.googlesource.com/external/webrtc/+/e1da27e543bdb198363811817...
==========

[commit-bot: I haz the power, 2016-04-19 13:53:26]

Description was changed from

==========
Use initial bitrates for software VP8.

Makes the software encoder start at VGA as well, since ~300k isn't good
enough to produce a good HD stream.

BUG=webrtc:5678
R=glaznev@webrtc.org, stefan@webrtc.org

Committed:
https://chromium.googlesource.com/external/webrtc/+/e1da27e543bdb198363811817...
==========

to

==========
Use initial bitrates for software VP8.

Makes the software encoder start at VGA as well, since ~300k isn't good
enough to produce a good HD stream.

BUG=webrtc:5678
R=glaznev@webrtc.org, stefan@webrtc.org

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

[pbos-webrtc, 2016-04-19 13:53:27]

Committed patchset #3 (id:40001) manually as
e1da27e543bdb1983638118172a4efd599ca51b5 (presubmit successful).

[commit-bot: I haz the power, 2016-04-19 13:53:27]

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

[kjellander_webrtc, 2016-04-19 15:18:33]

A revert of this CL (patchset #3 id:40001) has been created in
https://codereview.webrtc.org/1898183002/ by kjellander@webrtc.org.

The reason for reverting is: Likely broke Chromium:
https://build.chromium.org/p/chromium.webrtc.fyi/builders/Linux%20Tester/buil...
https://build.chromium.org/p/chromium.webrtc.fyi/builders/Win10%20Tester/buil....