Require use of AudioBus::create() to avoid ref-counting issues

Source/modules/webaudio/ConvolverNode.cpp

 /*
  * Copyright (C) 2010, Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1.  Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2.  Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in the
@@ skipping 114 matching lines @@
     size_t bufferLength = buffer->length();
 
     // The current implementation supports up to four channel impulse responses, which are interpreted as true-stereo (see Reverb class).
     bool isBufferGood = numberOfChannels > 0 && numberOfChannels <= 4 && bufferLength;
     ASSERT(isBufferGood);
     if (!isBufferGood)
         return;
 
     // Wrap the AudioBuffer by an AudioBus. It's an efficient pointer set and not a memcpy().
     // This memory is simply used in the Reverb constructor and no reference to it is kept for later use in that class.
-    AudioBus bufferBus(numberOfChannels, bufferLength, false);
+    RefPtr<AudioBus> bufferBus = AudioBus::create(numberOfChannels, bufferLength, false);
     for (unsigned i = 0; i < numberOfChannels; ++i)
-        bufferBus.setChannelMemory(i, buffer->getChannelData(i)->data(), bufferLength);
+        bufferBus->setChannelMemory(i, buffer->getChannelData(i)->data(), bufferLength);
 
-    bufferBus.setSampleRate(buffer->sampleRate());
+    bufferBus->setSampleRate(buffer->sampleRate());
 
     // Create the reverb with the given impulse response.
     bool useBackgroundThreads = !context()->isOfflineContext();
-    OwnPtr<Reverb> reverb = adoptPtr(new Reverb(&bufferBus, AudioNode::ProcessingSizeInFrames, MaxFFTSize, 2, useBackgroundThreads, m_normalize));
+    OwnPtr<Reverb> reverb = adoptPtr(new Reverb(bufferBus.get(), AudioNode::ProcessingSizeInFrames, MaxFFTSize, 2, useBackgroundThreads, m_normalize));
 
     {
         // Synchronize with process().
         MutexLocker locker(m_processLock);
         m_reverb = reverb.release();
         m_buffer = buffer;
     }
 }
 
 AudioBuffer* ConvolverNode::buffer()
 {
     ASSERT(isMainThread());
     return m_buffer.get();
 }
 
 double ConvolverNode::tailTime() const
 {
     return m_reverb ? m_reverb->impulseResponseLength() / static_cast<double>(sampleRate()) : 0;
 }
 
 double ConvolverNode::latencyTime() const
 {
     return m_reverb ? m_reverb->latencyFrames() / static_cast<double>(sampleRate()) : 0;
 }
 
 } // namespace WebCore
 
 #endif // ENABLE(WEB_AUDIO)