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

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

Issue 1255083004: media: Mojo interfaces renaming. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 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 unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 module media.interfaces; 5 module media.interfaces;
6 6
7 import "media/mojo/interfaces/media_types.mojom"; 7 import "media/mojo/interfaces/media_types.mojom";
8 8
9 // DemuxerStream is modeled after media::DemuxerStream using mojo in order to 9 // DemuxerStream is modeled after media::DemuxerStream using mojo in order to
10 // enable proxying between a media::Pipeline and media::Renderer living in two 10 // enable proxying between a media::Pipeline and media::Renderer living in two
(...skipping 17 matching lines...) Expand all
28 // Initializes the DemuxerStream. Read() can only be called after the callback 28 // Initializes the DemuxerStream. Read() can only be called after the callback
29 // is received. The returned |pipe| will be used to fill out the data section 29 // is received. The returned |pipe| will be used to fill out the data section
30 // of the media::DecoderBuffer returned via DemuxerStream::Read(). Only the 30 // of the media::DecoderBuffer returned via DemuxerStream::Read(). Only the
31 // config for |type| should be non-null, which is the initial config of the 31 // config for |type| should be non-null, which is the initial config of the
32 // stream. 32 // stream.
33 Initialize() => (Type type, 33 Initialize() => (Type type,
34 handle<data_pipe_consumer> pipe, 34 handle<data_pipe_consumer> pipe,
35 AudioDecoderConfig? audio_config, 35 AudioDecoderConfig? audio_config,
36 VideoDecoderConfig? video_config); 36 VideoDecoderConfig? video_config);
37 37
38 // Requests a MediaDecoderBuffer from this stream for decoding and rendering. 38 // Requests a DecoderBuffer from this stream for decoding and rendering.
39 // See media::DemuxerStream::ReadCB for a general explanation of the fields. 39 // See media::DemuxerStream::ReadCB for a general explanation of the fields.
40 // 40 //
41 // Notes on the callback: 41 // Notes on the callback:
42 // - If |status| is OK, |buffer| should be non-null and clients must fill out 42 // - If |status| is OK, |buffer| should be non-null and clients must fill out
43 // the data section of the returned media::DecoderBuffer by reading from 43 // the data section of the returned media::DecoderBuffer by reading from
44 // the |pipe| provided during Initialize(). 44 // the |pipe| provided during Initialize().
45 // - If |status| is ABORTED, all other fields should be null. 45 // - If |status| is ABORTED, all other fields should be null.
46 // - If |status| is CONFIG_CHANGED, the config for the stream type should be 46 // - If |status| is CONFIG_CHANGED, the config for the stream type should be
47 // non-null. 47 // non-null.
48 // 48 //
49 // TODO(dalecurtis): Remove this method in favor of serializing everything 49 // TODO(dalecurtis): Remove this method in favor of serializing everything
50 // into the DataPipe given to Initialize() once DataPipe supports framed data 50 // into the DataPipe given to Initialize() once DataPipe supports framed data
51 // in a nicer fashion. 51 // in a nicer fashion.
52 Read() => (Status status, 52 Read() => (Status status,
53 MediaDecoderBuffer? buffer, 53 DecoderBuffer? buffer,
54 AudioDecoderConfig? audio_config, 54 AudioDecoderConfig? audio_config,
55 VideoDecoderConfig? video_config); 55 VideoDecoderConfig? video_config);
56 }; 56 };
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698