Reland of use allocated encoders in SimulcastEncoderAdapter.

Reland of use allocated encoders in SimulcastEncoderAdapter. (patchset #1 id:1 of https://codereview.webrtc.org/2893003002/ )

Reason for reland:
Chrome encoder implementation fixed.

Original issue's description:
> Revert of Reuse allocated encoders in SimulcastEncoderAdapter. (patchset #15 id:320001 of https://codereview.webrtc.org/2830793005/ )
> 
> Reason for revert:
> Breaks Chrome tests.
> 
> Original issue's description:
> > Reuse allocated encoders in SimulcastEncoderAdapter.
> > 
> > Prior to this change, the SimulcastEncoderAdapter would destroy and create
> > encoders whenever it is being reinitialized. After this change, the
> > SimulcastEncoderAdapter will cache the already allocated encoders, and reuse
> > them after reinitialization.
> > 
> > This change will help in reducing the number of PictureID "jumps" that have
> > been seen around encoder reinitialization.
> > 
> > TESTED=AppRTCMobile, Chrome desktop, and internal app, with forced encoder reinits every 30 frames and https://codereview.webrtc.org/2833493003/ applied.
> > BUG=webrtc:7475
> > 
> > Review-Url: https://codereview.webrtc.org/2830793005
> > Cr-Commit-Position: refs/heads/master@{#18215}
> > Committed: https://chromium.googlesource.com/external/webrtc/+/0b8bfb9d98b7a2011d80bcdf3f430bc040370dad
> 
> TBR=stefan@webrtc.org,noahric@chromium.org,glaznev@webrtc.org,sprang@webrtc.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=webrtc:7475
> 
> Review-Url: https://codereview.webrtc.org/2893003002
> Cr-Commit-Position: refs/heads/master@{#18216}
> Committed: https://chromium.googlesource.com/external/webrtc/+/56e119e2e8b30d2d41c55c3a7983b077a5262718

TBR=stefan@webrtc.org,noahric@chromium.org,glaznev@webrtc.org,sprang@webrtc.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=webrtc:7475
Review-Url: https://codereview.webrtc.org/2901493002
Cr-Commit-Position: refs/heads/master@{#18228}
Committed: https://chromium.googlesource.com/external/webrtc/+/5e171752a21a90c4ee186e5e2fffe8a0a6675e99

[brandtr, 2017-05-22 07:12:22]

Created Reland of use allocated encoders in SimulcastEncoderAdapter.

[brandtr, 2017-05-22 07:43:34]

Patchset #2 (id:140001) has been deleted

[brandtr, 2017-05-22 07:45:50]

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

[commit-bot: I haz the power, 2017-05-22 07:45:53]

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-05-22 08:06:00]

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

[commit-bot: I haz the power, 2017-05-22 08:06:01]

Dry run: This issue passed the CQ dry run.

[brandtr, 2017-05-22 11:21:22]

Patchset #2 (id:160001) has been deleted

[brandtr, 2017-05-22 15:03:20]

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

[commit-bot: I haz the power, 2017-05-22 15:03:29]

Dry run: CQ is trying da patch.

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

[brandtr, 2017-05-22 15:03:37]

Description was changed from

==========
Reland of use allocated encoders in SimulcastEncoderAdapter. (patchset #1 id:1
of https://codereview.webrtc.org/2893003002/ )

Reason for revert:
Don't call RegisterEncodeCallback(nullptr), to satisfy test.

Original issue's description:
> Revert of Reuse allocated encoders in SimulcastEncoderAdapter. (patchset #15
id:320001 of https://codereview.webrtc.org/2830793005/ )
> 
> Reason for revert:
> Breaks Chrome tests.
> 
> Original issue's description:
> > Reuse allocated encoders in SimulcastEncoderAdapter.
> > 
> > Prior to this change, the SimulcastEncoderAdapter would destroy and create
> > encoders whenever it is being reinitialized. After this change, the
> > SimulcastEncoderAdapter will cache the already allocated encoders, and reuse
> > them after reinitialization.
> > 
> > This change will help in reducing the number of PictureID "jumps" that have
> > been seen around encoder reinitialization.
> > 
> > TESTED=AppRTCMobile, Chrome desktop, and internal app, with forced encoder
reinits every 30 frames and https://codereview.webrtc.org/2833493003/ applied.
> > BUG=webrtc:7475
> > 
> > Review-Url: https://codereview.webrtc.org/2830793005
> > Cr-Commit-Position: refs/heads/master@{#18215}
> > Committed:
https://chromium.googlesource.com/external/webrtc/+/0b8bfb9d98b7a2011d80bcdf3...
> 
>
TBR=stefan@webrtc.org,noahric@chromium.org,glaznev@webrtc.org,sprang@webrtc.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=webrtc:7475
> 
> Review-Url: https://codereview.webrtc.org/2893003002
> Cr-Commit-Position: refs/heads/master@{#18216}
> Committed:
https://chromium.googlesource.com/external/webrtc/+/56e119e2e8b30d2d41c55c3a7...

TBR=stefan@webrtc.org,noahric@chromium.org,glaznev@webrtc.org,sprang@webrtc.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=webrtc:7475
==========

to

==========
Reland of use allocated encoders in SimulcastEncoderAdapter. (patchset #1 id:1
of https://codereview.webrtc.org/2893003002/ )

Reason for revert:
Chrome encoder implementation fixed.

Original issue's description:
> Revert of Reuse allocated encoders in SimulcastEncoderAdapter. (patchset #15
id:320001 of https://codereview.webrtc.org/2830793005/ )
> 
> Reason for revert:
> Breaks Chrome tests.
> 
> Original issue's description:
> > Reuse allocated encoders in SimulcastEncoderAdapter.
> > 
> > Prior to this change, the SimulcastEncoderAdapter would destroy and create
> > encoders whenever it is being reinitialized. After this change, the
> > SimulcastEncoderAdapter will cache the already allocated encoders, and reuse
> > them after reinitialization.
> > 
> > This change will help in reducing the number of PictureID "jumps" that have
> > been seen around encoder reinitialization.
> > 
> > TESTED=AppRTCMobile, Chrome desktop, and internal app, with forced encoder
reinits every 30 frames and https://codereview.webrtc.org/2833493003/ applied.
> > BUG=webrtc:7475
> > 
> > Review-Url: https://codereview.webrtc.org/2830793005
> > Cr-Commit-Position: refs/heads/master@{#18215}
> > Committed:
https://chromium.googlesource.com/external/webrtc/+/0b8bfb9d98b7a2011d80bcdf3...
> 
>
TBR=stefan@webrtc.org,noahric@chromium.org,glaznev@webrtc.org,sprang@webrtc.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=webrtc:7475
> 
> Review-Url: https://codereview.webrtc.org/2893003002
> Cr-Commit-Position: refs/heads/master@{#18216}
> Committed:
https://chromium.googlesource.com/external/webrtc/+/56e119e2e8b30d2d41c55c3a7...

TBR=stefan@webrtc.org,noahric@chromium.org,glaznev@webrtc.org,sprang@webrtc.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=webrtc:7475
==========

[brandtr, 2017-05-22 15:03:49]

Description was changed from

==========
Reland of use allocated encoders in SimulcastEncoderAdapter. (patchset #1 id:1
of https://codereview.webrtc.org/2893003002/ )

Reason for revert:
Chrome encoder implementation fixed.

Original issue's description:
> Revert of Reuse allocated encoders in SimulcastEncoderAdapter. (patchset #15
id:320001 of https://codereview.webrtc.org/2830793005/ )
> 
> Reason for revert:
> Breaks Chrome tests.
> 
> Original issue's description:
> > Reuse allocated encoders in SimulcastEncoderAdapter.
> > 
> > Prior to this change, the SimulcastEncoderAdapter would destroy and create
> > encoders whenever it is being reinitialized. After this change, the
> > SimulcastEncoderAdapter will cache the already allocated encoders, and reuse
> > them after reinitialization.
> > 
> > This change will help in reducing the number of PictureID "jumps" that have
> > been seen around encoder reinitialization.
> > 
> > TESTED=AppRTCMobile, Chrome desktop, and internal app, with forced encoder
reinits every 30 frames and https://codereview.webrtc.org/2833493003/ applied.
> > BUG=webrtc:7475
> > 
> > Review-Url: https://codereview.webrtc.org/2830793005
> > Cr-Commit-Position: refs/heads/master@{#18215}
> > Committed:
https://chromium.googlesource.com/external/webrtc/+/0b8bfb9d98b7a2011d80bcdf3...
> 
>
TBR=stefan@webrtc.org,noahric@chromium.org,glaznev@webrtc.org,sprang@webrtc.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=webrtc:7475
> 
> Review-Url: https://codereview.webrtc.org/2893003002
> Cr-Commit-Position: refs/heads/master@{#18216}
> Committed:
https://chromium.googlesource.com/external/webrtc/+/56e119e2e8b30d2d41c55c3a7...

TBR=stefan@webrtc.org,noahric@chromium.org,glaznev@webrtc.org,sprang@webrtc.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=webrtc:7475
==========

to

==========
Reland of use allocated encoders in SimulcastEncoderAdapter. (patchset #1 id:1
of https://codereview.webrtc.org/2893003002/ )

Reason for reland:
Chrome encoder implementation fixed.

Original issue's description:
> Revert of Reuse allocated encoders in SimulcastEncoderAdapter. (patchset #15
id:320001 of https://codereview.webrtc.org/2830793005/ )
> 
> Reason for revert:
> Breaks Chrome tests.
> 
> Original issue's description:
> > Reuse allocated encoders in SimulcastEncoderAdapter.
> > 
> > Prior to this change, the SimulcastEncoderAdapter would destroy and create
> > encoders whenever it is being reinitialized. After this change, the
> > SimulcastEncoderAdapter will cache the already allocated encoders, and reuse
> > them after reinitialization.
> > 
> > This change will help in reducing the number of PictureID "jumps" that have
> > been seen around encoder reinitialization.
> > 
> > TESTED=AppRTCMobile, Chrome desktop, and internal app, with forced encoder
reinits every 30 frames and https://codereview.webrtc.org/2833493003/ applied.
> > BUG=webrtc:7475
> > 
> > Review-Url: https://codereview.webrtc.org/2830793005
> > Cr-Commit-Position: refs/heads/master@{#18215}
> > Committed:
https://chromium.googlesource.com/external/webrtc/+/0b8bfb9d98b7a2011d80bcdf3...
> 
>
TBR=stefan@webrtc.org,noahric@chromium.org,glaznev@webrtc.org,sprang@webrtc.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=webrtc:7475
> 
> Review-Url: https://codereview.webrtc.org/2893003002
> Cr-Commit-Position: refs/heads/master@{#18216}
> Committed:
https://chromium.googlesource.com/external/webrtc/+/56e119e2e8b30d2d41c55c3a7...

TBR=stefan@webrtc.org,noahric@chromium.org,glaznev@webrtc.org,sprang@webrtc.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=webrtc:7475
==========

[commit-bot: I haz the power, 2017-05-22 15:21:59]

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

[commit-bot: I haz the power, 2017-05-22 15:22:00]

Dry run: This issue passed the CQ dry run.

[brandtr, 2017-05-23 08:54:44]

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

[commit-bot: I haz the power, 2017-05-23 08:54:58]

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-05-23 09:24:45]

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

[commit-bot: I haz the power, 2017-05-23 09:24:46]

Dry run: This issue passed the CQ dry run.

[brandtr, 2017-05-23 10:30:29]

The CQ bit was checked by brandtr@webrtc.org

[commit-bot: I haz the power, 2017-05-23 10:30:35]

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-05-23 10:32:16]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1495535429874940, "parent_rev":
"884ab92d7c9587d7f6ade10e8a0442e56b34a74a", "commit_rev":
"5e171752a21a90c4ee186e5e2fffe8a0a6675e99"}

[commit-bot: I haz the power, 2017-05-23 10:32:25]

Description was changed from

==========
Reland of use allocated encoders in SimulcastEncoderAdapter. (patchset #1 id:1
of https://codereview.webrtc.org/2893003002/ )

Reason for reland:
Chrome encoder implementation fixed.

Original issue's description:
> Revert of Reuse allocated encoders in SimulcastEncoderAdapter. (patchset #15
id:320001 of https://codereview.webrtc.org/2830793005/ )
> 
> Reason for revert:
> Breaks Chrome tests.
> 
> Original issue's description:
> > Reuse allocated encoders in SimulcastEncoderAdapter.
> > 
> > Prior to this change, the SimulcastEncoderAdapter would destroy and create
> > encoders whenever it is being reinitialized. After this change, the
> > SimulcastEncoderAdapter will cache the already allocated encoders, and reuse
> > them after reinitialization.
> > 
> > This change will help in reducing the number of PictureID "jumps" that have
> > been seen around encoder reinitialization.
> > 
> > TESTED=AppRTCMobile, Chrome desktop, and internal app, with forced encoder
reinits every 30 frames and https://codereview.webrtc.org/2833493003/ applied.
> > BUG=webrtc:7475
> > 
> > Review-Url: https://codereview.webrtc.org/2830793005
> > Cr-Commit-Position: refs/heads/master@{#18215}
> > Committed:
https://chromium.googlesource.com/external/webrtc/+/0b8bfb9d98b7a2011d80bcdf3...
> 
>
TBR=stefan@webrtc.org,noahric@chromium.org,glaznev@webrtc.org,sprang@webrtc.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=webrtc:7475
> 
> Review-Url: https://codereview.webrtc.org/2893003002
> Cr-Commit-Position: refs/heads/master@{#18216}
> Committed:
https://chromium.googlesource.com/external/webrtc/+/56e119e2e8b30d2d41c55c3a7...

TBR=stefan@webrtc.org,noahric@chromium.org,glaznev@webrtc.org,sprang@webrtc.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=webrtc:7475
==========

to

==========
Reland of use allocated encoders in SimulcastEncoderAdapter. (patchset #1 id:1
of https://codereview.webrtc.org/2893003002/ )

Reason for reland:
Chrome encoder implementation fixed.

Original issue's description:
> Revert of Reuse allocated encoders in SimulcastEncoderAdapter. (patchset #15
id:320001 of https://codereview.webrtc.org/2830793005/ )
> 
> Reason for revert:
> Breaks Chrome tests.
> 
> Original issue's description:
> > Reuse allocated encoders in SimulcastEncoderAdapter.
> > 
> > Prior to this change, the SimulcastEncoderAdapter would destroy and create
> > encoders whenever it is being reinitialized. After this change, the
> > SimulcastEncoderAdapter will cache the already allocated encoders, and reuse
> > them after reinitialization.
> > 
> > This change will help in reducing the number of PictureID "jumps" that have
> > been seen around encoder reinitialization.
> > 
> > TESTED=AppRTCMobile, Chrome desktop, and internal app, with forced encoder
reinits every 30 frames and https://codereview.webrtc.org/2833493003/ applied.
> > BUG=webrtc:7475
> > 
> > Review-Url: https://codereview.webrtc.org/2830793005
> > Cr-Commit-Position: refs/heads/master@{#18215}
> > Committed:
https://chromium.googlesource.com/external/webrtc/+/0b8bfb9d98b7a2011d80bcdf3...
> 
>
TBR=stefan@webrtc.org,noahric@chromium.org,glaznev@webrtc.org,sprang@webrtc.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=webrtc:7475
> 
> Review-Url: https://codereview.webrtc.org/2893003002
> Cr-Commit-Position: refs/heads/master@{#18216}
> Committed:
https://chromium.googlesource.com/external/webrtc/+/56e119e2e8b30d2d41c55c3a7...

TBR=stefan@webrtc.org,noahric@chromium.org,glaznev@webrtc.org,sprang@webrtc.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=webrtc:7475

Review-Url: https://codereview.webrtc.org/2901493002
Cr-Commit-Position: refs/heads/master@{#18228}
Committed:
https://chromium.googlesource.com/external/webrtc/+/5e171752a21a90c4ee186e5e2...
==========

[commit-bot: I haz the power, 2017-05-23 10:32:26]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/external/webrtc/+/5e171752a21a90c4ee186e5e2...