Add an ASSERT to verify that AudioNode::dispose() is always called

Source/modules/webaudio/AudioNode.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 40 matching lines @@
     , m_context(context)
     , m_sampleRate(sampleRate)
     , m_lastProcessingTime(-1)
     , m_lastNonSilentTime(-1)
 #if !ENABLE(OILPAN)
     , m_normalRefCount(1) // start out with normal refCount == 1 (like WTF::RefCounted class)
 #endif
     , m_connectionRefCount(0)
     , m_isMarkedForDeletion(false)
     , m_isDisabled(false)
+#ifndef NDEBUG

[tkent, 2014/08/11 10:13:06]

#if ENABLE(ASSERT)

[haraken, 2014/08/11 10:32:59]

Done.

+    , m_disposeIsCalled(false)
+#endif
     , m_channelCount(2)
     , m_channelCountMode(Max)
     , m_channelInterpretation(AudioBus::Speakers)
 {
     ScriptWrappable::init(this);
 #if ENABLE(OILPAN)
     m_context->registerLiveNode(*this);
 #endif
 #if DEBUG_AUDIONODE_REFERENCES
     if (!s_isNodeCountInitialized) {
         s_isNodeCountInitialized = true;
         atexit(AudioNode::printNodeCounts);
     }
 #endif
     ++s_instanceCount;
 }
 
 AudioNode::~AudioNode()
 {
+    ASSERT(m_disposeIsCalled);
     --s_instanceCount;
 #if DEBUG_AUDIONODE_REFERENCES
     --s_nodeCount[nodeType()];
 #if ENABLE(OILPAN)
     fprintf(stderr, "%p: %d: AudioNode::~AudioNode() %d\n", this, nodeType(), m_connectionRefCount);
 #else
     fprintf(stderr, "%p: %d: AudioNode::~AudioNode() %d %d\n", this, nodeType(), m_normalRefCount, m_connectionRefCount);
 #endif
 #endif
 }
@@ skipping 12 matching lines @@
 {
     ASSERT(isMainThread());
     ASSERT(context()->isGraphOwner());
 #if ENABLE(OILPAN)
     context()->removeAutomaticPullNode(this);
     for (unsigned i = 0; i < m_outputs.size(); ++i)
         output(i)->disconnectAll();
     m_isMarkedForDeletion = true;
 #endif
     context()->unmarkDirtyNode(*this);
+#ifndef NDEBUG

[tkent, 2014/08/11 10:13:06]

Ditto.

+    m_disposeIsCalled = true;
+#endif
 }
 
 String AudioNode::nodeTypeName() const
 {
     switch (m_nodeType) {
     case NodeTypeDestination:
         return "AudioDestinationNode";
     case NodeTypeOscillator:
         return "OscillatorNode";
     case NodeTypeAudioBufferSource:
@@ skipping 517 matching lines @@
 {
     visitor->trace(m_context);
     visitor->trace(m_inputs);
     visitor->trace(m_outputs);
     EventTargetWithInlineData::trace(visitor);
 }
 
 } // namespace blink
 
 #endif // ENABLE(WEB_AUDIO)