946865 - LoadMonitor::Init uses preferences off the main thread.

Status: RESOLVED FIXED

(Core :: WebRTC: Audio/Video, defect)

Description

[Kyle Huey (Exited; not receiving bugmail, old account, do not use)]

Randall, can you find an owner for this?

Comment 1

[Maire Reavy [:mreavy]]

Gian-Carlo -- Can you take this one?   (I believe the regression was introduced in your patch from bug 902000.)  Thanks.

Comment 2

[Gian-Carlo Pascutto [:gcp]]

Attachment: Patch 1. Move Load adaption pref to main thread

Comment 3

[Randell Jesup [:jesup] (needinfo me)]

Comment on attachment 8343809 [details] [diff] [review]
Patch 1. Move Load adaption pref to main thread

Review of attachment 8343809 [details] [diff] [review]:
-----------------------------------------------------------------

Note that there will likely be a minor bitrot against bug 945614/bug 853356

Comment 4

[Gian-Carlo Pascutto [:gcp]]

Looks good on try:
https://tbpl.mozilla.org/?tree=Try&rev=fe1c8481f493

Sorry jesup :)

https://hg.mozilla.org/integration/mozilla-inbound/rev/d701e36c7ca8

Comment 5

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

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