Summary: | Update postMessage declaration in MessagePort.idl | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Chris Dumez <cdumez> | ||||
Component: | WebCore Misc. | Assignee: | Chris Dumez <cdumez> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | abarth, darin, eric, haraken, levin, ojan, webkit.review.bot | ||||
Priority: | P2 | ||||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
URL: | http://www.w3.org/TR/html5/comms.html#messageport | ||||||
Attachments: |
|
Description
Chris Dumez
2012-08-20 07:04:35 PDT
Created attachment 159419 [details]
Patch
Comment on attachment 159419 [details] Patch The change looks reasonable. DedicatedWorkerContext.idl already uses 'any' for postMessage() and webkitPostMessage(). > We currently don't notice the issue cause we provide custom implementation for those methods and the custom implementation handles the first argument as being of type 'any'. Note: Code generators do not yet support 'any', so we can use 'any' only for custom implementations. We want to fix it. Comment on attachment 159419 [details] Patch Clearing flags on attachment: 159419 Committed r126040: <http://trac.webkit.org/changeset/126040> All reviewed patches have been landed. Closing bug. |