RESOLVED FIXED 43943
[V8] Support readonly Date attribute 
https://bugs.webkit.org/show_bug.cgi?id=43943
Summary [V8] Support readonly Date attribute
Kinuko Yasuda
Reported 2010-08-12 17:27:20 PDT
Current CodeGeneratorV8 tries to create a hidden reference for a readonly Date attribute, but this makes compilation fail. Date is represented as a double (or v8::Date) and is not a RefPtr type, so I think we shouldn't generate the hidden reference code for Date.
Attachments
Patch (1.63 KB, patch)
2010-08-12 18:43 PDT, Kinuko Yasuda 		abarth: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Kinuko Yasuda
Comment 1 2010-08-12 18:43:27 PDT
Created attachment 64289 [details] Patch
Kinuko Yasuda
Comment 2 2010-08-12 18:50:40 PDT
I don't fully understand the code but I think it needs to go around the if statement around line 778 for Date (as we talked on chat). Can you take a look at the patch?
Adam Barth
Comment 3 2010-08-31 20:03:25 PDT
Seems related to jam magic.
Adam Barth
Comment 4 2010-08-31 20:04:13 PDT
Sorry, John. Turns out its not.
Adam Barth
Comment 5 2010-08-31 20:04:48 PDT
Comment on attachment 64289 [details] Patch ok.
Kinuko Yasuda
Comment 6 2010-09-01 23:18:12 PDT
Committed r66642: <http://trac.webkit.org/changeset/66642>
Note You need to log in before you can comment on or make changes to this bug.