Move ContextProvider binding to inside GpuVideoAcceleratorFactory

Move ContextProvider binding to inside GpuVideoAcceleratorFactory

Bind the ContextProvider inside the GpuVideoAcceleratorFactory itself, so the
Factory can decide appropriately what to do in casae of binding failure.

Also: replace references to RendererGpuVideoAcceleratorFactory with references
to its superclass, GpuVideoAcceleratorFactory.

BUG=345424
TEST=local build on desktop Linux
R=avi@chromium.org, fischman@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=253259

[sheu, 2014-02-24 23:59:05]

fischman@: while you're in the reviewing mood, here's another one.  PTAL.

[Ami GONE FROM CHROMIUM, 2014-02-25 00:06:42]

LGTM % please rewrite the comment noted below.
It's unclear to me why content/public is part of this CL; please either drop it
or note in the CL description.

https://chromiumcodereview.appspot.com/171773014/diff/130001/content/renderer...
File content/renderer/media/renderer_gpu_video_accelerator_factories.h (right):

https://chromiumcodereview.appspot.com/171773014/diff/130001/content/renderer...
content/renderer/media/renderer_gpu_video_accelerator_factories.h:74: // Helper
to bind |context_provider| to the |task_runner_| thread initially.
s/r|/r_|/
"initially" is vague.

[sheu, 2014-02-25 00:50:09]

avi@: PTAL at content/*, which is pretty much everything.  Simple change though.

Thanks!

https://chromiumcodereview.appspot.com/171773014/diff/130001/content/renderer...
File content/renderer/media/renderer_gpu_video_accelerator_factories.h (right):

https://chromiumcodereview.appspot.com/171773014/diff/130001/content/renderer...
content/renderer/media/renderer_gpu_video_accelerator_factories.h:74: // Helper
to bind |context_provider| to the |task_runner_| thread initially.
On 2014/02/25 00:06:42, Ami Fischman wrote:
> s/r|/r_|/
> "initially" is vague.

Done.

[Avi (use Gerrit), 2014-02-25 03:45:50]

lgtm

stampity stamp

[sheu, 2014-02-25 06:17:08]

The CQ bit was checked by sheu@chromium.org

[commit-bot: I haz the power, 2014-02-25 06:47:36]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/sheu@chromium.org/171773014/150001

[commit-bot: I haz the power, 2014-02-25 07:47:49]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/sheu@chromium.org/171773014/150001

[commit-bot: I haz the power, 2014-02-25 09:16:23]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/sheu@chromium.org/171773014/150001

[commit-bot: I haz the power, 2014-02-25 09:34:48]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/sheu@chromium.org/171773014/150001

[commit-bot: I haz the power, 2014-02-25 09:43:27]

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

[commit-bot: I haz the power, 2014-02-25 09:43:27]

Try jobs failed on following builders: mac_chromium_rel

[sheu, 2014-02-25 14:47:20]

The CQ bit was checked by sheu@chromium.org

[commit-bot: I haz the power, 2014-02-25 14:47:27]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/sheu@chromium.org/171773014/150001

[commit-bot: I haz the power, 2014-02-25 14:48:55]

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

[commit-bot: I haz the power, 2014-02-25 14:48:56]

Try jobs failed on following builders: mac_chromium_rel

[sheu, 2014-02-25 18:54:27]

The CQ bit was checked by sheu@chromium.org

[commit-bot: I haz the power, 2014-02-25 18:57:09]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/sheu@chromium.org/171773014/150001

[commit-bot: I haz the power, 2014-02-25 19:01:43]

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

[commit-bot: I haz the power, 2014-02-25 19:01:44]

Try jobs failed on following builders: mac_chromium_rel

[sheu, 2014-02-25 19:25:14]

The CQ bit was checked by sheu@chromium.org

[commit-bot: I haz the power, 2014-02-25 19:25:47]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/sheu@chromium.org/171773014/170001

[sheu, 2014-02-25 21:32:17]

Between the previous patchset and this one (which is just a rebase), we've
passed enough bots.  CQ being borked as it today I'm just going to chump this.

[sheu, 2014-02-25 21:33:20]

Committed patchset #3 manually as r253259 (presubmit successful).