Last Comment Bug 598869 - Allow migrating AsyncChannels to a different thread from the one on which they were created
: Allow migrating AsyncChannels to a different thread from the one on which the...
Status: RESOLVED FIXED
:
Product: Core
Classification: Components
Component: IPC (show other bugs)
: unspecified
: x86_64 Linux
: -- normal (vote)
: ---
Assigned To: Chris Jones [:cjones] inactive; ni?/f?/r? if you need me
:
:
Mentors:
Depends on: 613442
Blocks: 562770 598868
  Show dependency treegraph
 
Reported: 2010-09-23 00:57 PDT by Chris Jones [:cjones] inactive; ni?/f?/r? if you need me
Modified: 2011-06-03 11:35 PDT (History)
5 users (show)
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---
-


Attachments
Test that using actors on non-main threads works (7.95 KB, patch)
2010-11-24 16:20 PST, Chris Jones [:cjones] inactive; ni?/f?/r? if you need me
no flags Details | Diff | Splinter Review

Description Chris Jones [:cjones] inactive; ni?/f?/r? if you need me 2010-09-23 00:57:23 PDT
We'll want this for video decoder threads.  Essentially, on the main thread we'll want to create a bridge channel PMedia between PContent and PCompositor, or a direct PMedia channel to the chrome process when there's no compositor process (like with fennec).  Then we want to migrate this channel to a video decoder thread.
Comment 1 Matt Brubeck (:mbrubeck) 2010-10-12 15:55:24 PDT
Needed for faster video in Fennec.
Comment 2 Chris Jones [:cjones] inactive; ni?/f?/r? if you need me 2010-11-15 13:17:19 PST
Maybe blocks final if GL doesn't get our current impl playable, but doesn't block b3.
Comment 3 Brad Lassey [:blassey] (use needinfo?) 2010-11-16 13:17:43 PST
(In reply to comment #2)
> Maybe blocks final if GL doesn't get our current impl playable, but doesn't
> block b3.

in that case, 2.0-. Please re-nom if it becomes needed
Comment 4 Chris Jones [:cjones] inactive; ni?/f?/r? if you need me 2010-11-18 10:23:30 PST
While working on bug 564086, I realized we already support this --- the Channel binds to the thread on which Open() was called, and top-level actors have to call that manually.  Nothing more needed here.
Comment 5 Chris Jones [:cjones] inactive; ni?/f?/r? if you need me 2010-11-23 19:48:14 PST
Actually, let me leave this open so I can write a test that puts my money where my mouth is.  Might come across some fun things in the process.
Comment 6 Chris Jones [:cjones] inactive; ni?/f?/r? if you need me 2010-11-24 16:20:45 PST
Created attachment 493140 [details] [diff] [review]
Test that using actors on non-main threads works

Worked right after it first compiled, kinda scary.
Comment 7 Chris Jones [:cjones] inactive; ni?/f?/r? if you need me 2011-06-03 11:35:59 PDT
http://hg.mozilla.org/mozilla-central/rev/e864474d7404

Note You need to log in before you can comment on or make changes to this bug.