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

Side by Side Diff: content/browser/media/audio_output_impl.cc

Issue 1930393002: Switch stream creation and closing in Chrome audio rendering from IPC to Mojo (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebase + tests in progress 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 #include "content/browser/media/audio_output_impl.h"
6 #include "content/browser/media/audio_output_stream_impl.h"
7
8 namespace content {
9
10 AudioOutputImpl::AudioOutputImpl(
11 scoped_refptr<AudioRendererHost> audio_renderer_host,
12 int render_frame_id,
13 media::mojom::AudioOutputRequest request)
14 : audio_renderer_host_(audio_renderer_host.get()),
15 render_frame_id_(render_frame_id),
16 binding_(this) {
17 DCHECK_CURRENTLY_ON(BrowserThread::IO);
18 binding_.Bind(std::move(request));
19 LOG(ERROR) << "AudioOutputImpl";
20 factory_ = base::Bind(AudioOutputStreamFactory::Factory);
21 }
22
23 AudioOutputImpl::~AudioOutputImpl() {
24 LOG(ERROR) << "~AudioOutputImpl";
25 }
26
27 // static
28 void AudioOutputImpl::CreateService(
29 scoped_refptr<AudioRendererHost> audio_renderer_host,
30 int render_frame_id,
31 media::mojom::AudioOutputRequest request) {
32 DCHECK_CURRENTLY_ON(BrowserThread::UI);
33
34 BrowserThread::PostTask(
35 BrowserThread::IO, FROM_HERE,
36 base::Bind(&AudioOutputImpl::CreateServiceOnIOThread, audio_renderer_host,
37 render_frame_id, base::Passed(&request)));
38 }
39
40 // static
41 void AudioOutputImpl::CreateServiceOnIOThread(
42 scoped_refptr<AudioRendererHost> audio_renderer_host,
43 int render_frame_id,
44 media::mojom::AudioOutputRequest request) {
45 if (request.is_pending()) {
46 DCHECK_CURRENTLY_ON(BrowserThread::IO);
47 }
48
49 auto service = new AudioOutputImpl(audio_renderer_host, render_frame_id,
50 std::move(request));
51
52 audio_renderer_host->set_audio_output_impl(render_frame_id, service);
53 }
54
55 void AudioOutputImpl::CreateStream(int stream_id,
56 const media::AudioParameters& params,
57 const CreateStreamCallback& callback) {
58 DCHECK_CURRENTLY_ON(BrowserThread::IO);
59 media::mojom::AudioOutputStreamPtr stream;
60 audio_renderer_host_->CreateStream(stream_id, render_frame_id_, params,
61 callback);
62 }
63
64 media::mojom::AudioOutputStreamPtr AudioOutputImpl::StreamFactory(
65 int stream_id,
66 int render_frame_id,
67 AudioRendererHost* audio_renderer_host) {
68 DCHECK_CURRENTLY_ON(BrowserThread::IO);
69
70 media::mojom::AudioOutputStreamPtr stream =
71 media::mojom::AudioOutputStreamPtr();
72 std::unique_ptr<AudioOutputStreamImpl> stream_ptr(
73 factory_.Run(mojo::GetProxy(&stream), stream_id, render_frame_id,
74 audio_renderer_host));
75
76 auto ret =
77 stream_impls_.insert(std::make_pair(stream_id, std::move(stream_ptr)));
78 DCHECK(ret.second);
79 return stream;
80 }
81
82 bool AudioOutputImpl::RemoveStream(int stream_id) {
83 LOG(ERROR) << "rm stream_id" << stream_id;
84 if (stream_impls_.erase(stream_id))
85 return true;
86 else
87 return false;
88 }
89
90 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698