RESOLVED FIXED 87092
[EFL] webintents/web-intents-delivery-reuse.html is failing on EFL port 
https://bugs.webkit.org/show_bug.cgi?id=87092
Summary [EFL] webintents/web-intents-delivery-reuse.html is failing on EFL port
Chris Dumez
Reported 2012-05-22 00:34:55 PDT
webkitIntent cannot be assigned for some reason and is always null after assignment.
Attachments
Patch (2.78 KB, patch)
2012-05-23 06:05 PDT, Chris Dumez 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Chris Dumez
Comment 1 2012-05-23 05:47:41 PDT
gbillock: In DOMWindowIntents.idl, should the webkitIntent attribute be really marked as readonly? Because of this, the JSC bindings generator does not generate a setter and does not do any shadowing. As a consequence, the webintents/web-intents-delivery-reuse.html test is failing for us. If I remove the "readonly" for the attribute, then the test is passing just fine. Is it really something that should be fixed in the JSC bindings generator?
Chris Dumez
Comment 2 2012-05-23 06:05:03 PDT
Created attachment 143551 [details] Patch Here is a patch which removes the "readonly" keyword from the webkitIntent attribute so that it becomes truly replaceable with JSC. The patch also unskips the corresponding test case for EFL port. Note that in the whole source tree, there is no other IDL attribute which is both readonly and [Replaceable].
Greg Billock
Comment 3 2012-05-23 08:06:01 PDT
(In reply to comment #1) > gbillock: In DOMWindowIntents.idl, should the webkitIntent attribute be really marked as readonly? Because of this, the JSC bindings generator does not generate a setter and does not do any shadowing. > > As a consequence, the webintents/web-intents-delivery-reuse.html test is failing for us. If I remove the "readonly" for the attribute, then the test is passing just fine. Is it really something that should be fixed in the JSC bindings generator? Seems fine to me, but Adam is a better person to ask. I put readonly because when the internal object is returned, JS shouldn't be altering it -- since it is replaceable, they can just write their own. It sounds like JSC uses another mechanism to accomplish that, though.
Adam Barth
Comment 4 2012-05-23 13:20:35 PDT
Comment on attachment 143551 [details] Patch Ok.
WebKit Review Bot
Comment 5 2012-05-23 13:58:04 PDT
Comment on attachment 143551 [details] Patch Clearing flags on attachment: 143551 Committed r118241: <http://trac.webkit.org/changeset/118241>
WebKit Review Bot
Comment 6 2012-05-23 13:58:09 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.