[DO NOT COMMIT] MediaPlayerRenderer using StreamTextures.

[DO NOT COMMIT] MediaPlayerRenderer using StreamTextures.

UPDATE: The relevant code is mostly checked in. Closing this issue. See the bugs tied to https://bugs.chromium.org/p/chromium/issues/detail?id=619729 in order to get the full picture of the code that landed.

This code is still a prototype, and will be commited in smaller,
cleaner CLs.

Some of the comments are lacking and out of date, since the code has
already changed. The prototype also blatantly hardcodes some values
and leaks textures, and is not meant to be committed. I have also added
many 'TODOs', both as a note to myself, and as a flag for readers as to
what is likely to change. Some interfaces are also temporarily
implemented as a stepping stone before removing WMPA.

The rough order in which I intend to submit the code is:

- StreamTextureWrapper (after the proper 'OnFrameAvailable' signalling
  has been decided upon).
- MediaPlayerRenderer/MediaPlayerRendererHost.
- The registration mechanism for the StreamTexture.
- The full integration + flag for running experiments.

Feel free to add comments here or to ping me offline!
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation

[tguilbert, 2016-07-20 05:30:09]

Description was changed from

==========
[DO NOT COMMIT] MediaPlayerRenderer using StreamTextures.

This code is still a prototype, and will be commited in smaller,
cleaner CLs.

Some of the comments are lacking and out of date, since the code has
already changed. The prototype also blatantly hardcodes some values
and leaks textures, and is not meant to be committed. I have also added
many 'TODOs', both as a note to myself, and as a flag for readers as to
what is likely to change. Some interfaces are also temporarily
implemented as a stepping stone before removing WMPA.

The rough order in which I intend to submit the code is:

- StreamTextureWrapper (after the proper 'OnFrameAvailable' signalling
  has been decided upon).
- MediaPlayerRenderer/MediaPlayerRendererHost.
- The registration mechanism for the StreamTexture.
- The full integration + flag for running experiments.

Feel free to add comments here or to ping me offline!
==========

to

==========
[DO NOT COMMIT] MediaPlayerRenderer using StreamTextures.

This code is still a prototype, and will be commited in smaller,
cleaner CLs.

Some of the comments are lacking and out of date, since the code has
already changed. The prototype also blatantly hardcodes some values
and leaks textures, and is not meant to be committed. I have also added
many 'TODOs', both as a note to myself, and as a flag for readers as to
what is likely to change. Some interfaces are also temporarily
implemented as a stepping stone before removing WMPA.

The rough order in which I intend to submit the code is:

- StreamTextureWrapper (after the proper 'OnFrameAvailable' signalling
  has been decided upon).
- MediaPlayerRenderer/MediaPlayerRendererHost.
- The registration mechanism for the StreamTexture.
- The full integration + flag for running experiments.

Feel free to add comments here or to ping me offline!
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
==========

[Julien Isorce Samsung, 2016-08-08 12:58:01]

https://codereview.chromium.org/2161083004/diff/1/media/blink/webmediaplayer_...
File media/blink/webmediaplayer_impl.cc (right):

https://codereview.chromium.org/2161083004/diff/1/media/blink/webmediaplayer_...
media/blink/webmediaplayer_impl.cc:1370: return;
Hi, sorry for the short introduction, I was thinking of creating a new
MojoPipelineService but then I saw your solution that re-use MojoRenderer + this
pseudo demuxer that just passes the url. You said that creating a proper service
would have required too much unnecessary refactoring "Removing the Demuxer from
the pipeline when using a MediaPlayer would require a non-trivial amount of
refactoring, that might not be worth it."
So I was wondering if after all this work you are still thinking that there is
no need for a proper MojoPipelineService ?
Nice solution btw !

[tguilbert, 2016-09-26 23:32:05]

Description was changed from

==========
[DO NOT COMMIT] MediaPlayerRenderer using StreamTextures.

This code is still a prototype, and will be commited in smaller,
cleaner CLs.

Some of the comments are lacking and out of date, since the code has
already changed. The prototype also blatantly hardcodes some values
and leaks textures, and is not meant to be committed. I have also added
many 'TODOs', both as a note to myself, and as a flag for readers as to
what is likely to change. Some interfaces are also temporarily
implemented as a stepping stone before removing WMPA.

The rough order in which I intend to submit the code is:

- StreamTextureWrapper (after the proper 'OnFrameAvailable' signalling
  has been decided upon).
- MediaPlayerRenderer/MediaPlayerRendererHost.
- The registration mechanism for the StreamTexture.
- The full integration + flag for running experiments.

Feel free to add comments here or to ping me offline!
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
==========

to

==========
[DO NOT COMMIT] MediaPlayerRenderer using StreamTextures.

UPDATE: The relevant code is mostly checked in. Closing this issue. See the bugs
tied to https://bugs.chromium.org/p/chromium/issues/detail?id=619729 in order to
get the full picture of the code that landed.

This code is still a prototype, and will be commited in smaller,
cleaner CLs.

Some of the comments are lacking and out of date, since the code has
already changed. The prototype also blatantly hardcodes some values
and leaks textures, and is not meant to be committed. I have also added
many 'TODOs', both as a note to myself, and as a flag for readers as to
what is likely to change. Some interfaces are also temporarily
implemented as a stepping stone before removing WMPA.

The rough order in which I intend to submit the code is:

- StreamTextureWrapper (after the proper 'OnFrameAvailable' signalling
  has been decided upon).
- MediaPlayerRenderer/MediaPlayerRendererHost.
- The registration mechanism for the StreamTexture.
- The full integration + flag for running experiments.

Feel free to add comments here or to ping me offline!
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
==========