bug 1242268 document mLifecycleState multithread access management r?padenot draft
authorKarl Tomlinson <karlt+@karlt.net>
Mon, 04 Jul 2016 16:33:05 +1200
changeset 383471 7a39150f43249e0f398cd8f294f42df6e92d2655
parent 383328 39dffbba764210b25bfc1e749b4f16db77fa0d46
child 383472 623c04b4029848e25bebdbfd1c4da7f39dd0ab29
push id22038
push userktomlinson@mozilla.com
push dateMon, 04 Jul 2016 09:07:50 +0000
reviewerspadenot
bugs1242268
milestone50.0a1
bug 1242268 document mLifecycleState multithread access management r?padenot MozReview-Commit-ID: 1479atSNipN
dom/media/MediaStreamGraphImpl.h
--- a/dom/media/MediaStreamGraphImpl.h
+++ b/dom/media/MediaStreamGraphImpl.h
@@ -738,16 +738,19 @@ public:
     // to shut down the graph thread(s).
     LIFECYCLE_WAITING_FOR_THREAD_SHUTDOWN,
     // Graph threads have shut down but we're waiting for remaining streams
     // to be destroyed. Only happens during application shutdown and on
     // completed non-realtime graphs, since normally we'd only shut down a
     // realtime graph when it has no streams.
     LIFECYCLE_WAITING_FOR_STREAM_DESTRUCTION
   };
+  /**
+   * Modified only on the main thread in mMonitor.
+   */
   LifecycleState mLifecycleState;
   /**
    * The graph should stop processing at or after this time.
    */
   GraphTime mEndTime;
 
   /**
    * True when we need to do a forced shutdown during application shutdown.