Fix BaseAudioContext::hasPendingActivity() to make it GCed correctly

third_party/WebKit/Source/modules/webaudio/BaseAudioContext.cpp

Index: third_party/WebKit/Source/modules/webaudio/BaseAudioContext.cpp
diff --git a/third_party/WebKit/Source/modules/webaudio/BaseAudioContext.cpp b/third_party/WebKit/Source/modules/webaudio/BaseAudioContext.cpp
index 2e72ec0778d2f4d10bedb4b47c8f0f8f00d29de2..86e23d556af8e405a9a28298f34af8343dfbb367 100644
--- a/third_party/WebKit/Source/modules/webaudio/BaseAudioContext.cpp
+++ b/third_party/WebKit/Source/modules/webaudio/BaseAudioContext.cpp
@@ -175,6 +175,7 @@ void BaseAudioContext::initialize()
 
 void BaseAudioContext::clear()
 {
+    printf("BaseAudioContext::clear\n");
     m_destinationNode.clear();
     // The audio rendering thread is dead.  Nobody will schedule AudioHandler
     // deletion.  Let's do it ourselves.
@@ -184,6 +185,7 @@ void BaseAudioContext::clear()
 
 void BaseAudioContext::uninitialize()
 {
+    printf("BaseAudioContext::uninitialize\n");
     DCHECK(isMainThread());
 
     if (!isDestinationInitialized())
@@ -208,11 +210,13 @@ void BaseAudioContext::uninitialize()
 
 void BaseAudioContext::stop()
 {
+    printf("BaseAudioContext::stop\n");
     uninitialize();
 }
 
 bool BaseAudioContext::hasPendingActivity() const
 {
+    printf("BaseAudioContext::hasPendingActivity\n");
     // There's no pending activity if the audio context has been cleared.
     return !m_isCleared;

[Raymond Toy, 2016/09/02 18:19:21]

This is different from PS1, where you were returning whether the destination had
connections or not.  Could this explain the results of this test?

 }