Don't run removeFinishedSourceNodes with the context lock.

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

 /*
  * Copyright (C) 2012, 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 18 matching lines @@
 #include "bindings/core/v8/ScriptState.h"
 #include "core/dom/DOMException.h"
 #include "core/dom/Document.h"
 #include "core/dom/ExceptionCode.h"
 #include "core/dom/ExecutionContext.h"
 #include "modules/webaudio/AudioListener.h"
 #include "modules/webaudio/DeferredTaskHandler.h"
 #include "modules/webaudio/OfflineAudioCompletionEvent.h"
 #include "modules/webaudio/OfflineAudioDestinationNode.h"
 
+#include "platform/CrossThreadFunctional.h"
 #include "platform/Histogram.h"
 #include "platform/audio/AudioUtilities.h"
+#include "public/platform/Platform.h"
 
 namespace blink {
 
 OfflineAudioContext* OfflineAudioContext::create(
     ExecutionContext* context,
     unsigned numberOfChannels,
     unsigned numberOfFrames,
     float sampleRate,
     ExceptionState& exceptionState) {
   // FIXME: add support for workers.
@@ skipping 316 matching lines @@
   handleStoppableSourceNodes();
 
   return shouldSuspend();
 }
 
 void OfflineAudioContext::handlePostOfflineRenderTasks() {
   DCHECK(isAudioThread());
 
   // OfflineGraphAutoLocker here locks the audio graph for the same reason
   // above in |handlePreOfflineRenderTasks|.
-  OfflineGraphAutoLocker locker(this);
+  bool didRemove = false;
+  {
+    OfflineGraphAutoLocker locker(this);
 
-  deferredTaskHandler().breakConnections();
-  releaseFinishedSourceNodes();
-  deferredTaskHandler().handleDeferredTasks();
-  deferredTaskHandler().requestToDeleteHandlersOnMainThread();
+    deferredTaskHandler().breakConnections();
+    didRemove = releaseFinishedSourceNodes();
+    deferredTaskHandler().handleDeferredTasks();
+    deferredTaskHandler().requestToDeleteHandlersOnMainThread();
+  }
+
+  removeFinishedSourceNodes(didRemove);
 }
 
 OfflineAudioDestinationHandler& OfflineAudioContext::destinationHandler() {
   return static_cast<OfflineAudioDestinationHandler&>(
       destination()->audioDestinationHandler());
 }
 
 void OfflineAudioContext::resolveSuspendOnMainThread(size_t frame) {
   DCHECK(isMainThread());
 
@@ skipping 41 matching lines @@
 
   // Note that the GraphLock is required before this check. Since this needs
   // to run on the audio thread, OfflineGraphAutoLocker must be used.
   if (m_scheduledSuspends.contains(currentSampleFrame()))
     return true;
 
   return false;
 }
 
 }  // namespace blink