Bug 1341666 - Allow running a `close` message during an MSG shutdown. r=jesup
☠☠ backed out by 2c3b473b6247 ☠ ☠
authorPaul Adenot <paul@paul.cx>
Tue, 07 Mar 2017 16:50:37 +0100
changeset 500859 62f80de05fa2039bc9be0e9cfed70d420e49ffbb
parent 500858 5800d1391c10dcd268421a374bde6a514f8dcd10
child 500860 937c01cb5e551a7bd9377ab6450646f71d0e3a47
child 500863 2c3b473b6247603ffe01f721f9579a4982674caa
child 502479 a9065e0014df7ac3420a930227698e9b2c90f3e5
push id49816
push userbmo:tchiovoloni@mozilla.com
push dateFri, 17 Mar 2017 20:44:02 +0000
reviewersjesup
bugs1341666
milestone55.0a1
Bug 1341666 - Allow running a `close` message during an MSG shutdown. r=jesup MozReview-Commit-ID: Hqg0porLEXT
dom/media/MediaStreamGraph.cpp
--- a/dom/media/MediaStreamGraph.cpp
+++ b/dom/media/MediaStreamGraph.cpp
@@ -4004,17 +4004,17 @@ MediaStreamGraph::ApplyAudioContextOpera
     }
     void Run() override
     {
       mStream->GraphImpl()->ApplyAudioContextOperationImpl(mStream,
         mStreams, mAudioContextOperation, mPromise);
     }
     void RunDuringShutdown() override
     {
-      MOZ_ASSERT(false, "We should be reviving the graph?");
+      MOZ_ASSERT(aOperation == AudioContextOperation::Close, "We should be reviving the graph?");
     }
 
   private:
     // We don't need strong references here for the same reason ControlMessage
     // doesn't.
     nsTArray<MediaStream*> mStreams;
     AudioContextOperation mAudioContextOperation;
     void* mPromise;