Closed Bug 1665225 Opened 5 years ago Closed 5 years ago

Remove unnecessary to-main-thread dispatch in WindowsSMTCProvider

Categories

(Core :: Audio/Video: Playback, enhancement, P1)

Desktop
Windows 10
enhancement

Tracking

()

RESOLVED FIXED
82 Branch
Tracking Status
firefox82 --- fixed

People

(Reporter: chunmin, Assigned: chunmin)

References

Details

Attachments

(1 file)

The callback set for IAsyncOperation in LoadImage is on the main thread actually, so there is no need to dispatch the thumbnail-setting task to main thread in that callback.

The SMTC-thumbnail-update task is dispatched to main-thread from the
on-complete callback of the async image storing operation. However, the
callback is actually on the main-thread itself so the
thumbnail-update-task can be executed in that callback directly instead
of being dispatched. The task should be finished slightly faster if the
task dispatch is replaced by executing the task.

Pushed by cchang@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/1a69e257dbd0 Remove unnecessary to-main-thread dispatch r=alwu
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → 82 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: