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

Unified Diff: chrome/browser/media/router/mojo/media_controller.mojom

Issue 2727123002: [Media Router] Custom Controls 1 - Add MediaStatus, MediaRouteController, and mojo interfaces (Closed)
Patch Set: . Created 3 years, 10 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: chrome/browser/media/router/mojo/media_controller.mojom
diff --git a/chrome/browser/media/router/mojo/media_controller.mojom b/chrome/browser/media/router/mojo/media_controller.mojom
new file mode 100644
index 0000000000000000000000000000000000000000..9d814140d5fc09a87c15e94ab1b192e76934f468
--- /dev/null
+++ b/chrome/browser/media/router/mojo/media_controller.mojom
@@ -0,0 +1,31 @@
+// Copyright 2017 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+module media_router.mojom;
+
+import "chrome/browser/media/router/mojo/media_status.mojom";
+
+// Interface for a controller to change the current state of a media content.
+// This interface should be kept free of details specific to Media Router, so
+// that it can be moved to the media namespace and be reused for other features
+// in the future.
+interface MediaController {
+ // Starts playing the media if it is paused. Is a no-op if not supported by
+ // the media or the media is already playing.
+ Play();
+ // Pauses the media if it is playing. Is a no-op if not supported by the media
imcheng 2017/03/06 23:11:41 nit: add a line break between each method for bett
takumif 2017/03/08 04:24:02 Done.
+ // or the media is already paused.
+ Pause();
+ // Mutes the media if |mute| is true, and unmutes it if false. Is a no-op if
+ // not supported by the media.
+ SetMute(bool mute);
+ // Changes the current volume of the media, with 1 being the highest and 0
+ // being the lowest/no sound. Does not change the (un)muted state of the
+ // media. Is a no-op if not supported by the media.
+ SetVolume(float volume);
+ // Sets the current playback position in milliseconds. |time| must be less
+ // than or equal to the duration of the media. Is a no-op if the media
+ // doesn't support seeking.
+ Seek(uint32 time);
dcheng 2017/03/08 01:08:19 base::TimeDelta here as well, if possible
takumif 2017/03/08 04:24:02 Done.
+};

Powered by Google App Engine
This is Rietveld 408576698