Bug 70186

Summary: Pass MessagePortArray to JSC's SerializedScriptValue::serialize/deserialize
Product: WebKit Reporter: Dmitry Lomov <dslomov>
Component: JavaScriptCoreAssignee: Dmitry Lomov <dslomov>
Status: RESOLVED FIXED    
Severity: Normal CC: dslomov, levin, oliver, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 65209    
Attachments:
Description Flags
Fix none

Description Dmitry Lomov 2011-10-15 16:17:32 PDT
SerializedScriptValue needs to know the message ports being sent with the message to implement trasfer
Comment 1 Dmitry Lomov 2011-10-15 16:20:15 PDT
Created attachment 111154 [details]
Fix
Comment 2 David Levin 2011-10-15 19:27:47 PDT
At first glance this seems fine. I'll look more and see if I notice anything that doesn't seem straightforward.

Adding Oliver in case he wants to look it over since he implemented SeriazlizedScriptValue for JSC and is familiar with JSC.
Comment 3 Oliver Hunt 2011-10-16 11:13:41 PDT
Comment on attachment 111154 [details]
Fix

Looks sensible to me
Comment 4 WebKit Review Bot 2011-10-16 12:18:52 PDT
Comment on attachment 111154 [details]
Fix

Clearing flags on attachment: 111154

Committed r97576: <http://trac.webkit.org/changeset/97576>
Comment 5 WebKit Review Bot 2011-10-16 12:18:56 PDT
All reviewed patches have been landed.  Closing bug.