Revert of Initialize/configure video encoders asychronously.

Revert of Initialize/configure video encoders asychronously. (patchset #4 id:60001 of https://codereview.webrtc.org/1757313002/ )

Reason for revert:
Breaks RTCVideoEncoder which has incorrect assumptions on where InitEncode etc. is called from. Temporarily reverting until RTCVideoEncoder has been updated.

Original issue's description:
> Initialize/configure video encoders asychronously.
>
> Greatly speeds up setRemoteDescription() by moving encoder initialization
> off the main worker thread, which is free to move onto gathering ICE
> candidates and other tasks while InitEncode() is performed. It also
> un-blocks PeerConnection GetStats() which is no longer blocked on
> encoder initialization.
>
> BUG=webrtc:5410
> R=stefan@webrtc.org
>
> Committed: https://chromium.googlesource.com/external/webrtc/+/fb647a67be94bb3c940d8b5fba01972f7ce91a29

R=stefan@webrtc.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=chromium:595274, chromium:595308, webrtc:5410
Committed: https://crrev.com/81cbd924447d507559dbd6e6d1f9fe439fcf2716
Cr-Commit-Position: refs/heads/master@{#12086}

[pbos-webrtc, 2016-03-22 10:25:14]

Created Revert of Initialize/configure video encoders asychronously.

[stefan-webrtc, 2016-03-22 10:26:53]

lgtm

[pbos-webrtc, 2016-03-22 10:29:11]

Description was changed from

==========
Revert of Initialize/configure video encoders asychronously. (patchset #4
id:60001 of https://codereview.webrtc.org/1757313002/ )

Reason for revert:
Breaks RTCVideoEncoder which has incorrect assumptions on where InitEncode etc.
is called from. Temporarily reverting until RTCVideoEncoder has been updated.

Original issue's description:
> Initialize/configure video encoders asychronously.
> 
> Greatly speeds up setRemoteDescription() by moving encoder initialization
> off the main worker thread, which is free to move onto gathering ICE
> candidates and other tasks while InitEncode() is performed. It also
> un-blocks PeerConnection GetStats() which is no longer blocked on
> encoder initialization.
> 
> BUG=webrtc:5410
> R=stefan@webrtc.org
> 
> Committed:
https://chromium.googlesource.com/external/webrtc/+/fb647a67be94bb3c940d8b5fb...

TBR=stefan@webrtc.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=webrtc:5410
==========

to

==========
Revert of Initialize/configure video encoders asychronously. (patchset #4
id:60001 of https://codereview.webrtc.org/1757313002/ )

Reason for revert:
Breaks RTCVideoEncoder which has incorrect assumptions on where InitEncode etc.
is called from. Temporarily reverting until RTCVideoEncoder has been updated.

Original issue's description:
> Initialize/configure video encoders asychronously.
> 
> Greatly speeds up setRemoteDescription() by moving encoder initialization
> off the main worker thread, which is free to move onto gathering ICE
> candidates and other tasks while InitEncode() is performed. It also
> un-blocks PeerConnection GetStats() which is no longer blocked on
> encoder initialization.
> 
> BUG=webrtc:5410
> R=stefan@webrtc.org
> 
> Committed:
https://chromium.googlesource.com/external/webrtc/+/fb647a67be94bb3c940d8b5fb...

TBR=stefan@webrtc.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=chromium:595274, chromium:595308, webrtc:5410
==========

[pbos-webrtc, 2016-03-22 10:29:45]

The CQ bit was checked by pbos@webrtc.org

[commit-bot: I haz the power, 2016-03-22 10:29:51]

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

[pbos-webrtc, 2016-03-22 10:31:16]

Description was changed from

==========
Revert of Initialize/configure video encoders asychronously. (patchset #4
id:60001 of https://codereview.webrtc.org/1757313002/ )

Reason for revert:
Breaks RTCVideoEncoder which has incorrect assumptions on where InitEncode etc.
is called from. Temporarily reverting until RTCVideoEncoder has been updated.

Original issue's description:
> Initialize/configure video encoders asychronously.
> 
> Greatly speeds up setRemoteDescription() by moving encoder initialization
> off the main worker thread, which is free to move onto gathering ICE
> candidates and other tasks while InitEncode() is performed. It also
> un-blocks PeerConnection GetStats() which is no longer blocked on
> encoder initialization.
> 
> BUG=webrtc:5410
> R=stefan@webrtc.org
> 
> Committed:
https://chromium.googlesource.com/external/webrtc/+/fb647a67be94bb3c940d8b5fb...

TBR=stefan@webrtc.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=chromium:595274, chromium:595308, webrtc:5410
==========

to

==========
Revert of Initialize/configure video encoders asychronously. (patchset #4
id:60001 of https://codereview.webrtc.org/1757313002/ )

Reason for revert:
Breaks RTCVideoEncoder which has incorrect assumptions on where InitEncode etc.
is called from. Temporarily reverting until RTCVideoEncoder has been updated.

Original issue's description:
> Initialize/configure video encoders asychronously.
> 
> Greatly speeds up setRemoteDescription() by moving encoder initialization
> off the main worker thread, which is free to move onto gathering ICE
> candidates and other tasks while InitEncode() is performed. It also
> un-blocks PeerConnection GetStats() which is no longer blocked on
> encoder initialization.
> 
> BUG=webrtc:5410
> R=stefan@webrtc.org
> 
> Committed:
https://chromium.googlesource.com/external/webrtc/+/fb647a67be94bb3c940d8b5fb...

R=stefan@webrtc.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=chromium:595274, chromium:595308, webrtc:5410
==========

[pbos-webrtc, 2016-03-22 11:19:19]

Description was changed from

==========
Revert of Initialize/configure video encoders asychronously. (patchset #4
id:60001 of https://codereview.webrtc.org/1757313002/ )

Reason for revert:
Breaks RTCVideoEncoder which has incorrect assumptions on where InitEncode etc.
is called from. Temporarily reverting until RTCVideoEncoder has been updated.

Original issue's description:
> Initialize/configure video encoders asychronously.
> 
> Greatly speeds up setRemoteDescription() by moving encoder initialization
> off the main worker thread, which is free to move onto gathering ICE
> candidates and other tasks while InitEncode() is performed. It also
> un-blocks PeerConnection GetStats() which is no longer blocked on
> encoder initialization.
> 
> BUG=webrtc:5410
> R=stefan@webrtc.org
> 
> Committed:
https://chromium.googlesource.com/external/webrtc/+/fb647a67be94bb3c940d8b5fb...

R=stefan@webrtc.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=chromium:595274, chromium:595308, webrtc:5410
==========

to

==========
Revert of Initialize/configure video encoders asychronously. (patchset #4
id:60001 of https://codereview.webrtc.org/1757313002/ )

Reason for revert:
Breaks RTCVideoEncoder which has incorrect assumptions on where InitEncode etc.
is called from. Temporarily reverting until RTCVideoEncoder has been updated.

Original issue's description:
> Initialize/configure video encoders asychronously.
>
> Greatly speeds up setRemoteDescription() by moving encoder initialization
> off the main worker thread, which is free to move onto gathering ICE
> candidates and other tasks while InitEncode() is performed. It also
> un-blocks PeerConnection GetStats() which is no longer blocked on
> encoder initialization.
>
> BUG=webrtc:5410
> R=stefan@webrtc.org
>
> Committed:
https://chromium.googlesource.com/external/webrtc/+/fb647a67be94bb3c940d8b5fb...

R=stefan@webrtc.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=chromium:595274, chromium:595308, webrtc:5410

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

[pbos-webrtc, 2016-03-22 11:19:21]

Committed patchset #1 (id:1) manually as
81cbd924447d507559dbd6e6d1f9fe439fcf2716 (presubmit successful).

[commit-bot: I haz the power, 2016-03-22 11:19:24]

Description was changed from

==========
Revert of Initialize/configure video encoders asychronously. (patchset #4
id:60001 of https://codereview.webrtc.org/1757313002/ )

Reason for revert:
Breaks RTCVideoEncoder which has incorrect assumptions on where InitEncode etc.
is called from. Temporarily reverting until RTCVideoEncoder has been updated.

Original issue's description:
> Initialize/configure video encoders asychronously.
>
> Greatly speeds up setRemoteDescription() by moving encoder initialization
> off the main worker thread, which is free to move onto gathering ICE
> candidates and other tasks while InitEncode() is performed. It also
> un-blocks PeerConnection GetStats() which is no longer blocked on
> encoder initialization.
>
> BUG=webrtc:5410
> R=stefan@webrtc.org
>
> Committed:
https://chromium.googlesource.com/external/webrtc/+/fb647a67be94bb3c940d8b5fb...

R=stefan@webrtc.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=chromium:595274, chromium:595308, webrtc:5410

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

to

==========
Revert of Initialize/configure video encoders asychronously. (patchset #4
id:60001 of https://codereview.webrtc.org/1757313002/ )

Reason for revert:
Breaks RTCVideoEncoder which has incorrect assumptions on where InitEncode etc.
is called from. Temporarily reverting until RTCVideoEncoder has been updated.

Original issue's description:
> Initialize/configure video encoders asychronously.
>
> Greatly speeds up setRemoteDescription() by moving encoder initialization
> off the main worker thread, which is free to move onto gathering ICE
> candidates and other tasks while InitEncode() is performed. It also
> un-blocks PeerConnection GetStats() which is no longer blocked on
> encoder initialization.
>
> BUG=webrtc:5410
> R=stefan@webrtc.org
>
> Committed:
https://chromium.googlesource.com/external/webrtc/+/fb647a67be94bb3c940d8b5fb...

R=stefan@webrtc.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=chromium:595274, chromium:595308, webrtc:5410

Committed: https://crrev.com/81cbd924447d507559dbd6e6d1f9fe439fcf2716
Cr-Commit-Position: refs/heads/master@{#12086}
==========

[commit-bot: I haz the power, 2016-03-22 11:19:26]

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

[pbos-webrtc, 2016-04-20 09:45:30]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.webrtc.org/1896413002/ by pbos@webrtc.org.

The reason for reverting is: RTCVideoEncoder has been updated to not make
assumptions on calling threads/post back to a worker thread. This should now be
landable again..