Bug 84093 - Add support for MessagePortArray type to JSC
Summary: Add support for MessagePortArray type to JSC
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebCore JavaScript (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Chris Dumez
URL:
Keywords:
Depends on:
Blocks: 86868
  Show dependency treegraph
 
Reported: 2012-04-16 16:11 PDT by Greg Billock
Modified: 2012-05-21 09:42 PDT (History)
7 users (show)

See Also:


Attachments
Patch (4.02 KB, patch)
2012-05-21 05:35 PDT, Chris Dumez
abarth: review+
abarth: commit-queue-
Details | Formatted Diff | Diff
Diff for Tools/Scripts/run-bindings-tests --reset-results (5.76 KB, patch)
2012-05-21 05:36 PDT, Chris Dumez
no flags Details | Formatted Diff | Diff
stdout output for Tools/Scripts/run-bindings-tests --reset-results (2.99 KB, text/plain)
2012-05-21 05:37 PDT, Chris Dumez
no flags Details
Patch for landing (10.60 KB, patch)
2012-05-21 09:20 PDT, Chris Dumez
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Greg Billock 2012-04-16 16:11:36 PDT
See https://bugs.webkit.org/show_bug.cgi?id=83943 for V8 implementation.
Comment 1 Chris Dumez 2012-05-20 22:52:09 PDT
This blocks WebIntents support for EFL port since the code added in r117384 (Web Intents Delivery) does not compile without this generator fix and there is no way to disable the Web Intents Delivery code at compile time.
Comment 2 Chris Dumez 2012-05-21 02:45:42 PDT
A similar fix to the one made to the V8 engine in Bug 81127 is also needed for the new Web Intents Delivery code to compile.
Comment 3 Chris Dumez 2012-05-21 05:35:57 PDT
Created attachment 143009 [details]
Patch
Comment 4 Chris Dumez 2012-05-21 05:36:55 PDT
Created attachment 143010 [details]
Diff for Tools/Scripts/run-bindings-tests --reset-results
Comment 5 Chris Dumez 2012-05-21 05:37:27 PDT
Created attachment 143011 [details]
stdout output for Tools/Scripts/run-bindings-tests --reset-results
Comment 6 Chris Dumez 2012-05-21 05:49:33 PDT
Note that I had to make the following fix as well in order for the new Web Intents Delivery code to compile:
- return "SerializedScriptValue::create(exec, $value)";
+ return "SerializedScriptValue::create(exec, $value, 0, 0)";

This is because this function does not exist anymore:
static PassRefPtr<SerializedScriptValue> create(JSC::ExecState*, JSC::JSValue, SerializationErrorMode = Throwing);

It has been changed a while ago to:
static PassRefPtr<SerializedScriptValue> create(JSC::ExecState*, JSC::JSValue, MessagePortArray*, ArrayBufferArray*, SerializationErrorMode = Throwing);

I believe Bug 70186 added the MessagePortArray* argument.

I probably should file a second bug for this one line fix? Just say the word and I'll do that.
Comment 7 Early Warning System Bot 2012-05-21 06:03:55 PDT
Comment on attachment 143009 [details]
Patch

Attachment 143009 [details] did not pass qt-wk2-ews (qt):
Output: http://queues.webkit.org/results/12747030
Comment 8 Csaba Osztrogonác 2012-05-21 06:06:46 PDT
(In reply to comment #7)
> (From update of attachment 143009 [details])
> Attachment 143009 [details] did not pass qt-wk2-ews (qt):
> Output: http://queues.webkit.org/results/12747030

Sorry, it was false positive alarm.
Comment 9 Chris Dumez 2012-05-21 06:08:04 PDT
Comment on attachment 143009 [details]
Patch

Restoring cq? flag since the ews warning was a false positive.
Comment 10 Early Warning System Bot 2012-05-21 06:08:20 PDT
Comment on attachment 143009 [details]
Patch

Attachment 143009 [details] did not pass qt-ews (qt):
Output: http://queues.webkit.org/results/12738416
Comment 11 Adam Barth 2012-05-21 09:02:59 PDT
Comment on attachment 143009 [details]
Patch

Ok.  Please run-bindings-tests with --reset-results before landing so that the updated results are committed with your change.
Comment 12 Chris Dumez 2012-05-21 09:20:21 PDT
Created attachment 143045 [details]
Patch for landing

Include result from "Tools/Scripts/run-bindings-tests --reset-results" in the patch and update the Changelog accordingly.
Comment 13 WebKit Review Bot 2012-05-21 09:42:39 PDT
Comment on attachment 143045 [details]
Patch for landing

Clearing flags on attachment: 143045

Committed r117795: <http://trac.webkit.org/changeset/117795>
Comment 14 WebKit Review Bot 2012-05-21 09:42:45 PDT
All reviewed patches have been landed.  Closing bug.