Revert of Introduce GpuVideoDecodeAcceleratorFactory.

Revert of Introduce GpuVideoDecodeAcceleratorFactory. (patchset #22 id:600001 of https://codereview.chromium.org/1745903002/ )

Reason for revert:
Too many problems: Android, Win64 linking, component linking (duplicate code die to incorrect visibility)

Original issue's description:
> Introduce GpuVideoDecodeAcceleratorFactory.
>
> - Move platform-specific code from GpuVideoDecodeAccelerator to
> GpuVideoDecodeAcceleratorFactory.
>
> - Make GVDAFactory a content/public interface, to provide the ability to
> instantiate VDAs from outside content/.
>
> - Unify how we obtain access to various GL functionality/classes from VDAs
> by introducing a set of callbacks provided by the client.
>
> - Replace VDA::CanDecodeOnIOThread() with
> VDA::TryInitializeDecodeOnSeparateThread(). This allows us to remove
> additional client/taskrunner arguments from VDA constructors, and give client
> the option to use a separate thread to decode, instead of having to make this
> decision in the factory, and enforcing these arguments in the constructors.
>
> - Deduplicate VDA creation code across users (currently GVDA and vdaunittest).
>
> BUG=b/27687678
> TEST=compile/run various VDA impls
> CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel
>
> Committed: https://crrev.com/6977e5243786901a766a38c2c291464875dffbd6
> Cr-Commit-Position: refs/heads/master@{#383256}

TBR=jochen@chromium.org,ananta@chromium.org,fsamuel@chromium.org,kcwu@chromium.org,liberato@chromium.org,owenlin@chromium.org,sandersd@chromium.org,jam@chromium.org,boliu@chromium.org,posciak@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=b/27687678

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

[brettw, 2016-03-25 17:53:58]

Created Revert of Introduce GpuVideoDecodeAcceleratorFactory.

[commit-bot: I haz the power, 2016-03-25 17:54:27]

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

[commit-bot: I haz the power, 2016-03-25 17:55:27]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-03-25 17:56:59]

Description was changed from

==========
Revert of Introduce GpuVideoDecodeAcceleratorFactory. (patchset #22 id:600001 of
https://codereview.chromium.org/1745903002/ )

Reason for revert:
Too many problems: Android, Win64 linking, component linking (duplicate code die
to incorrect visibility)

Original issue's description:
> Introduce GpuVideoDecodeAcceleratorFactory.
> 
> - Move platform-specific code from GpuVideoDecodeAccelerator to
> GpuVideoDecodeAcceleratorFactory.
> 
> - Make GVDAFactory a content/public interface, to provide the ability to
> instantiate VDAs from outside content/.
> 
> - Unify how we obtain access to various GL functionality/classes from VDAs
> by introducing a set of callbacks provided by the client.
> 
> - Replace VDA::CanDecodeOnIOThread() with
> VDA::TryInitializeDecodeOnSeparateThread(). This allows us to remove
> additional client/taskrunner arguments from VDA constructors, and give client
> the option to use a separate thread to decode, instead of having to make this
> decision in the factory, and enforcing these arguments in the constructors.
> 
> - Deduplicate VDA creation code across users (currently GVDA and vdaunittest).
> 
> BUG=b/27687678
> TEST=compile/run various VDA impls
> CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel
> 
> Committed: https://crrev.com/6977e5243786901a766a38c2c291464875dffbd6
> Cr-Commit-Position: refs/heads/master@{#383256}

TBR=jochen@chromium.org,ananta@chromium.org,fsamuel@chromium.org,kcwu@chromiu...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=b/27687678
==========

to

==========
Revert of Introduce GpuVideoDecodeAcceleratorFactory. (patchset #22 id:600001 of
https://codereview.chromium.org/1745903002/ )

Reason for revert:
Too many problems: Android, Win64 linking, component linking (duplicate code die
to incorrect visibility)

Original issue's description:
> Introduce GpuVideoDecodeAcceleratorFactory.
>
> - Move platform-specific code from GpuVideoDecodeAccelerator to
> GpuVideoDecodeAcceleratorFactory.
>
> - Make GVDAFactory a content/public interface, to provide the ability to
> instantiate VDAs from outside content/.
>
> - Unify how we obtain access to various GL functionality/classes from VDAs
> by introducing a set of callbacks provided by the client.
>
> - Replace VDA::CanDecodeOnIOThread() with
> VDA::TryInitializeDecodeOnSeparateThread(). This allows us to remove
> additional client/taskrunner arguments from VDA constructors, and give client
> the option to use a separate thread to decode, instead of having to make this
> decision in the factory, and enforcing these arguments in the constructors.
>
> - Deduplicate VDA creation code across users (currently GVDA and vdaunittest).
>
> BUG=b/27687678
> TEST=compile/run various VDA impls
> CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel
>
> Committed: https://crrev.com/6977e5243786901a766a38c2c291464875dffbd6
> Cr-Commit-Position: refs/heads/master@{#383256}

TBR=jochen@chromium.org,ananta@chromium.org,fsamuel@chromium.org,kcwu@chromiu...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=b/27687678

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

[commit-bot: I haz the power, 2016-03-25 17:57:04]

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