Reland of Periodically update codec bit/frame rate settings.

Reland of Periodically update codec bit/frame rate settings.

Patch set 1 is a reland + trivial rebase.
Patch set >= 2 contains bug fixes.

> Original issue's description:
> > Fix bug in vie_encoder.cc which caused channel parameters not to be updated at regular intervals, as it was intended.
> >
> > That however exposes a bunch of failed test, so this CL also fixed a few other things:
> > * FakeEncoder should trust the configured FPS value rather than guesstimating itself based on the realtime clock, so as not to completely undershoot targets in offline mode. Also, compensate for key-frame overshoots when outputting delta frames.
> > * FrameDropper should not assuming incoming frame rate is 0 if no frames have been seen.
> > * Fix a bunch of test cases that started failing because they were relying on the fake encoder undershooting.
> > * Fix test
> >
> > BUG=7664
> >
> > Review-Url: https://codereview.webrtc.org/2883963002
> > Cr-Commit-Position: refs/heads/master@{#18473}
> > Committed: https://chromium.googlesource.com/external/webrtc/+/6431e21da672a5f3bbf166d3d4d98b171d015706

BUG=webrtc:7664
Review-Url: https://codereview.webrtc.org/2953053002
Cr-Commit-Position: refs/heads/master@{#18782}
Committed: https://chromium.googlesource.com/external/webrtc/+/4847ae6b51ea55b9db9bf16c7b97c3cb8d000f23

[sprang_webrtc, 2017-06-22 15:17:10]

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

[commit-bot: I haz the power, 2017-06-22 15:17:20]

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-06-22 15:35:29]

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

[commit-bot: I haz the power, 2017-06-22 15:35:30]

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

[sprang_webrtc, 2017-06-26 14:52:18]

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

[commit-bot: I haz the power, 2017-06-26 14:52:21]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...

[sprang_webrtc, 2017-06-26 14:54:23]

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

[commit-bot: I haz the power, 2017-06-26 15:26:46]

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

[commit-bot: I haz the power, 2017-06-26 15:26:47]

Dry run: This issue passed the CQ dry run.

[stefan-webrtc, 2017-06-27 07:33:15]

lgtm

[sprang_webrtc, 2017-06-27 14:04:41]

The CQ bit was checked by sprang@webrtc.org

[commit-bot: I haz the power, 2017-06-27 14:04:54]

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-06-27 14:06:53]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1498572281967620,
"parent_rev": "f0a6fb19c6e8a7fc7c3da6be7b268bb376d33701", "commit_rev":
"4847ae6b51ea55b9db9bf16c7b97c3cb8d000f23"}

[commit-bot: I haz the power, 2017-06-27 14:07:13]

Description was changed from

==========
Reland of Periodically update codec bit/frame rate settings.

Patch set 1 is a reland + trivial rebase.
Patch set >= 2 contains bug fixes.

> Original issue's description:
> > Fix bug in vie_encoder.cc which caused channel parameters not to be updated
at regular intervals, as it was intended.
> >
> > That however exposes a bunch of failed test, so this CL also fixed a few
other things:
> > * FakeEncoder should trust the configured FPS value rather than
guesstimating itself based on the realtime clock, so as not to completely
undershoot targets in offline mode. Also, compensate for key-frame overshoots
when outputting delta frames.
> > * FrameDropper should not assuming incoming frame rate is 0 if no frames
have been seen.
> > * Fix a bunch of test cases that started failing because they were relying
on the fake encoder undershooting.
> > * Fix test
> >
> > BUG=7664
> >
> > Review-Url: https://codereview.webrtc.org/2883963002
> > Cr-Commit-Position: refs/heads/master@{#18473}
> > Committed:
https://chromium.googlesource.com/external/webrtc/+/6431e21da672a5f3bbf166d3d...

BUG=webrtc:7664
==========

to

==========
Reland of Periodically update codec bit/frame rate settings.

Patch set 1 is a reland + trivial rebase.
Patch set >= 2 contains bug fixes.

> Original issue's description:
> > Fix bug in vie_encoder.cc which caused channel parameters not to be updated
at regular intervals, as it was intended.
> >
> > That however exposes a bunch of failed test, so this CL also fixed a few
other things:
> > * FakeEncoder should trust the configured FPS value rather than
guesstimating itself based on the realtime clock, so as not to completely
undershoot targets in offline mode. Also, compensate for key-frame overshoots
when outputting delta frames.
> > * FrameDropper should not assuming incoming frame rate is 0 if no frames
have been seen.
> > * Fix a bunch of test cases that started failing because they were relying
on the fake encoder undershooting.
> > * Fix test
> >
> > BUG=7664
> >
> > Review-Url: https://codereview.webrtc.org/2883963002
> > Cr-Commit-Position: refs/heads/master@{#18473}
> > Committed:
https://chromium.googlesource.com/external/webrtc/+/6431e21da672a5f3bbf166d3d...

BUG=webrtc:7664

Review-Url: https://codereview.webrtc.org/2953053002
Cr-Commit-Position: refs/heads/master@{#18782}
Committed:
https://chromium.googlesource.com/external/webrtc/+/4847ae6b51ea55b9db9bf16c7...
==========

[commit-bot: I haz the power, 2017-06-27 14:07:14]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/external/webrtc/+/4847ae6b51ea55b9db9bf16c7...