18737 2008-04-25 01:12:46 -0700 __lookupSetter__ and __lookupGetter__ don't work for getting native setters and getters 2012-03-07 00:10:12 -0800 1 1 1 Unclassified WebKit JavaScriptCore 528+ (Nightly build) PC Windows XP RESOLVED DUPLICATE 49739 javascript:alert(Attr.prototype.__lookupSetter__("value")); P2 Normal --- 1 shadow2531 webkit-unassigned barraclough joker806 webmaster oldest_to_newest 78890 0 shadow2531 2008-04-25 01:12:46 -0700 Safari 3.1 (525.13) with WebKit-SVN-r32486. Getting native setters and getters works in Firefox. 78956 1 shadow2531 2008-04-25 23:05:54 -0700 It doesn't work on instances either. javascript:alert(document.createAttribute("instance").__lookupGetter__("value")) That works in FF and Opera 9.5. 83536 2 sam 2008-06-16 13:49:03 -0700 The reason this does not work is that value is a not a getter/setter, but rather an attribute. It is an implementation detail that Firefox implements native attributes as getter/setter functions. Changing our implementation to do this would be a probable performance hit, so I can only really see doing this if there is a real world need. 83590 3 shadow2531 2008-06-17 03:57:46 -0700 (In reply to comment #2) > The reason this does not work is that value is a not a getter/setter, but > rather an attribute. It is an implementation detail that Firefox implements > native attributes as getter/setter functions. Changing our implementation to > do this would be a probable performance hit, so I can only really see doing > this if there is a real world need. > Would making it work for instances be acceptable, or would that be just as much as a performance concern? 86861 4 22467 shadow2531 2008-07-24 09:49:09 -0700 Created attachment 22467 Example that works in Opera 9.5 and Firefox 3, but not in Safari with latest webkit (In reply to comment #2) > so I can only really see doing > this if there is a real world need. See the source of the attachment for an example. If not, how bout just for compatibility with Firefox and Opera? Or, maybe try out the change and see how much performance is affected? (Maybe it'd be acceptable?) 89605 5 shadow2531 2008-08-27 21:50:05 -0700 (In reply to comment #1) > It doesn't work on instances either. > > javascript:alert(document.createAttribute("instance").__lookupGetter__("value")) > > That works in FF and Opera 9.5. > This works in Opera 9.5+, Firefox and IE8 beta 2. javascript:alert(document.createTextNode("instance").__lookupGetter__("nodeValue")) 124304 6 ap 2009-06-04 07:27:12 -0700 See also: bug 12721. 572629 7 barraclough 2012-03-07 00:10:12 -0800 The problem here isn't that __lookupGetter__/Setter are doing the wrong thing, the problem is that the properties aren't actually accessors (but per the WebIDL spec should be). *** This bug has been marked as a duplicate of bug 49739 *** 22467 2008-07-24 09:49:09 -0700 2008-07-24 09:49:09 -0700 Example that works in Opera 9.5 and Firefox 3, but not in Safari with latest webkit getset.html text/html 3301 shadow2531 PCFET0NUWVBFIGh0bWw+CjxodG1sIGxhbmc9ImVuIj4KICAgIDxoZWFkPgogICAgICAgIDxtZXRh IGNoYXJzZXQ9InV0Zi04Ij4KICAgICAgICA8dGl0bGU+PC90aXRsZT4KICAgICAgICA8c2NyaXB0 PgogICAgICAgICAgICAvLyBCcm93c2VyIHRleHRhcmVhLnZhbHVlIG5ld2xpbmUgbm9ybWFsaXph dGlvbiB2YXJpZXMgYmV0d2VlbiBwYXJzaW5nLCB1c2VyIGlucHV0LCBKUyBpbnB1dC4KICAgICAg ICAgICAgLy8gYW5kIHN1Ym1pdHRpbmcuCiAgICAgICAgICAgIAogICAgICAgICAgICAvLyBGdXJ0 aGVyLCBuZXdsaW5lIG5vcm1hbGl6YXRpb24gdmFyaWVzIGFjcm9zcyBicm93c2Vycy4gRXZlbiB0 aGUgbmV3bGluZSBmb3JtYXQgb2YgdGhlIHBhZ2UKICAgICAgICAgICAgLy8gY2FuIGFmZmVjdCBj ZXJ0YWluIGJyb3dzZXJzLgogICAgICAgICAgICAKICAgICAgICAgICAgLy8gSGF2aW5nIGRpZmZl cmVudCBuZXdsaW5lIG5vcm1hbGl6YXRpb24gY2FuIGFmZmVjdCB0ZXh0YXJlYSBjaGFyYWN0ZXIg Y291bnQgdmFsdWVzLCB3aGljaAogICAgICAgICAgICAvLyBjYW4gY2F1c2Ugb25lIGJyb3dzZXIg dG8gYmUgYWJsZSB0byBlbnRlciBtb3JlIGRhdGEgdGhhbiBhbm90aGVyIG9uIHBhZ2VzIHRoYXQg bGltaXQKICAgICAgICAgICAgLy8gaW5wdXQgYnkgdGV4dGFyZWEudmFsdWUubGVuZ3RoLgogICAg ICAgICAgICAKICAgICAgICAgICAgLy8gRnVydGhlciwgYSBjaGFyYWN0ZXIgY291bnRlciB3aWxs IG5vdCBjb252ZXkgdGhlICMgb2YgY2hhcmFjdGVycyB0aGF0IHdpbGwgYmUgc3VibWl0dGVkCiAg ICAgICAgICAgIC8vIGlmIHRoZXcgbmV3bGluZXMgZm9yIHRleHRhcmVhLnZhbHVlIGFyZSBub3Qg bm9ybWFsaXplZCB0byBcclxuLgogICAgICAgICAgICAvLyAoQmVjYXVzZSBicm93c2VycyBub3Jt YWxpemUgdG8gXHJcbiAoYmVmb3JlIGVuY29kaW5nKSB3aGVuIHN1Ym1pdHRpbmcgc28gdGhhdAog ICAgICAgICAgICAvLyBuZXdsaW5lcyBhcmUgcmVwcmVzZW50ZWQgYXMgJTBEJTBBLikKICAgICAg ICAgICAgCiAgICAgICAgICAgIC8vIDEuIFBhdGNoIHRoZSBUZXh0YXJlYS52YWx1ZSBnZXR0ZXIg dG8gbm9ybWFsaXplIG5ld2xpbmVzIHRvIFxyXG4gb24gZ2V0dGluZy4KICAgICAgICAgICAgLy8g Mi4gUGF0Y2ggdGhlIFRleHRhcmVhLnZhbHVlIHNldHRlciB0byBub3JtYWxpemUgbmV3bGluZXMg dG8gXHJcbiBvbiBzZXR0aW5nCiAgICAgICAgICAgIAogICAgICAgICAgICAvLyBGb3IgIzIsIHRo aXMgd29ya3MgYmVjYXVzZSB0aGUgb3JpZ2luYWwgLnZhbHVlIHNldHRlciBmb3IgYnJvd3NlcnMg aXMgbW9yZSBjb25zaXN0ZW50CiAgICAgICAgICAgIC8vIHdoZW4gdGhlIHNldHRlciBnZXRzIGFu IGFscmVhZHktbm9ybWFsaXplZCB2YWx1ZS4gKFRoZSBvcmlnaW5hbCBzZXR0ZXIgb2Ygb25lIGJy b3dzZXIgbWlnaHQKICAgICAgICAgICAgLy8gbm9ybWFsaXplIHRvIFxyXG4sIHRoZSBzZXR0ZXIg b2YgYW5vdGhlciBtaWdodCBub3QuKQogICAgICAgICAgICAKICAgICAgICAgICAgaWYgKHdpbmRv dy5IVE1MVGV4dEFyZWFFbGVtZW50ICYmIEhUTUxUZXh0QXJlYUVsZW1lbnQucHJvdG90eXBlICYm IEhUTUxUZXh0QXJlYUVsZW1lbnQucHJvdG90eXBlLl9fZGVmaW5lR2V0dGVyX18gJiYgSFRNTFRl eHRBcmVhRWxlbWVudC5wcm90b3R5cGUuX19kZWZpbmVTZXR0ZXJfXyAmJiBkb2N1bWVudC5jcmVh dGVFbGVtZW50TlMpIHsKICAgICAgICAgICAgICAgIHZhciBpbnN0YW5jZSA9IGRvY3VtZW50LmNy ZWF0ZUVsZW1lbnROUygiaHR0cDovL3d3dy53My5vcmcvMTk5OS94aHRtbCIsICJ0ZXh0YXJlYSIp OwogICAgICAgICAgICAgICAgaWYgKGluc3RhbmNlICYmIGluc3RhbmNlLl9fbG9va3VwR2V0dGVy X18gJiYgaW5zdGFuY2UuX19sb29rdXBTZXR0ZXJfXykgewogICAgICAgICAgICAgICAgICAgIHZh ciBnZXRWYWx1ZSA9IGluc3RhbmNlLl9fbG9va3VwR2V0dGVyX18oInZhbHVlIik7CiAgICAgICAg ICAgICAgICAgICAgdmFyIHNldFZhbHVlID0gaW5zdGFuY2UuX19sb29rdXBTZXR0ZXJfXygidmFs dWUiKTsKICAgICAgICAgICAgICAgICAgICBpZiAodHlwZW9mIGdldFZhbHVlID09ICJmdW5jdGlv biIgJiYgdHlwZW9mIHNldFZhbHVlID09ICJmdW5jdGlvbiIpIHsKICAgICAgICAgICAgICAgICAg ICAgICAgSFRNTFRleHRBcmVhRWxlbWVudC5wcm90b3R5cGUuX19kZWZpbmVHZXR0ZXJfXygidmFs dWUiLCBmdW5jdGlvbigpIHsKICAgICAgICAgICAgICAgICAgICAgICAgICAgIHJldHVybiBnZXRW YWx1ZS5jYWxsKHRoaXMpLnJlcGxhY2UoL1xyXG58XHJ8XG4vZywgIlxyXG4iKTsKICAgICAgICAg ICAgICAgICAgICAgICAgfSk7CiAgICAgICAgICAgICAgICAgICAgICAgIEhUTUxUZXh0QXJlYUVs ZW1lbnQucHJvdG90eXBlLl9fZGVmaW5lU2V0dGVyX18oInZhbHVlIiwgZnVuY3Rpb24odikgewog ICAgICAgICAgICAgICAgICAgICAgICAgICAgc2V0VmFsdWUuY2FsbCh0aGlzLCB2LnJlcGxhY2Uo L1xyXG58XHJ8XG4vZywgIlxyXG4iKSk7CiAgICAgICAgICAgICAgICAgICAgICAgIH0pOwogICAg ICAgICAgICAgICAgICAgIH0KICAgICAgICAgICAgICAgIH0KICAgICAgICAgICAgfQogICAgICAg ICAgICBpZiAoZG9jdW1lbnQuYWRkRXZlbnRMaXN0ZW5lcikgewogICAgICAgICAgICAgICAgZG9j dW1lbnQuYWRkRXZlbnRMaXN0ZW5lcigiRE9NQ29udGVudExvYWRlZCIsIGZ1bmN0aW9uKCkgewog ICAgICAgICAgICAgICAgICAgIHZhciB0YSA9IGRvY3VtZW50LmV2YWx1YXRlKCIvL3RleHRhcmVh IiwgZG9jdW1lbnQuYm9keSwgbnVsbCwgWFBhdGhSZXN1bHQuRklSU1RfT1JERVJFRF9OT0RFX1RZ UEUsIG51bGwpLnNpbmdsZU5vZGVWYWx1ZTsKICAgICAgICAgICAgICAgICAgICB0YS5mb3JtLnJl c2V0KCk7CiAgICAgICAgICAgICAgICAgICAgdGEudmFsdWUgKz0gIlxuTGluZTMiOwogICAgICAg ICAgICAgICAgICAgIGFsZXJ0KGVuY29kZVVSSUNvbXBvbmVudCh0YS52YWx1ZSkgPT0gIkxpbmUx JTBEJTBBTGluZTIlMEQlMEFMaW5lMyIpOwogICAgICAgICAgICAgICAgfSwgZmFsc2UpOwogICAg ICAgICAgICB9CiAgICAgICAgPC9zY3JpcHQ+CiAgICA8L2hlYWQ+CiAgICA8Ym9keT4KICAgICAg ICA8Zm9ybT4KICAgICAgICAgICAgPHA+PHRleHRhcmVhIHJvd3M9IjMiIG9uaW5wdXQ9ImFsZXJ0 KGVuY29kZVVSSUNvbXBvbmVudCh0aGlzLnZhbHVlKSkiPkxpbmUxCkxpbmUyPC90ZXh0YXJlYT4K ICAgICAgICAgICAgPC9wPgogICAgICAgIDwvZm9ybT4KICAgIDwvYm9keT4KPC9odG1sPg==