RESOLVED FIXED 17094
Array.prototype functions create length properties with DontEnum/DontDelete 
https://bugs.webkit.org/show_bug.cgi?id=17094
Summary Array.prototype functions create length properties with DontEnum/DontDelete
Darin Adler
Reported 2008-01-30 09:48:12 PST
When looking at the use of put in the engine, I discovered a mistake in the Array.prototype functions. When they operate on non-array objects, they create length attributes with the DontEnum and DontDelete flags set. That's wrong.
Attachments
patch (51.04 KB, patch)
2008-01-30 10:05 PST, Darin Adler 		aroben: review-
DetailsFormatted DiffDiff
patch with whitespace-merge-conflicts resolved (50.18 KB, patch)
2008-01-30 11:53 PST, Darin Adler 		sam: review+
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Darin Adler
Comment 1 2008-01-30 10:05:19 PST
Created attachment 18792 [details] patch
Adam Roben (:aroben)
Comment 2 2008-01-30 11:18:53 PST
Comment on attachment 18792 [details] patch It looks like this patch contains conflict markers. Maybe you could post a new one with the conflicts resolved?
Darin Adler
Comment 3 2008-01-30 11:53:54 PST
Created attachment 18796 [details] patch with whitespace-merge-conflicts resolved When I resolved the conflict with the whitespace, I missed a couple cases and didn't recompile. Now I recompiled and re-tested.
Sam Weinig
Comment 4 2008-02-05 20:19:41 PST
Comment on attachment 18796 [details] patch with whitespace-merge-conflicts resolved While this looks great, I think it would be better to land the fix to the Array.prototype functions separate to all the other whitespace and putDirect changes, It's not necessary, but would be nice. r=me.
Darin Adler
Comment 5 2008-02-06 09:42:49 PST
Committed revision 30041.
Note You need to log in before you can comment on or make changes to this bug.