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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: media/mojo/interfaces/renderer.mojom

Issue 2230583002: Add MediaPlayerRenderer/MediaPlayerRendererClient (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fixed Chris' comment Created 4 years, 4 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
« media/mojo/clients/mojo_renderer.h ('K') | « media/mojo/clients/mojo_renderer.cc ('k') | media/mojo/services/BUILD.gn » ('j') | media/mojo/services/mojo_renderer_service.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/mojo/interfaces/renderer.mojom
diff --git a/media/mojo/interfaces/renderer.mojom b/media/mojo/interfaces/renderer.mojom
index de14441142ba279003e4cdddc1b07cfc8c77d06b..f3158910e9bfefbcd67ec580c1fd458620bf7fb3 100644
--- a/media/mojo/interfaces/renderer.mojom
+++ b/media/mojo/interfaces/renderer.mojom
@@ -7,13 +7,15 @@ module media.mojom;
import "media/mojo/interfaces/demuxer_stream.mojom";
import "media/mojo/interfaces/media_types.mojom";
import "ui/gfx/geometry/mojo/geometry.mojom";
+import "url/mojo/url.mojom";
interface Renderer {
// Initializes the Renderer with one or both of an audio and video stream,
// executing the callback with whether the initialization succeeded.
Initialize(RendererClient client,
DemuxerStream? audio,
- DemuxerStream? video) => (bool success);
+ DemuxerStream? video,
+ url.mojom.Url url) => (bool success);
DaleCurtis 2016/08/10 21:14:06 ? for url parameter ?
tguilbert 2016/08/11 00:29:00 Added, and updated to use the proper base::Optiona
// Discards any buffered data, executing callback when completed.
// NOTE: If an error occurs, RendererClient::OnError() can be called
« media/mojo/clients/mojo_renderer.h ('K') | « media/mojo/clients/mojo_renderer.cc ('k') | media/mojo/services/BUILD.gn » ('j') | media/mojo/services/mojo_renderer_service.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698