Bug 1148102: P1. Mark WebMBufferedParser refcounting as thread-safe. r=kinetik
authorJean-Yves Avenard <jyavenard@mozilla.com>
Fri, 10 Jul 2015 16:08:13 +1000
changeset 253668 3dae3b9b3ec106f292cf15287dc6d09110eeba36
parent 253667 192cf71f768f87d79094ffe1b2d319ea9505b89d
child 253669 a535f9fa0da5d040c61d81ef1a9a5973e2bef0e1
push id29074
push userryanvm@gmail.com
push dateMon, 20 Jul 2015 19:51:26 +0000
treeherdermozilla-central@b86e3ab5e974 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewerskinetik
bugs1148102
milestone42.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1148102: P1. Mark WebMBufferedParser refcounting as thread-safe. r=kinetik This is required should we use WebMBufferedParser in a MediaTaskQueue as we don't know which actual thread will actually be used.
dom/media/webm/WebMBufferedParser.h
--- a/dom/media/webm/WebMBufferedParser.h
+++ b/dom/media/webm/WebMBufferedParser.h
@@ -216,17 +216,17 @@ private:
 
   // True if we read the timecode scale from the segment info or have
   // confirmed that the default value is to be used.
   bool mGotTimecodeScale;
 };
 
 class WebMBufferedState final
 {
-  NS_INLINE_DECL_REFCOUNTING(WebMBufferedState)
+  NS_INLINE_DECL_THREADSAFE_REFCOUNTING(WebMBufferedState)
 
 public:
   WebMBufferedState() : mReentrantMonitor("WebMBufferedState") {
     MOZ_COUNT_CTOR(WebMBufferedState);
   }
 
   void NotifyDataArrived(const unsigned char* aBuffer, uint32_t aLength, int64_t aOffset);
   bool CalculateBufferedForRange(int64_t aStartOffset, int64_t aEndOffset,