Bug 35379

Summary: Make the context that was passed to the ThreadFunction accessible.
Product: WebKit Reporter: jochen
Component: New BugsAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, jorlow
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Other   
OS: OS X 10.5   
Attachments:
Description Flags
Patch
none
patch none

Description jochen 2010-02-25 04:59:37 PST
Make the context that was passed to the ThreadFunction accessible.
Comment 1 jochen 2010-02-25 05:03:55 PST
Created attachment 49477 [details]
Patch
Comment 2 jochen 2010-02-25 05:08:47 PST
I can't think of any way to test this using a layout test
Comment 3 Jeremy Orlow 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;
>
Comment 4 jochen 2010-02-25 05:50:02 PST
Created attachment 49481 [details]
patch
Comment 5 WebKit Commit Bot 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>
Comment 6 WebKit Commit Bot 2010-02-25 06:49:23 PST
All reviewed patches have been landed.  Closing bug.