This runnable seems pretty easy to label since there's a document readily available.

Comment on attachment 8882004 [details] [diff] [review] patch > nsBindingManager::PostProcessAttachedQueueEvent() > { >+ MOZ_ASSERT(NS_IsMainThread()); > mProcessAttachedQueueEvent = > NewRunnableMethod("nsBindingManager::DoProcessAttachedQueue", > this, &nsBindingManager::DoProcessAttachedQueue); >- nsresult rv = NS_DispatchToCurrentThread(mProcessAttachedQueueEvent); >+ nsresult rv = mDocument->EventTargetFor(TaskCategory::Other)->Dispatch(do_AddRef(mProcessAttachedQueueEvent)); > if (NS_SUCCEEDED(rv) && mDocument) { So here we do null check for mDocument. Silly to first use mDocument and then null check it. I think you could add a null check early in the method and just return then and remove null check from the if (...mDocument) With that, r+

Pushed by wmccloskey@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/b5ca3d02cd72 Label nsBindingManager::PostProcessAttachedQueueEvent (r=smaug)