RESOLVED FIXED 35379
Make the context that was passed to the ThreadFunction accessible. 
https://bugs.webkit.org/show_bug.cgi?id=35379
Summary Make the context that was passed to the ThreadFunction accessible.
jochen
Reported 2010-02-25 04:59:37 PST
Make the context that was passed to the ThreadFunction accessible.
Attachments
Patch (7.37 KB, patch)
2010-02-25 05:03 PST, jochen 		no flags
DetailsFormatted DiffDiff
patch (7.37 KB, patch)
2010-02-25 05:50 PST, jochen 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
jochen
Comment 1 2010-02-25 05:03:55 PST
Created attachment 49477 [details] Patch
jochen
Comment 2 2010-02-25 05:08:47 PST
I can't think of any way to test this using a layout test
Jeremy Orlow
Comment 3 2010-02-25 05:34:27 PST
Comment on attachment 49477 [details] Patch r=me but because there are nits to fix, cq=- > --- a/JavaScriptCore/wtf/ThreadingPthreads.cpp > +++ b/JavaScriptCore/wtf/ThreadingPthreads.cpp > @@ -53,7 +53,13 @@ > > namespace WTF { > > -typedef HashMap<ThreadIdentifier, pthread_t> ThreadMap; > +typedef struct { > + pthread_t handle; > + void* context; > +} ThreadInfo; > + extra new line > + > +typedef HashMap<ThreadIdentifier, ThreadInfo> ThreadMap; > > static Mutex* atomicallyInitializedStaticMutex; >
jochen
Comment 4 2010-02-25 05:50:02 PST
Created attachment 49481 [details] patch
WebKit Commit Bot
Comment 5 2010-02-25 06:49:18 PST
Comment on attachment 49481 [details] patch Clearing flags on attachment: 49481 Committed r55234: <http://trac.webkit.org/changeset/55234>
WebKit Commit Bot
Comment 6 2010-02-25 06:49:23 PST
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.