720760 - Object.defineProperty get function has incorrect `this`

Status: RESOLVED WORKSFORME

(Core :: JavaScript Engine, defect)

Description

[Benjamin Smedberg]

I'm filing this bug report from a newsgroup post in mozilla.dev.apps.firefox:

Object.defineProperty(String.prototype, "com_example_bool", {
     get : function() {
         return (/^(true|1)$/i).test(this);
     }
});

alert("true".com_example_bool);

That returns true in Opera, IE9, Chrome Canary and Safari, but returns  
false in Firefox (at least latest trunk).

Anyone know why?

A quick investigation shows that in the get() function, uneval(this) gives me:

(new String("")) instead of the "true" string I would expect. I don't think this has anything to do with the regex part of the testcase.

Comment 1

[Michael A. Puls II]

Attachment: PASS/FAIL TC

Comment 2

[Michael A. Puls II]

Attachment: Pass/Fail TC

Comment 3

[Michael A. Puls II]

System doesn't want to set text/html mime type for the attachment, even if I try explicitly.

Comment 4

[Jeff Walden [:Waldo]]

Yeah, getters and setters on the boxed-type prototypes get a boxed |this| when invoked through property accesses on primitive values.  Known, certainly needs fixing.  I'll take this and see what I can do for it.

Comment 5

[Magnus Melin [:mkmelin]]

It returns true now in firefox too.