Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(133)

Unified Diff: mojo/services/media/control/interfaces/media_player.mojom

Issue 2006093004: Motown: Convert MediaSink to expose MediaTimelineControlSite (Closed) Base URL: https://github.com/domokit/mojo.git@master
Patch Set: Created 4 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: mojo/services/media/control/interfaces/media_player.mojom
diff --git a/mojo/services/media/control/interfaces/media_player.mojom b/mojo/services/media/control/interfaces/media_player.mojom
index 49f4aa427df6d21cd178ac033d2069ed3d9f1fa5..6b3d0ed051519e9ba1f75d5e5e578a98864b0af9 100644
--- a/mojo/services/media/control/interfaces/media_player.mojom
+++ b/mojo/services/media/control/interfaces/media_player.mojom
@@ -6,7 +6,6 @@
module mojo.media;
import "mojo/services/media/common/interfaces/media_metadata.mojom";
-import "mojo/services/media/common/interfaces/media_state.mojom";
import "mojo/services/media/common/interfaces/timelines.mojom";
// Plays media.
@@ -22,8 +21,6 @@ interface MediaPlayer {
Pause();
// Seeks to the specified position, specified in nanoseconds.
- // TODO(dalesat): Consider adding relative vs remote flag.
- // TODO(dalesat): Consider adding parameters regarding desired precision.
Seek(int64 position);
// Gets the status. To get the status immediately, call
@@ -35,13 +32,13 @@ interface MediaPlayer {
// MediaPlayer status information.
struct MediaPlayerStatus {
- // Current state of the player.
- MediaState state;
-
// Transform translating local time to presentation time. Reverse translation
// (presentation time to local time) is only valid when media is playing.
mojo.TimelineTransform? timeline_transform;
+ // Indicates whether presentation has reached end-of-stream.
+ bool end_of_stream;
+
// Describes the media.
MediaMetadata? metadata;
};

Powered by Google App Engine
This is Rietveld 408576698