1640243 - Convert 'ReadAheadDlls` thread to background thread

Status: RESOLVED FIXED

(Core :: Widget: Win32, enhancement, P5)

Description

[Emma Malysz]

Instead of using PR_CreateThread (https://searchfox.org/mozilla-central/source/toolkit/xre/nsAppRunner.cpp#4303-4305), we should consider moving this to NS_DispatchBackgroundTask.

Comment 1

[Oliver Old [:oold]]

Hello, I would like to work on this bug. I have actually already prepared the fix.

Comment 2

[Ray Kraesig [:rkraesig]]

(In reply to Oliver Old from comment #1)

Hello, I would like to work on this bug. I have actually already prepared the fix.

Excellent! All you need to do is upload the fix to Phabricator; this bug will be automatically assigned to you once you do. For the reviewer you can select the #win-reviewers group.

For reference, although you've probably already seen it, our page for new contributors is here.

Comment 3

[Oliver Old [:oold]]

Attachment: Bug 1640243 - Convert ReadAheadDlls() thread to background task. r=#win-reviewers

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/e31949c01c4e
Convert ReadAheadDlls() thread to background task. r=win-reviewers,rkraesig

Comment 5

[agoloman]

https://hg.mozilla.org/mozilla-central/rev/e31949c01c4e

Comment 6

[Ray Kraesig [:rkraesig]]

Hope to see you around on future bugs. Even if not, thanks for the patch!