Bug 35988 - [chromium] Add API wrapper for SerializedScriptValue
Summary: [chromium] Add API wrapper for SerializedScriptValue
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit API (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Darin Fisher (:fishd, Google)
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-03-10 15:12 PST by Darin Fisher (:fishd, Google)
Modified: 2010-03-10 16:59 PST (History)
2 users (show)

See Also:


Attachments
v1 patch (18.74 KB, patch)
2010-03-10 15:13 PST, Darin Fisher (:fishd, Google)
no flags Details | Formatted Diff | Diff
v2 patch (18.74 KB, patch)
2010-03-10 15:18 PST, Darin Fisher (:fishd, Google)
jorlow: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Darin Fisher (:fishd, Google) 2010-03-10 15:12:39 PST
[chromium] Add API wrapper for SerializedScriptValue

Expose this through WebHistoryItem as a 'stateObject' property
to support history.{push,replace}State.
Comment 1 Darin Fisher (:fishd, Google) 2010-03-10 15:13:58 PST
Created attachment 50443 [details]
v1 patch
Comment 2 WebKit Review Bot 2010-03-10 15:15:49 PST
Attachment 50443 [details] did not pass style-queue:

Failed to run "WebKitTools/Scripts/check-webkit-style" exit_code: 1
WebKit/chromium/public/WebPrivatePtr.h:48:  More than one command on the same line  [whitespace/newline] [4]
Total errors found: 1 in 7 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 3 Darin Fisher (:fishd, Google) 2010-03-10 15:18:25 PST
Created attachment 50444 [details]
v2 patch
Comment 4 Jeremy Orlow 2010-03-10 16:33:16 PST
Comment on attachment 50444 [details]
v2 patch

r=me
Comment 5 Darin Fisher (:fishd, Google) 2010-03-10 16:59:36 PST
Landed as http://trac.webkit.org/changeset/55820