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

Side by Side Diff: content/browser/media/capture/web_contents_audio_input_stream.cc

Issue 1074963003: Clean up potentially leaky use of base::DeletePointer. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fixed another probleb Created 5 years, 8 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) 2013 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2013 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 #include "content/browser/media/capture/web_contents_audio_input_stream.h" 5 #include "content/browser/media/capture/web_contents_audio_input_stream.h"
6 6
7 #include <string> 7 #include <string>
8 8
9 #include "base/bind.h" 9 #include "base/bind.h"
10 #include "base/bind_helpers.h" 10 #include "base/bind_helpers.h"
(...skipping 320 matching lines...) Expand 10 before | Expand all | Expand 10 after
331 int main_render_frame_id; 331 int main_render_frame_id;
332 if (!WebContentsCaptureUtil::ExtractTabCaptureTarget( 332 if (!WebContentsCaptureUtil::ExtractTabCaptureTarget(
333 device_id, &render_process_id, &main_render_frame_id)) { 333 device_id, &render_process_id, &main_render_frame_id)) {
334 return NULL; 334 return NULL;
335 } 335 }
336 336
337 return new WebContentsAudioInputStream( 337 return new WebContentsAudioInputStream(
338 render_process_id, main_render_frame_id, 338 render_process_id, main_render_frame_id,
339 audio_mirroring_manager, 339 audio_mirroring_manager,
340 new WebContentsTracker(false), 340 new WebContentsTracker(false),
341 new media::VirtualAudioInputStream( 341 new media::VirtualAudioInputStream(params, worker_task_runner, false));
342 params, worker_task_runner,
343 media::VirtualAudioInputStream::AfterCloseCallback()));
344 } 342 }
345 343
346 WebContentsAudioInputStream::WebContentsAudioInputStream( 344 WebContentsAudioInputStream::WebContentsAudioInputStream(
347 int render_process_id, int main_render_frame_id, 345 int render_process_id, int main_render_frame_id,
348 AudioMirroringManager* mirroring_manager, 346 AudioMirroringManager* mirroring_manager,
349 const scoped_refptr<WebContentsTracker>& tracker, 347 const scoped_refptr<WebContentsTracker>& tracker,
350 media::VirtualAudioInputStream* mixer_stream) 348 media::VirtualAudioInputStream* mixer_stream)
351 : impl_(new Impl(render_process_id, main_render_frame_id, 349 : impl_(new Impl(render_process_id, main_render_frame_id,
352 mirroring_manager, tracker, mixer_stream)) {} 350 mirroring_manager, tracker, mixer_stream)) {}
353 351
(...skipping 34 matching lines...) Expand 10 before | Expand all | Expand 10 after
388 386
389 bool WebContentsAudioInputStream::GetAutomaticGainControl() { 387 bool WebContentsAudioInputStream::GetAutomaticGainControl() {
390 return impl_->mixer_stream()->GetAutomaticGainControl(); 388 return impl_->mixer_stream()->GetAutomaticGainControl();
391 } 389 }
392 390
393 bool WebContentsAudioInputStream::IsMuted() { 391 bool WebContentsAudioInputStream::IsMuted() {
394 return false; 392 return false;
395 } 393 }
396 394
397 } // namespace content 395 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698