Add MojoVideoDecoder.

Add MojoVideoDecoder.

This skeleton adds support for a MojoVideoDecoder, but does not
implement a decoder service.

To enable, use these gn args:
  enable_mojo_media = true
  mojo_media_services = [ "video_decoder" ]

BUG=522298
Committed: https://crrev.com/02864e3315b0959644d47603a2d2627474deb9fb
Cr-Commit-Position: refs/heads/master@{#381331}

[sandersd (OOO until July 31), 2016-03-15 01:27:09]

sandersd@chromium.org changed reviewers:
+ xhwang@chromium.org

[xhwang, 2016-03-15 05:13:18]

LGTM

Thanks for the fix!

https://chromiumcodereview.appspot.com/1800953002/diff/20001/media/media_opti...
File media/media_options.gni (right):

https://chromiumcodereview.appspot.com/1800953002/diff/20001/media/media_opti...
media/media_options.gni:89: # - "cdm": Use mojo based Content Decryption Module.
add "-" as well

https://chromiumcodereview.appspot.com/1800953002/diff/20001/media/mojo/servi...
File media/mojo/services/mojo_decoder_factory.h (right):

https://chromiumcodereview.appspot.com/1800953002/diff/20001/media/mojo/servi...
media/mojo/services/mojo_decoder_factory.h:15: class ServiceFactory;
remove please

https://chromiumcodereview.appspot.com/1800953002/diff/20001/media/mojo/servi...
File media/mojo/services/mojo_video_decoder.h (right):

https://chromiumcodereview.appspot.com/1800953002/diff/20001/media/mojo/servi...
media/mojo/services/mojo_video_decoder.h:18: // A VideoDecoder that proxies to a
interfaces::VideoDecoder.
s/a/an, please help fix the audio one as well. thanks!

https://chromiumcodereview.appspot.com/1800953002/diff/20001/media/renderers/...
File media/renderers/default_renderer_factory.cc (right):

https://chromiumcodereview.appspot.com/1800953002/diff/20001/media/renderers/...
media/renderers/default_renderer_factory.cc:73:
decoder_factory_->CreateVideoDecoders(&video_decoders);
Add a comment why we like to put this as the first one.

[sandersd (OOO until July 31), 2016-03-15 18:31:02]

https://codereview.chromium.org/1800953002/diff/20001/media/media_options.gni
File media/media_options.gni (right):

https://codereview.chromium.org/1800953002/diff/20001/media/media_options.gni...
media/media_options.gni:89: # - "cdm": Use mojo based Content Decryption Module.
On 2016/03/15 05:13:18, xhwang wrote:
> add "-" as well

Done.

https://codereview.chromium.org/1800953002/diff/20001/media/mojo/services/moj...
File media/mojo/services/mojo_video_decoder.h (right):

https://codereview.chromium.org/1800953002/diff/20001/media/mojo/services/moj...
media/mojo/services/mojo_video_decoder.h:18: // A VideoDecoder that proxies to a
interfaces::VideoDecoder.
On 2016/03/15 05:13:18, xhwang wrote:
> s/a/an, please help fix the audio one as well. thanks!

Done.

https://codereview.chromium.org/1800953002/diff/20001/media/renderers/default...
File media/renderers/default_renderer_factory.cc (right):

https://codereview.chromium.org/1800953002/diff/20001/media/renderers/default...
media/renderers/default_renderer_factory.cc:73:
decoder_factory_->CreateVideoDecoders(&video_decoders);
On 2016/03/15 05:13:18, xhwang wrote:
> Add a comment why we like to put this as the first one.

Done.

[sandersd (OOO until July 31), 2016-03-15 18:31:27]

sandersd@chromium.org changed reviewers:
+ jochen@chromium.org

[sandersd (OOO until July 31), 2016-03-15 18:31:28]

jochen@chromium.org: Please review changes in RenderFrameImpl.

[jochen (gone - plz use gerrit), 2016-03-15 20:27:10]

lgtm

[sandersd (OOO until July 31), 2016-03-15 20:27:46]

The CQ bit was checked by sandersd@chromium.org

[sandersd (OOO until July 31), 2016-03-15 20:27:47]

The patchset sent to the CQ was uploaded after l-g-t-m from xhwang@chromium.org
Link to the patchset: https://codereview.chromium.org/1800953002/#ps40001
(title: "Comments.")

[commit-bot: I haz the power, 2016-03-15 20:28:11]

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

[commit-bot: I haz the power, 2016-03-15 22:02:22]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-03-15 22:04:00]

Description was changed from

==========
Add MojoVideoDecoder.

This skeleton adds support for a MojoVideoDecoder, but does not
implement a decoder service.

To enable, use these gn args:
  enable_mojo_media = true
  mojo_media_services = [ "video_decoder" ]

BUG=522298
==========

to

==========
Add MojoVideoDecoder.

This skeleton adds support for a MojoVideoDecoder, but does not
implement a decoder service.

To enable, use these gn args:
  enable_mojo_media = true
  mojo_media_services = [ "video_decoder" ]

BUG=522298

Committed: https://crrev.com/02864e3315b0959644d47603a2d2627474deb9fb
Cr-Commit-Position: refs/heads/master@{#381331}
==========

[commit-bot: I haz the power, 2016-03-15 22:04:01]

Patchset 3 (id:??) landed as
https://crrev.com/02864e3315b0959644d47603a2d2627474deb9fb
Cr-Commit-Position: refs/heads/master@{#381331}