Revert of Let ViEEncoder handle resolution changes.

Revert of Let ViEEncoder handle resolution changes. (patchset #17 id:340001 of https://codereview.webrtc.org/2351633002/ )

Reason for revert:
Fails on a content_browsertest (and also webrtc_perf?)

https://build.chromium.org/p/chromium.webrtc.fyi/builders/Mac%20Tester/builds/34336

https://build.chromium.org/p/client.webrtc/builders/Linux64%20Release%20%5Blarge%20tests%5D/builds/9091/steps/webrtc_perf_tests/logs/stdio
[  FAILED  ] FullStackTest.ParisQcifWithoutPacketLoss (59436 ms)

Original issue's description:
> Let ViEEncoder handle resolution changes.
>
> This cl move codec reconfiguration due to video frame size changes from WebRtcVideoSendStream to ViEEncoder.
>
> With this change, many variables in WebRtcVideoSendStream no longer need to be locked.
>
> BUG=webrtc:5687, webrtc:6371, webrtc:5332
>
> Committed: https://crrev.com/26105b41b4f97642ee30cb067dc786c2737709ad
> Cr-Commit-Position: refs/heads/master@{#14445}

TBR=sprang@webrtc.org,mflodman@webrtc.org,stefan@webrtc.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=webrtc:5687, webrtc:6371, webrtc:5332
Committed: https://crrev.com/3b703ede8b84267a4ff216aec9296e5dcdd7b6b7
Cr-Commit-Position: refs/heads/master@{#14447}

[perkj_webrtc, 2016-09-30 06:25:25]

The CQ bit was checked by perkj@webrtc.org

[perkj_webrtc, 2016-09-30 06:25:25]

Created Revert of Let ViEEncoder handle resolution changes.

[commit-bot: I haz the power, 2016-09-30 06:25:31]

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-09-30 06:25:43]

Description was changed from

==========
Revert of Let ViEEncoder handle resolution changes. (patchset #17 id:340001 of
https://codereview.webrtc.org/2351633002/ )

Reason for revert:
Fails on a content_browsertest (and also webrtc_perf?)

https://build.chromium.org/p/chromium.webrtc.fyi/builders/Mac%20Tester/builds...

https://build.chromium.org/p/client.webrtc/builders/Linux64%20Release%20%5Bla...
[  FAILED  ] FullStackTest.ParisQcifWithoutPacketLoss (59436 ms)

Original issue's description:
> Let ViEEncoder handle resolution changes.
> 
> This cl move codec reconfiguration due to video frame size changes from
WebRtcVideoSendStream to ViEEncoder.
> 
> With this change, many variables in WebRtcVideoSendStream no longer need to be
locked.
> 
> BUG=webrtc:5687, webrtc:6371, webrtc:5332
> 
> Committed: https://crrev.com/26105b41b4f97642ee30cb067dc786c2737709ad
> Cr-Commit-Position: refs/heads/master@{#14445}

TBR=sprang@webrtc.org,mflodman@webrtc.org,stefan@webrtc.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=webrtc:5687, webrtc:6371, webrtc:5332
==========

to

==========
Revert of Let ViEEncoder handle resolution changes. (patchset #17 id:340001 of
https://codereview.webrtc.org/2351633002/ )

Reason for revert:
Fails on a content_browsertest (and also webrtc_perf?)

https://build.chromium.org/p/chromium.webrtc.fyi/builders/Mac%20Tester/builds...

https://build.chromium.org/p/client.webrtc/builders/Linux64%20Release%20%5Bla...
[  FAILED  ] FullStackTest.ParisQcifWithoutPacketLoss (59436 ms)

Original issue's description:
> Let ViEEncoder handle resolution changes.
> 
> This cl move codec reconfiguration due to video frame size changes from
WebRtcVideoSendStream to ViEEncoder.
> 
> With this change, many variables in WebRtcVideoSendStream no longer need to be
locked.
> 
> BUG=webrtc:5687, webrtc:6371, webrtc:5332
> 
> Committed: https://crrev.com/26105b41b4f97642ee30cb067dc786c2737709ad
> Cr-Commit-Position: refs/heads/master@{#14445}

TBR=sprang@webrtc.org,mflodman@webrtc.org,stefan@webrtc.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=webrtc:5687, webrtc:6371, webrtc:5332
==========

[commit-bot: I haz the power, 2016-09-30 06:25:44]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-09-30 06:25:53]

Description was changed from

==========
Revert of Let ViEEncoder handle resolution changes. (patchset #17 id:340001 of
https://codereview.webrtc.org/2351633002/ )

Reason for revert:
Fails on a content_browsertest (and also webrtc_perf?)

https://build.chromium.org/p/chromium.webrtc.fyi/builders/Mac%20Tester/builds...

https://build.chromium.org/p/client.webrtc/builders/Linux64%20Release%20%5Bla...
[  FAILED  ] FullStackTest.ParisQcifWithoutPacketLoss (59436 ms)

Original issue's description:
> Let ViEEncoder handle resolution changes.
> 
> This cl move codec reconfiguration due to video frame size changes from
WebRtcVideoSendStream to ViEEncoder.
> 
> With this change, many variables in WebRtcVideoSendStream no longer need to be
locked.
> 
> BUG=webrtc:5687, webrtc:6371, webrtc:5332
> 
> Committed: https://crrev.com/26105b41b4f97642ee30cb067dc786c2737709ad
> Cr-Commit-Position: refs/heads/master@{#14445}

TBR=sprang@webrtc.org,mflodman@webrtc.org,stefan@webrtc.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=webrtc:5687, webrtc:6371, webrtc:5332
==========

to

==========
Revert of Let ViEEncoder handle resolution changes. (patchset #17 id:340001 of
https://codereview.webrtc.org/2351633002/ )

Reason for revert:
Fails on a content_browsertest (and also webrtc_perf?)

https://build.chromium.org/p/chromium.webrtc.fyi/builders/Mac%20Tester/builds...

https://build.chromium.org/p/client.webrtc/builders/Linux64%20Release%20%5Bla...
[  FAILED  ] FullStackTest.ParisQcifWithoutPacketLoss (59436 ms)

Original issue's description:
> Let ViEEncoder handle resolution changes.
>
> This cl move codec reconfiguration due to video frame size changes from
WebRtcVideoSendStream to ViEEncoder.
>
> With this change, many variables in WebRtcVideoSendStream no longer need to be
locked.
>
> BUG=webrtc:5687, webrtc:6371, webrtc:5332
>
> Committed: https://crrev.com/26105b41b4f97642ee30cb067dc786c2737709ad
> Cr-Commit-Position: refs/heads/master@{#14445}

TBR=sprang@webrtc.org,mflodman@webrtc.org,stefan@webrtc.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=webrtc:5687, webrtc:6371, webrtc:5332

Committed: https://crrev.com/3b703ede8b84267a4ff216aec9296e5dcdd7b6b7
Cr-Commit-Position: refs/heads/master@{#14447}
==========

[commit-bot: I haz the power, 2016-09-30 06:25:54]

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