Bug 95131 - Remove use of ClassInfo from compileGetByValOnArguments and compileGetArgumentsLength
Summary: Remove use of ClassInfo from compileGetByValOnArguments and compileGetArgumen...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: JavaScriptCore (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Mark Hahnenberg
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-08-27 14:59 PDT by Mark Hahnenberg
Modified: 2012-08-29 20:23 PDT (History)
4 users (show)

See Also:

Attachments
Patch (1.70 KB, patch)
2012-08-27 15:01 PDT, Mark Hahnenberg 		no flags Details | Formatted Diff | Diff
Patch (1.71 KB, patch)
2012-08-29 14:30 PDT, Mark Hahnenberg 		no flags Details | Formatted Diff | Diff
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Mark Hahnenberg 2012-08-27 14:59:48 PDT 
We don't need it!
Comment 1 Mark Hahnenberg 2012-08-27 15:01:12 PDT 
Created attachment 160816 [details]
Patch
Comment 2 WebKit Review Bot 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>
Comment 3 WebKit Review Bot 2012-08-27 16:05:20 PDT 
All reviewed patches have been landed.  Closing bug.
Comment 4 Mark Hahnenberg 2012-08-28 12:38:10 PDT 
Reopening because we have to remove the other use too.
Comment 5 Mark Hahnenberg 2012-08-29 14:30:10 PDT 
Created attachment 161318 [details]
Patch
Comment 6 Filip Pizlo 2012-08-29 14:30:53 PDT 
Comment on attachment 161318 [details]
Patch

I've seen this patch again!
Comment 7 Peter Beverloo (cr-android ews) 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
Comment 8 Mark Hahnenberg 2012-08-29 18:04:48 PDT 
Looks like Android bot is failing b/c it's unable to sync. Gonna CQ+ this one.
Comment 9 WebKit Review Bot 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>
Comment 10 WebKit Review Bot 2012-08-29 20:23:17 PDT 
All reviewed patches have been landed.  Closing bug.