Closed Bug 1172479 Opened 5 years ago Closed 5 years ago

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

Categories

(Firefox OS Graveyard :: General, defect)

ARM
Gonk (Firefox OS)
defect
Not set

Tracking

(firefox41 fixed)

RESOLVED FIXED
Tracking Status
firefox41 --- fixed

People

(Reporter: tzimmermann, Assigned: tzimmermann)

References

Details

Attachments

(1 file)

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
Attachment #8616644 - Flags: review?(kyle) → review+
https://hg.mozilla.org/mozilla-central/rev/b6c31e8a2424
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.