RESOLVED FIXED 95131
Remove use of ClassInfo from compileGetByValOnArguments and compileGetArgumentsLength 
https://bugs.webkit.org/show_bug.cgi?id=95131
Summary Remove use of ClassInfo from compileGetByValOnArguments and compileGetArgumen...
Mark Hahnenberg
Reported 2012-08-27 14:59:48 PDT
We don't need it!
Attachments
Patch (1.70 KB, patch)
2012-08-27 15:01 PDT, Mark Hahnenberg 		no flags
DetailsFormatted DiffDiff
Patch (1.71 KB, patch)
2012-08-29 14:30 PDT, Mark Hahnenberg 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Mark Hahnenberg
Comment 1 2012-08-27 15:01:12 PDT
Created attachment 160816 [details] Patch
WebKit Review Bot
Comment 2 2012-08-27 16:05:17 PDT
Comment on attachment 160816 [details] Patch Clearing flags on attachment: 160816 Committed r126815: <http://trac.webkit.org/changeset/126815>
WebKit Review Bot
Comment 3 2012-08-27 16:05:20 PDT
All reviewed patches have been landed. Closing bug.
Mark Hahnenberg
Comment 4 2012-08-28 12:38:10 PDT
Reopening because we have to remove the other use too.
Mark Hahnenberg
Comment 5 2012-08-29 14:30:10 PDT
Created attachment 161318 [details] Patch
Filip Pizlo
Comment 6 2012-08-29 14:30:53 PDT
Comment on attachment 161318 [details] Patch I've seen this patch again!
Peter Beverloo (cr-android ews)
Comment 7 2012-08-29 16:43:16 PDT
Comment on attachment 161318 [details] Patch Attachment 161318 [details] did not pass cr-android-ews (chromium-android): Output: http://queues.webkit.org/results/13683487
Mark Hahnenberg
Comment 8 2012-08-29 18:04:48 PDT
Looks like Android bot is failing b/c it's unable to sync. Gonna CQ+ this one.
WebKit Review Bot
Comment 9 2012-08-29 20:23:14 PDT
Comment on attachment 161318 [details] Patch Clearing flags on attachment: 161318 Committed r127090: <http://trac.webkit.org/changeset/127090>
WebKit Review Bot
Comment 10 2012-08-29 20:23:17 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.