Closed Bug 1362941 Opened 4 years ago Closed 4 years ago
Label Runnables in dom/file/File
FileReader holds FileReader::mTarget as owning event target for runnable dispatching, timercallback and AsyncWait from InputStream. File this bug to set FileReader::mTarget to aGlobal->EventTargetFor(TaskCategory::Other) at Constructor instead of do_GetCurrentThread() when creating from a window global.
The runnables for TimerCallback and AsyncWait from InputStream are done on FileReader::mEventTarget. To support DocGroup labeling, we set FileReader::mEventTarget to the one provided by nsIGlobalObject instead of the current thread. :baku, May I have your review for this change? Thanks!
Attachment #8865355 - Flags: review?(amarchesini)
Attachment #8865355 - Flags: review?(amarchesini) → review+
treeherder result looks fine: https://treeherder.mozilla.org/#/jobs?repo=try&revision=079c3ed6c4f9b295182bb07c6efc530e3afb4159
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/mozilla-inbound/rev/1d08d55ccca0 Set DocGroup-EventTarget from nsIGlobalObject to have dispatched runnables labeled. r=baku
You need to log in before you can comment on or make changes to this bug.