All users were logged out of Bugzilla on October 13th, 2018

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

RESOLVED FIXED

Status

RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: tzimmermann, Assigned: tzimmermann)

Tracking

unspecified
ARM
Gonk (Firefox OS)
Dependency tree / graph

Firefox Tracking Flags

(firefox41 fixed)

Details

Attachments

(1 attachment)

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
Assignee: nobody → tzimmermann
Status: NEW → ASSIGNED
Blocks: 1171994
Created attachment 8616644 [details] [diff] [review]
[01] Bug 1172479: Replace |nsIThread| by |MessageLoop| in socket I/O code
Attachment #8616644 - Flags: review?(kyle)
Blocks: 1146822
Blocks: 1171017
https://hg.mozilla.org/mozilla-central/rev/b6c31e8a2424
Status: ASSIGNED → RESOLVED
Last Resolved: 3 years ago
status-firefox41: --- → fixed
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.