Bug 1631304 - Don't lazily create a TailDispatcher from MaybeDrainDirectTasks. r=jya

bholley · bholley · commit b5422991d2a1 · 2020-04-28T21:18:15.000Z
Differential Revision: https://phabricator.services.mozilla.com/D72262
diff --git a/xpcom/threads/TaskDispatcher.h b/xpcom/threads/TaskDispatcher.h
@@ -206,7 +206,7 @@ class AutoTaskDispatcher : public TaskDispatcher {
    private:
     void MaybeDrainDirectTasks() {
       AbstractThread* currentThread = AbstractThread::GetCurrent();
-      if (currentThread) {
+      if (currentThread && currentThread->MightHaveTailTasks()) {
         currentThread->TailDispatcher().DrainDirectTasks();
       }
     }

Original file line number	Diff line number	Diff line change
`@@ -206,7 +206,7 @@ class AutoTaskDispatcher : public TaskDispatcher {`
`206`	`206`	`private:`
`207`	`207`	`void MaybeDrainDirectTasks() {`
`208`	`208`	`AbstractThread* currentThread = AbstractThread::GetCurrent();`
`209`		`- if (currentThread) {`
	`209`	`+ if (currentThread && currentThread->MightHaveTailTasks()) {`
`210`	`210`	`currentThread->TailDispatcher().DrainDirectTasks();`
`211`	`211`	`}`
`212`	`212`	`}`