Move VideoDecoderSoftwareFallbackWrapper from webrtc/video_decoder.h to webrtc/media/engine/

Move VideoDecoderSoftwareFallbackWrapper from webrtc/video_decoder.h to webrtc/media/engine/

The class VideoDecoderSoftwareFallbackWrapper is an implementation
detail of webrtc/media/engine/webrtcvideoengine2.cc and should not be
directly under webrtc/video_decoder.h. The main purpose is to improve
the dependency graph in WebRTC so that VideoDecoderSoftwareFallbackWrapper
can depend on cricket::VideoCodec.

The test for VideoDecoderSoftwareFallbackWrapper is also moved from
webrtc/video/video_decoder_unittest.cc to
webrtc/media/engine/videodecodersoftwarefallbackwrapper_unittest.cc.

BUG=webrtc:6743
TBR=stefan@webrtc.org

Committed: https://crrev.com/f6acc2a7107323b6078edd5b15ee757d9da10dae
Cr-Commit-Position: refs/heads/master@{#15180}

[magjed_webrtc, 2016-11-22 08:56:52]

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

[commit-bot: I haz the power, 2016-11-22 08:57:03]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...

[magjed_webrtc, 2016-11-22 08:59:57]

Patchset #2 (id:20001) has been deleted

[magjed_webrtc, 2016-11-22 09:06:13]

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

[commit-bot: I haz the power, 2016-11-22 09:06:25]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...

[magjed_webrtc, 2016-11-22 09:33:43]

Description was changed from

==========
Move VideoDecoderSoftwareFallbackWrapper from webrtc/video_decoder.h to
webrtc/media/engine/

The class VideoDecoderSoftwareFallbackWrapper is an implementation
detail of webrtc/media/engine/webrtcvideoengine2.cc and should not be
directly under webrtc/video_decoder.h. The main purpose is to improve
the dependency graph in WebRTC so that VideoDecoderSoftwareFallbackWrapper
can depend on cricket::VideoCodec.

The test for VideoDecoderSoftwareFallbackWrapper is also moved from
webrtc/video/video_decoder_unittest.cc to
webrtc/media/engine/videodecodersoftwarefallbackwrapper_unittest.cc.

BUG=webrtc:6337
==========

to

==========
Move VideoDecoderSoftwareFallbackWrapper from webrtc/video_decoder.h to
webrtc/media/engine/

The class VideoDecoderSoftwareFallbackWrapper is an implementation
detail of webrtc/media/engine/webrtcvideoengine2.cc and should not be
directly under webrtc/video_decoder.h. The main purpose is to improve
the dependency graph in WebRTC so that VideoDecoderSoftwareFallbackWrapper
can depend on cricket::VideoCodec.

The test for VideoDecoderSoftwareFallbackWrapper is also moved from
webrtc/video/video_decoder_unittest.cc to
webrtc/media/engine/videodecodersoftwarefallbackwrapper_unittest.cc.

BUG=webrtc:6743
==========

[magjed_webrtc, 2016-11-22 09:33:58]

Patchset #1 (id:1) has been deleted

[commit-bot: I haz the power, 2016-11-22 09:38:07]

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

[commit-bot: I haz the power, 2016-11-22 09:38:07]

Dry run: This issue passed the CQ dry run.

[magjed_webrtc, 2016-11-22 09:38:12]

Description was changed from

==========
Move VideoDecoderSoftwareFallbackWrapper from webrtc/video_decoder.h to
webrtc/media/engine/

The class VideoDecoderSoftwareFallbackWrapper is an implementation
detail of webrtc/media/engine/webrtcvideoengine2.cc and should not be
directly under webrtc/video_decoder.h. The main purpose is to improve
the dependency graph in WebRTC so that VideoDecoderSoftwareFallbackWrapper
can depend on cricket::VideoCodec.

The test for VideoDecoderSoftwareFallbackWrapper is also moved from
webrtc/video/video_decoder_unittest.cc to
webrtc/media/engine/videodecodersoftwarefallbackwrapper_unittest.cc.

BUG=webrtc:6743
==========

to

==========
Move VideoDecoderSoftwareFallbackWrapper from webrtc/video_decoder.h to
webrtc/media/engine/

The class VideoDecoderSoftwareFallbackWrapper is an implementation
detail of webrtc/media/engine/webrtcvideoengine2.cc and should not be
directly under webrtc/video_decoder.h. The main purpose is to improve
the dependency graph in WebRTC so that VideoDecoderSoftwareFallbackWrapper
can depend on cricket::VideoCodec.

The test for VideoDecoderSoftwareFallbackWrapper is also moved from
webrtc/video/video_decoder_unittest.cc to
webrtc/media/engine/videodecodersoftwarefallbackwrapper_unittest.cc.

BUG=webrtc:6743
TBR=stefan@webrtc.org
==========

[magjed_webrtc, 2016-11-22 09:38:12]

magjed@webrtc.org changed reviewers:
+ stefan@webrtc.org

[magjed_webrtc, 2016-11-22 09:41:34]

Stefan - I know that you are busy so I'm TBR:ing this CL. It's pure moving of
code and no functional changes. I want to get as much implementation details as
possible out from the webrtc root (webrtc/video_decoder.h) to simplify the
dependency graph. This CL is an exact decode analogy of this encode CL that you
have already stamped here: https://codereview.webrtc.org/2484863009/.

[magjed_webrtc, 2016-11-22 09:41:38]

The CQ bit was checked by magjed@webrtc.org

[commit-bot: I haz the power, 2016-11-22 09:41:48]

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-11-22 09:43:03]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1479807698700040,
"parent_rev": "be24d675e20e0308f3446d2318308f73271e5588", "commit_rev":
"3e90662969596a535c9f88d5a938476676772385"}

[commit-bot: I haz the power, 2016-11-22 09:43:05]

Description was changed from

==========
Move VideoDecoderSoftwareFallbackWrapper from webrtc/video_decoder.h to
webrtc/media/engine/

The class VideoDecoderSoftwareFallbackWrapper is an implementation
detail of webrtc/media/engine/webrtcvideoengine2.cc and should not be
directly under webrtc/video_decoder.h. The main purpose is to improve
the dependency graph in WebRTC so that VideoDecoderSoftwareFallbackWrapper
can depend on cricket::VideoCodec.

The test for VideoDecoderSoftwareFallbackWrapper is also moved from
webrtc/video/video_decoder_unittest.cc to
webrtc/media/engine/videodecodersoftwarefallbackwrapper_unittest.cc.

BUG=webrtc:6743
TBR=stefan@webrtc.org
==========

to

==========
Move VideoDecoderSoftwareFallbackWrapper from webrtc/video_decoder.h to
webrtc/media/engine/

The class VideoDecoderSoftwareFallbackWrapper is an implementation
detail of webrtc/media/engine/webrtcvideoengine2.cc and should not be
directly under webrtc/video_decoder.h. The main purpose is to improve
the dependency graph in WebRTC so that VideoDecoderSoftwareFallbackWrapper
can depend on cricket::VideoCodec.

The test for VideoDecoderSoftwareFallbackWrapper is also moved from
webrtc/video/video_decoder_unittest.cc to
webrtc/media/engine/videodecodersoftwarefallbackwrapper_unittest.cc.

BUG=webrtc:6743
TBR=stefan@webrtc.org
==========

[commit-bot: I haz the power, 2016-11-22 09:43:06]

Committed patchset #1 (id:40001)

[commit-bot: I haz the power, 2016-11-22 09:43:11]

Description was changed from

==========
Move VideoDecoderSoftwareFallbackWrapper from webrtc/video_decoder.h to
webrtc/media/engine/

The class VideoDecoderSoftwareFallbackWrapper is an implementation
detail of webrtc/media/engine/webrtcvideoengine2.cc and should not be
directly under webrtc/video_decoder.h. The main purpose is to improve
the dependency graph in WebRTC so that VideoDecoderSoftwareFallbackWrapper
can depend on cricket::VideoCodec.

The test for VideoDecoderSoftwareFallbackWrapper is also moved from
webrtc/video/video_decoder_unittest.cc to
webrtc/media/engine/videodecodersoftwarefallbackwrapper_unittest.cc.

BUG=webrtc:6743
TBR=stefan@webrtc.org
==========

to

==========
Move VideoDecoderSoftwareFallbackWrapper from webrtc/video_decoder.h to
webrtc/media/engine/

The class VideoDecoderSoftwareFallbackWrapper is an implementation
detail of webrtc/media/engine/webrtcvideoengine2.cc and should not be
directly under webrtc/video_decoder.h. The main purpose is to improve
the dependency graph in WebRTC so that VideoDecoderSoftwareFallbackWrapper
can depend on cricket::VideoCodec.

The test for VideoDecoderSoftwareFallbackWrapper is also moved from
webrtc/video/video_decoder_unittest.cc to
webrtc/media/engine/videodecodersoftwarefallbackwrapper_unittest.cc.

BUG=webrtc:6743
TBR=stefan@webrtc.org

Committed: https://crrev.com/f6acc2a7107323b6078edd5b15ee757d9da10dae
Cr-Commit-Position: refs/heads/master@{#15180}
==========

[commit-bot: I haz the power, 2016-11-22 09:43:11]

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