Bug 1630304: Remove invalid assertion in CanvasThreadHolder::StaticRelease. r=mattwoodrow

TaskQueues hold onto their nsIEventTarget (in this case mCanvasWorkers) until
after they resolve their shutdown promise, which is what causes
CanvasThreadHolder::StaticRelease to be submitted to the compositor thread.
So this assertion can't be guaranteed.

Differential Revision: https://phabricator.services.mozilla.com/D71176
This commit is contained in:
Bob Owen 2020-04-17 04:56:35 +00:00
Родитель 0c95cd8924
Коммит f03c78362b
1 изменённых файлов: 0 добавлений и 5 удалений

Просмотреть файл

@ -81,11 +81,6 @@ void CanvasThreadHolder::StaticRelease(
auto lockedCanvasThreadHolder = sCanvasThreadHolder.Lock();
if (lockedCanvasThreadHolder.ref()->mRefCnt == 1) {
MOZ_ASSERT(lockedCanvasThreadHolder.ref()
->mCanvasWorkers.forget()
.take()
->Release() == 0,
"There should be no other references to mCanvasWorkers.");
lockedCanvasThreadHolder.ref() = nullptr;
}
}