1172479 - Replace |nsIThread| by |MessageLoop| in socket I/O code

Status: RESOLVED FIXED

(Firefox OS Graveyard :: General, defect)

Description

[Thomas Zimmermann [:tzimmermann] [:tdz]]

While working on bug 1171994, I found that it's not possible to easily use an |nsIThread| of a worker in the socket I/O code. Dispatching socket events to a worker requires extra setup on the worker and consequently fails at [1].

Fortunately, we don't need nsIThread in the socket I/O code. We can simply use |MessageLoop| for passing messages to a consumer thread.

[1] https://dxr.mozilla.org/mozilla-central/source/dom/workers/WorkerThread.cpp#230

Comment 1

[Thomas Zimmermann [:tzimmermann] [:tdz]]

Attachment: [01] Bug 1172479: Replace |nsIThread| by |MessageLoop| in socket I/O code

Comment 2

[Thomas Zimmermann [:tzimmermann] [:tdz]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=39e71fb9896d

Comment 3

[Pulsebot]

https://hg.mozilla.org/integration/b2g-inbound/rev/b6c31e8a2424

Comment 4

[Thomas Zimmermann [:tzimmermann] [:tdz]]

https://treeherder.mozilla.org/#/jobs?repo=b2g-inbound&revision=b6c31e8a2424

Comment 5

[Carsten Book [:Tomcat]]

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