Closed
Bug 1172479
Opened 9 years ago
Closed 9 years ago
Replace |nsIThread| by |MessageLoop| in socket I/O code
Categories
(Firefox OS Graveyard :: General, defect)
Tracking
(firefox41 fixed)
RESOLVED
FIXED
Tracking | Status | |
---|---|---|
firefox41 | --- | fixed |
People
(Reporter: tzimmermann, Assigned: tzimmermann)
References
Details
Attachments
(1 file)
58.56 KB,
patch
|
qdot
:
review+
|
Details | Diff | Splinter Review |
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 | ||
Updated•9 years ago
|
Assignee: nobody → tzimmermann
Status: NEW → ASSIGNED
Assignee | ||
Comment 1•9 years ago
|
||
Attachment #8616644 -
Flags: review?(kyle)
Assignee | ||
Comment 2•9 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=39e71fb9896d
Updated•9 years ago
|
Attachment #8616644 -
Flags: review?(kyle) → review+
Assignee | ||
Comment 4•9 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=b2g-inbound&revision=b6c31e8a2424
You need to log in
before you can comment on or make changes to this bug.
Description
•