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

Side by Side Diff: Source/modules/webaudio/MediaStreamAudioDestinationNode.cpp

Issue 14628008: Require use of AudioBus::create() to avoid ref-counting issues (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Created 7 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
1 /* 1 /*
2 * Copyright (C) 2012, Google Inc. All rights reserved. 2 * Copyright (C) 2012, Google Inc. All rights reserved.
3 * 3 *
4 * Redistribution and use in source and binary forms, with or without 4 * Redistribution and use in source and binary forms, with or without
5 * modification, are permitted provided that the following conditions 5 * modification, are permitted provided that the following conditions
6 * are met: 6 * are met:
7 * 1. Redistributions of source code must retain the above copyright 7 * 1. Redistributions of source code must retain the above copyright
8 * notice, this list of conditions and the following disclaimer. 8 * notice, this list of conditions and the following disclaimer.
9 * 2. Redistributions in binary form must reproduce the above copyright 9 * 2. Redistributions in binary form must reproduce the above copyright
10 * notice, this list of conditions and the following disclaimer in the 10 * notice, this list of conditions and the following disclaimer in the
(...skipping 27 matching lines...) Expand all
38 38
39 namespace WebCore { 39 namespace WebCore {
40 40
41 PassRefPtr<MediaStreamAudioDestinationNode> MediaStreamAudioDestinationNode::cre ate(AudioContext* context, size_t numberOfChannels) 41 PassRefPtr<MediaStreamAudioDestinationNode> MediaStreamAudioDestinationNode::cre ate(AudioContext* context, size_t numberOfChannels)
42 { 42 {
43 return adoptRef(new MediaStreamAudioDestinationNode(context, numberOfChannel s)); 43 return adoptRef(new MediaStreamAudioDestinationNode(context, numberOfChannel s));
44 } 44 }
45 45
46 MediaStreamAudioDestinationNode::MediaStreamAudioDestinationNode(AudioContext* c ontext, size_t numberOfChannels) 46 MediaStreamAudioDestinationNode::MediaStreamAudioDestinationNode(AudioContext* c ontext, size_t numberOfChannels)
47 : AudioBasicInspectorNode(context, context->sampleRate(), numberOfChannels) 47 : AudioBasicInspectorNode(context, context->sampleRate(), numberOfChannels)
48 , m_mixBus(numberOfChannels, ProcessingSizeInFrames) 48 , m_mixBus(new AudioBus(numberOfChannels, ProcessingSizeInFrames))
Chris Rogers 2013/05/02 17:48:46 adoptRef()
49 { 49 {
50 setNodeType(NodeTypeMediaStreamAudioDestination); 50 setNodeType(NodeTypeMediaStreamAudioDestination);
51 51
52 m_source = MediaStreamSource::create(ASCIILiteral("WebAudio-") + createCanon icalUUIDString(), MediaStreamSource::TypeAudio, "MediaStreamAudioDestinationNode ", MediaStreamSource::ReadyStateLive, true); 52 m_source = MediaStreamSource::create(ASCIILiteral("WebAudio-") + createCanon icalUUIDString(), MediaStreamSource::TypeAudio, "MediaStreamAudioDestinationNode ", MediaStreamSource::ReadyStateLive, true);
53 MediaStreamSourceVector audioSources; 53 MediaStreamSourceVector audioSources;
54 audioSources.append(m_source); 54 audioSources.append(m_source);
55 MediaStreamSourceVector videoSources; 55 MediaStreamSourceVector videoSources;
56 m_stream = MediaStream::create(context->scriptExecutionContext(), MediaStrea mDescriptor::create(audioSources, videoSources)); 56 m_stream = MediaStream::create(context->scriptExecutionContext(), MediaStrea mDescriptor::create(audioSources, videoSources));
57 MediaStreamCenter::instance().didCreateMediaStream(m_stream->descriptor()); 57 MediaStreamCenter::instance().didCreateMediaStream(m_stream->descriptor());
58 58
59 m_source->setAudioFormat(numberOfChannels, context->sampleRate()); 59 m_source->setAudioFormat(numberOfChannels, context->sampleRate());
60 60
61 initialize(); 61 initialize();
62 } 62 }
63 63
64 MediaStreamSource* MediaStreamAudioDestinationNode::mediaStreamSource() 64 MediaStreamSource* MediaStreamAudioDestinationNode::mediaStreamSource()
65 { 65 {
66 return m_source.get(); 66 return m_source.get();
67 } 67 }
68 68
69 MediaStreamAudioDestinationNode::~MediaStreamAudioDestinationNode() 69 MediaStreamAudioDestinationNode::~MediaStreamAudioDestinationNode()
70 { 70 {
71 uninitialize(); 71 uninitialize();
72 } 72 }
73 73
74 void MediaStreamAudioDestinationNode::process(size_t numberOfFrames) 74 void MediaStreamAudioDestinationNode::process(size_t numberOfFrames)
75 { 75 {
76 m_mixBus.copyFrom(*input(0)->bus()); 76 m_mixBus->copyFrom(*input(0)->bus());
77 m_source->consumeAudio(&m_mixBus, numberOfFrames); 77 m_source->consumeAudio(m_mixBus.get(), numberOfFrames);
78 } 78 }
79 79
80 void MediaStreamAudioDestinationNode::reset() 80 void MediaStreamAudioDestinationNode::reset()
81 { 81 {
82 } 82 }
83 83
84 } // namespace WebCore 84 } // namespace WebCore
85 85
86 #endif // ENABLE(WEB_AUDIO) && ENABLE(MEDIA_STREAM) 86 #endif // ENABLE(WEB_AUDIO) && ENABLE(MEDIA_STREAM)
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698