Don't reconfigure the encoder if the video options aren't changing.

Don't reconfigure the encoder if the video options aren't changing.

Committed: https://crrev.com/119760aa65cafca0d6178be954b56f4671d0013a
Cr-Commit-Position: refs/heads/master@{#12222}

[Taylor Brandstetter, 2016-03-30 18:24:47]

deadbeef@webrtc.org changed reviewers:
+ pbos@webrtc.org

[Taylor Brandstetter, 2016-03-30 18:24:48]

This is pulled out of my larger CL. I know it might conflict slightly with
nisse@'s CL, but it should be much easier to resolve the conflicts with this one
(I think).

[nisse-webrtc, 2016-03-31 06:23:13]

On 2016/03/30 18:24:48, Taylor Brandstetter wrote:
> This is pulled out of my larger CL. I know it might conflict slightly with
> nisse@'s CL, but it should be much easier to resolve the conflicts with this
one
> (I think).

lgtm. Shouldn't cause any trouble for the ongoing refactoring.

Then I should add that I'd really like to get rid of those optionals... In the
comparison of old and new options, one ought to take into account what the
options will be after any defaults for unset options have been applied (which is
trickier than it sounds, since the denoising default is codec-dependent...). But
that's for later.

[pbos-webrtc, 2016-04-04 12:05:31]

lgtm

[Taylor Brandstetter, 2016-04-04 17:46:01]

The CQ bit was checked by deadbeef@webrtc.org

[commit-bot: I haz the power, 2016-04-04 17:46:07]

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

[commit-bot: I haz the power, 2016-04-04 18:43:31]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-04-04 18:43:38]

Description was changed from

==========
Don't reconfigure the encoder if the video options aren't changing.
==========

to

==========
Don't reconfigure the encoder if the video options aren't changing.

Committed: https://crrev.com/119760aa65cafca0d6178be954b56f4671d0013a
Cr-Commit-Position: refs/heads/master@{#12222}
==========

[commit-bot: I haz the power, 2016-04-04 18:43:39]

Patchset 2 (id:??) landed as
https://crrev.com/119760aa65cafca0d6178be954b56f4671d0013a
Cr-Commit-Position: refs/heads/master@{#12222}