Revert of Prepare for injectable SW decoders

Revert of Prepare for injectable SW decoders (patchset #3 id:40001 of https://codereview.webrtc.org/3009973002/ )

Reason for revert:
Tentative revert since it seems to cause problems in Chrome, MAC.

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



Original issue's description:
> Prepare for injectable SW decoders
> 
> Pretty much mirrors the work done on the encoding side in CLs:
> 
> "Clean up ownership of webrtc::VideoEncoder"
> https://codereview.webrtc.org/3007643002/
> 
> "Let VideoEncoderSoftwareFallbackWrapper own the wrapped encoder"
> https://codereview.webrtc.org/3007683002/
> 
> "WebRtcVideoEngine: Encapsulate logic for unifying internal and external video codecs"
> https://codereview.webrtc.org/3006713002/
> 
> BUG=webrtc:7925
> 
> Review-Url: https://codereview.webrtc.org/3009973002
> Cr-Commit-Position: refs/heads/master@{#19641}
> Committed: https://chromium.googlesource.com/external/webrtc/+/084c55a63a2d9bdc71579458406d44f8bab9f454

TBR=magjed@webrtc.org,andersc@webrtc.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=webrtc:7925
Review-Url: https://codereview.webrtc.org/3010953002
Cr-Commit-Position: refs/heads/master@{#19647}
Committed: https://chromium.googlesource.com/external/webrtc/+/1f88531038c24c5ce3b0f4cfc682b970770a71f6

[perkj_webrtc, 2017-09-04 07:45:08]

The CQ bit was checked by perkj@webrtc.org

[perkj_webrtc, 2017-09-04 07:45:09]

Created Revert of Prepare for injectable SW decoders

[commit-bot: I haz the power, 2017-09-04 07:45:14]

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-09-04 09:43:10]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1504511108692430, "parent_rev":
"678636bf7ce90d8f0dfb587ea880101184cd7a9d", "commit_rev":
"1f88531038c24c5ce3b0f4cfc682b970770a71f6"}

[commit-bot: I haz the power, 2017-09-04 09:43:25]

Description was changed from

==========
Revert of Prepare for injectable SW decoders (patchset #3 id:40001 of
https://codereview.webrtc.org/3009973002/ )

Reason for revert:
Tentative revert since it seems to cause problems in Chrome, MAC.

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



Original issue's description:
> Prepare for injectable SW decoders
> 
> Pretty much mirrors the work done on the encoding side in CLs:
> 
> "Clean up ownership of webrtc::VideoEncoder"
> https://codereview.webrtc.org/3007643002/
> 
> "Let VideoEncoderSoftwareFallbackWrapper own the wrapped encoder"
> https://codereview.webrtc.org/3007683002/
> 
> "WebRtcVideoEngine: Encapsulate logic for unifying internal and external video
codecs"
> https://codereview.webrtc.org/3006713002/
> 
> BUG=webrtc:7925
> 
> Review-Url: https://codereview.webrtc.org/3009973002
> Cr-Commit-Position: refs/heads/master@{#19641}
> Committed:
https://chromium.googlesource.com/external/webrtc/+/084c55a63a2d9bdc715794584...

TBR=magjed@webrtc.org,andersc@webrtc.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=webrtc:7925
==========

to

==========
Revert of Prepare for injectable SW decoders (patchset #3 id:40001 of
https://codereview.webrtc.org/3009973002/ )

Reason for revert:
Tentative revert since it seems to cause problems in Chrome, MAC.

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



Original issue's description:
> Prepare for injectable SW decoders
> 
> Pretty much mirrors the work done on the encoding side in CLs:
> 
> "Clean up ownership of webrtc::VideoEncoder"
> https://codereview.webrtc.org/3007643002/
> 
> "Let VideoEncoderSoftwareFallbackWrapper own the wrapped encoder"
> https://codereview.webrtc.org/3007683002/
> 
> "WebRtcVideoEngine: Encapsulate logic for unifying internal and external video
codecs"
> https://codereview.webrtc.org/3006713002/
> 
> BUG=webrtc:7925
> 
> Review-Url: https://codereview.webrtc.org/3009973002
> Cr-Commit-Position: refs/heads/master@{#19641}
> Committed:
https://chromium.googlesource.com/external/webrtc/+/084c55a63a2d9bdc715794584...

TBR=magjed@webrtc.org,andersc@webrtc.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=webrtc:7925

Review-Url: https://codereview.webrtc.org/3010953002
Cr-Commit-Position: refs/heads/master@{#19647}
Committed:
https://chromium.googlesource.com/external/webrtc/+/1f88531038c24c5ce3b0f4cfc...
==========

[commit-bot: I haz the power, 2017-09-04 09:43:26]

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

[andersc, 2017-09-11 13:42:10]

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

The reason for reverting is: Fix bug introduced by keeping the allocated
decoders in a map..