Magic-Mirror/gecko-dev
1
0
Fork 0
mirror of https://github.com/mozilla/gecko-dev.git synced 2024-10-22 01:35:35 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Bug 1409274 - remove the lock for SourceBufferTaskQueue. r=jya

Browse Source 
It would only be accessed on task queue now, so we don't need to lock it.

MozReview-Commit-ID: 6jd36TQW4aA

--HG--
extra : rebase_source : e8bb53a226154312496149ab8f6b00dead49a3b6
This commit is contained in:
Alastor Wu 2017-10-17 15:46:26 +08:00
parent 6d1f8863a7
commit def6113401
1 changed files with 2 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
dom/media/mediasource/TrackBuffersManager.h
View File

@ -35,9 +35,8 @@ class SourceBufferResource;
class SourceBufferTaskQueue
{
public:
SourceBufferTaskQueue()
: mMutex("SourceBufferTaskQueue")
{}
SourceBufferTaskQueue() { }
~SourceBufferTaskQueue()
{
MOZ_ASSERT(mQueue.IsEmpty(), "All tasks must have been processed");
@ -45,13 +44,11 @@ public:
void Push(SourceBufferTask* aTask)
{
MutexAutoLock mut(mMutex);
mQueue.AppendElement(aTask);
}
already_AddRefed<SourceBufferTask> Pop()
{
MutexAutoLock mut(mMutex);
if (!mQueue.Length()) {
return nullptr;
}
@ -62,12 +59,9 @@ public:
nsTArray<SourceBufferTask>::size_type Length() const
{
MutexAutoLock mut(mMutex);
return mQueue.Length();
}
private:
mutable Mutex mMutex;
nsTArray<RefPtr<SourceBufferTask>> mQueue;
};