Bug 47390

Summary: Add Private API for creating a WebKit1 WebSerializedJSValue from the internal representation of a WebKit2 WebSerializedScriptValue
Product: WebKit Reporter: Jessie Berlin <jberlin>
Component: WebKit2Assignee: Jessie Berlin <jberlin>
Status: RESOLVED FIXED    
Severity: Normal CC: aroben, jberlin, sam, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Attachments:
Description Flags
Adds the Private APIs to create a WebKit1 WebSerializedJSValue from the internal representation of a WebKit2 WebSerializedScriptValue
none
Fixes style issue in previous patch none

Description Jessie Berlin 2010-10-07 18:59:00 PDT
Both the WebKit1 and WebKit2 types have the same underlying internal representation: WebCore::SerializedScriptValue.
Comment 1 Jessie Berlin 2010-10-07 19:33:11 PDT
Created attachment 70189 [details]
Adds the Private APIs to create a WebKit1 WebSerializedJSValue from the internal representation of a WebKit2 WebSerializedScriptValue
Comment 2 Jessie Berlin 2010-10-07 19:37:31 PDT
Created attachment 70192 [details]
Fixes style issue in previous patch
Comment 3 WebKit Review Bot 2010-10-07 19:38:43 PDT
Attachment 70189 [details] did not pass style-queue:

Failed to run "['WebKitTools/Scripts/check-webkit-style']" exit_code: 1
WebKit/mac/WebView/WebSerializedJSValuePrivate.h:31:  Extra space before ( in function call  [whitespace/parens] [4]
Total errors found: 1 in 18 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 4 Sam Weinig 2010-10-08 10:39:38 PDT
Comment on attachment 70192 [details]
Fixes style issue in previous patch

View in context: https://bugs.webkit.org/attachment.cgi?id=70192&action=review

> WebKit2/Shared/API/c/WKSerializedScriptValuePrivate.h:35
> +WK_EXPORT void* WKSerializedScriptValueGetInternalRepresentation(WKSerializedScriptValueRef);

We usually give a parameter name here.
Comment 5 Jessie Berlin 2010-10-08 12:02:54 PDT
(In reply to comment #4)
> (From update of attachment 70192 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=70192&action=review
> 
> > WebKit2/Shared/API/c/WKSerializedScriptValuePrivate.h:35
> > +WK_EXPORT void* WKSerializedScriptValueGetInternalRepresentation(WKSerializedScriptValueRef);
> 
> We usually give a parameter name here.

Done.

Thanks for the review!
Comment 6 Jessie Berlin 2010-10-08 12:54:54 PDT
Comment on attachment 70192 [details]
Fixes style issue in previous patch

Committed in r69415
http://trac.webkit.org/changeset/69415