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

Side by Side Diff: media/mojo/interfaces/audio_output.mojom

Issue 1896883002: Mojo interfaces needed for switching audio rendering stream creation and closing from IPC to Mojo (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: move interface to media/use convertor rebase 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 unified diff | Download patch
OLDNEW
(Empty)
1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 module media.mojom;
6
7 import "media/mojo/interfaces/audio_parameters.mojom";
8
9 // This interface handles audio output stream operations.
10 // It allows to close a stream.
11 // TODO(rchtara): Add methods that allow the interaction with audio output
12 // streams: Play, Pause and SetVolume to this interface.
13 // See crbug.com/606707 for more details.
14 interface AudioOutputStream {
15 Close();
16 };
17
18 // This interface manages audio output streams.
19 // It allows to create an AudioOutputStream.
20 // TODO(rchtara): Add method to request a device authorization to this
Henrik Grunell 2016/05/12 10:34:37 Nit: "request device authorization"
rchtara 2016/05/13 12:00:09 Done.
21 // interface.
22 // See crbug.com/606707 for more details.
23 interface AudioOutput {
24 // TODO(rchtara): Remove |stream_id| from AudioOutput::CreateStream when all
25 // the stream operations are mojofied.
26 CreateStream(
27 int32 stream_id,
28 media.interfaces.AudioParameters params) =>
29 (int32 stream_id,
30 AudioOutputStream? stream,
31 handle<shared_buffer>? shared_buffer,
32 handle? socket_descriptor);
33 // Used to close streams which are just authorized but not created yet.
34 CloseStream(int32 stream_id);
Henrik Grunell 2016/05/12 10:34:37 I suppose there should only be one way of closing
rchtara 2016/05/13 12:00:09 After a discussion with grunell: we decided to kee
35 };
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698