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

Side by Side Diff: content/browser/renderer_host/media/audio_input_renderer_host.h

Issue 2624403002: Refactor AudioInputController and related interfaces. (Closed)
Patch Set: Created 3 years, 11 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 (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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 // AudioInputRendererHost serves audio related requests from audio capturer 5 // AudioInputRendererHost serves audio related requests from audio capturer
6 // which lives inside the render process and provide access to audio hardware. 6 // which lives inside the render process and provide access to audio hardware.
7 // 7 //
8 // Create stream sequence (AudioInputController = AIC): 8 // Create stream sequence (AudioInputController = AIC):
9 // 9 //
10 // AudioInputHostMsg_CreateStream -> OnCreateStream -> AIC::CreateLowLatency -> 10 // AudioInputHostMsg_CreateStream -> OnCreateStream -> AIC::CreateLowLatency ->
(...skipping 90 matching lines...) Expand 10 before | Expand all | Expand 10 after
101 101
102 // BrowserMessageFilter implementation. 102 // BrowserMessageFilter implementation.
103 void OnChannelClosing() override; 103 void OnChannelClosing() override;
104 void OnDestruct() const override; 104 void OnDestruct() const override;
105 bool OnMessageReceived(const IPC::Message& message) override; 105 bool OnMessageReceived(const IPC::Message& message) override;
106 106
107 // AudioInputController::EventHandler implementation. 107 // AudioInputController::EventHandler implementation.
108 void OnCreated(media::AudioInputController* controller) override; 108 void OnCreated(media::AudioInputController* controller) override;
109 void OnError(media::AudioInputController* controller, 109 void OnError(media::AudioInputController* controller,
110 media::AudioInputController::ErrorCode error_code) override; 110 media::AudioInputController::ErrorCode error_code) override;
111 void OnData(media::AudioInputController* controller,
o1ka 2017/01/12 12:07:38 \o/ that's just what we need for audio process wor
tommi (sloooow) - chröme 2017/01/12 14:21:00 :D
112 const media::AudioBus* data) override;
113 void OnLog(media::AudioInputController* controller, 111 void OnLog(media::AudioInputController* controller,
114 const std::string& message) override; 112 const std::string& message) override;
115 113
116 // Sets the PID renderer. This is used for constructing the debug recording 114 // Sets the PID renderer. This is used for constructing the debug recording
117 // filename. 115 // filename.
118 void set_renderer_pid(int32_t renderer_pid); 116 void set_renderer_pid(int32_t renderer_pid);
119 117
120 protected: 118 protected:
121 ~AudioInputRendererHost() override; 119 ~AudioInputRendererHost() override;
122 120
(...skipping 108 matching lines...) Expand 10 before | Expand all | Expand 10 after
231 229
232 // Used to access to AudioInputDeviceManager. 230 // Used to access to AudioInputDeviceManager.
233 MediaStreamManager* media_stream_manager_; 231 MediaStreamManager* media_stream_manager_;
234 232
235 AudioMirroringManager* audio_mirroring_manager_; 233 AudioMirroringManager* audio_mirroring_manager_;
236 234
237 // A map of stream IDs to audio sources. 235 // A map of stream IDs to audio sources.
238 AudioEntryMap audio_entries_; 236 AudioEntryMap audio_entries_;
239 237
240 // Raw pointer of the UserInputMonitor. 238 // Raw pointer of the UserInputMonitor.
241 media::UserInputMonitor* user_input_monitor_; 239 media::UserInputMonitor* user_input_monitor_;
o1ka 2017/01/12 12:07:38 Should this be const?
tommi (sloooow) - chröme 2017/01/12 14:21:00 I haven't dug into this class that much yet as far
242 240
243 std::unique_ptr<media::AudioLog> audio_log_; 241 std::unique_ptr<media::AudioLog> audio_log_;
244 242
245 DISALLOW_COPY_AND_ASSIGN(AudioInputRendererHost); 243 DISALLOW_COPY_AND_ASSIGN(AudioInputRendererHost);
246 }; 244 };
247 245
248 } // namespace content 246 } // namespace content
249 247
250 #endif // CONTENT_BROWSER_RENDERER_HOST_MEDIA_AUDIO_INPUT_RENDERER_HOST_H_ 248 #endif // CONTENT_BROWSER_RENDERER_HOST_MEDIA_AUDIO_INPUT_RENDERER_HOST_H_
OLDNEW
« no previous file with comments | « no previous file | content/browser/renderer_host/media/audio_input_renderer_host.cc » ('j') | media/audio/audio_input_controller.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698