Bug 865532 - Don't attempt to stop an AudioBufferSourceNode that doesn't have a context; r=padenot

Ehsan Akhgari · Ehsan Akhgari · commit 8f22491495c5 · 2013-04-24T22:24:25.000-04:00
diff --git a/content/media/webaudio/AudioBufferSourceNode.cpp b/content/media/webaudio/AudioBufferSourceNode.cpp
@@ -527,7 +527,7 @@ AudioBufferSourceNode::Stop(double aWhen, ErrorResult& aRv)
   }
 
   AudioNodeStream* ns = static_cast<AudioNodeStream*>(mStream.get());
-  if (!ns) {
+  if (!ns || !Context()) {
     // We've already stopped and had our stream shut down
     return;
   }

Original file line number	Diff line number	Diff line change
`@@ -527,7 +527,7 @@ AudioBufferSourceNode::Stop(double aWhen, ErrorResult& aRv)`
`527`	`527`	`}`
`528`	`528`
`529`	`529`	`AudioNodeStream* ns = static_cast<AudioNodeStream*>(mStream.get());`
`530`		`- if (!ns) {`
	`530`	`+ if (!ns \|\| !Context()) {`
`531`	`531`	`// We've already stopped and had our stream shut down`
`532`	`532`	`return;`
`533`	`533`	`}`