Motown: Convert MediaSink to expose MediaTimelineControlSite

Motown: Convert MediaSink to expose MediaTimelineControlSite
1) Remove play/pause and status from MediaSink.
2) Support GetTimelineControlSite on MediaSink so player has access to
   the MediaTimelineControlSite on the underlying renderer.
3) Remove unneeded MediaState in favor of the current transform and
   an end-of-stream indication.
4) Make AudioTrack report end-of-stream (by way of TimelineControlSite),
   removing this logic from the MediaProducer.
5) Eliminate unneeded logic from MediaProducer now that timeline control
   no longer requires packet PTS.

R=kulakowski@chromium.org

Committed: https://chromium.googlesource.com/external/mojo/+/8f792ddc84935f637bfdde2c87980d8d3e3c874c

[dalesat, 2016-05-24 19:42:34]

Description was changed from

==========
Motown: Convert MediaSink to expose MediaTimelineControlSite
1) Remove play/pause and status from MediaSink.
2) Support GetTimelineControlSite on MediaSink so player has access to
   the MediaTimelineControlSite on the underlying renderer.
3) Remove unneeded MediaState in favor of the current transform and
   an end-of-stream indication.
4) Make AudioTrack report end-of-stream (by way of TimelineControlSite),
   removing this logic from the MediaProducer.
5) Eliminate unneeded logic from MediaProducer now that timeline control
   no longer requires packet PTS.
==========

to

==========
Motown: Convert MediaSink to expose MediaTimelineControlSite
1) Remove play/pause and status from MediaSink.
2) Support GetTimelineControlSite on MediaSink so player has access to
   the MediaTimelineControlSite on the underlying renderer.
3) Remove unneeded MediaState in favor of the current transform and
   an end-of-stream indication.
4) Make AudioTrack report end-of-stream (by way of TimelineControlSite),
   removing this logic from the MediaProducer.
5) Eliminate unneeded logic from MediaProducer now that timeline control
   no longer requires packet PTS.
==========

[dalesat, 2016-05-24 19:42:34]

dalesat@chromium.org changed reviewers:
+ kulakowski@chromium.org

[dalesat, 2016-05-24 19:42:48]

Please take a look. Thanks!

[kulakowski, 2016-05-24 20:27:52]

lgtm

https://codereview.chromium.org/2006093004/diff/1/services/media/framework_mo...
File services/media/framework_mojo/mojo_producer.cc (right):

https://codereview.chromium.org/2006093004/diff/1/services/media/framework_mo...
services/media/framework_mojo/mojo_producer.cc:83: base::AutoLock lock(lock_);
I don't think you need to lock here.

[dalesat, 2016-05-24 21:37:26]

Description was changed from

==========
Motown: Convert MediaSink to expose MediaTimelineControlSite
1) Remove play/pause and status from MediaSink.
2) Support GetTimelineControlSite on MediaSink so player has access to
   the MediaTimelineControlSite on the underlying renderer.
3) Remove unneeded MediaState in favor of the current transform and
   an end-of-stream indication.
4) Make AudioTrack report end-of-stream (by way of TimelineControlSite),
   removing this logic from the MediaProducer.
5) Eliminate unneeded logic from MediaProducer now that timeline control
   no longer requires packet PTS.
==========

to

==========
Motown: Convert MediaSink to expose MediaTimelineControlSite
1) Remove play/pause and status from MediaSink.
2) Support GetTimelineControlSite on MediaSink so player has access to
   the MediaTimelineControlSite on the underlying renderer.
3) Remove unneeded MediaState in favor of the current transform and
   an end-of-stream indication.
4) Make AudioTrack report end-of-stream (by way of TimelineControlSite),
   removing this logic from the MediaProducer.
5) Eliminate unneeded logic from MediaProducer now that timeline control
   no longer requires packet PTS.

R=kulakowski@chromium.org

Committed:
https://chromium.googlesource.com/external/mojo/+/8f792ddc84935f637bfdde2c879...
==========

[dalesat, 2016-05-24 21:37:28]

Committed patchset #1 (id:1) manually as
8f792ddc84935f637bfdde2c87980d8d3e3c874c (presubmit successful).