Oilpan: WebAudio: Apply the weak HashMap pattern to remove an entry from AudioContext::m_dirtyAudio…

Source/modules/webaudio/AudioSummingJunction.cpp

Index: Source/modules/webaudio/AudioSummingJunction.cpp
diff --git a/Source/modules/webaudio/AudioSummingJunction.cpp b/Source/modules/webaudio/AudioSummingJunction.cpp
index 91b9e9104254668e7e2b85bc218ffc372d7a2a56..97d84f37402c8d5e80faecc5c1ad08bcc706e692 100644
--- a/Source/modules/webaudio/AudioSummingJunction.cpp
+++ b/Source/modules/webaudio/AudioSummingJunction.cpp
@@ -38,14 +38,18 @@ AudioSummingJunction::AudioSummingJunction(AudioContext* context)
     : m_context(context)
     , m_renderingStateNeedUpdating(false)
 {
+    ASSERT(context);
+#if ENABLE(OILPAN)
+    m_context->registerLiveAudioSummingJunction(*this);

[haraken, 2014/07/18 10:01:01]

Instead of registering an AudioSummingJunction when it's created, would it be
possible to only register the AudioSummingJunction when
markSummingJunctionDirty() is called?

What m_liveAudioSummingJunctions has to manage is a list of
AudioSummingJunctions that are in m_dirtySummingJunctions (not a list of all
AudioSummingJunctions).

[tkent, 2014/07/18 13:43:09]

We can't do it.  markSummingJunctionDirty can be called in audio thread, we want
to avoid multi-thread access to m_liveAudioSummingJunctions.

[haraken, 2014/07/18 13:49:01]

Makes sense. Probably we can add a comment and mention that.
m_liveAudioSummingJunctions can contain AudioSummingJunctions that are not in
m_dirtySummingJunctions. Thus we can m_dirtySummingJunctions.remove() for
AudioSummingJunctions that are not contained in the m_dirtySummingJunctions, but
that's no problem.

+#endif
 }
 
 AudioSummingJunction::~AudioSummingJunction()
 {
-    // Oilpan: m_context is null if AudioContext and this object die
-    // together. It's non-null if this object dies before AudioContext.
+#if !ENABLE(OILPAN)
     if (m_renderingStateNeedUpdating && m_context.get())
         m_context->removeMarkedSummingJunction(this);
+#endif
 }
 
 void AudioSummingJunction::trace(Visitor* visitor)