Bug 70740

Summary: Remove putVirtual
Product: WebKit Reporter: Mark Hahnenberg <mhahnenberg>
Component: JavaScriptCoreAssignee: Mark Hahnenberg <mhahnenberg>
Status: RESOLVED FIXED    
Severity: Normal CC: abarth, japhet, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 67690    
Attachments:
Description Flags
Patch
none
Patch
none
Patch
none
Patch none

Mark Hahnenberg
Reported 2011-10-24 10:39:38 PDT
Now that we have an entry in the MethodTable for both versions of put, we can remove their virtual counterparts and replace all call sites with an explicit lookup in the MethodTable.
Attachments
Patch (98.42 KB, patch)
2011-10-24 12:01 PDT, Mark Hahnenberg 		no flags
DetailsFormatted DiffDiff
Patch (99.93 KB, patch)
2011-10-24 16:23 PDT, Mark Hahnenberg 		no flags
DetailsFormatted DiffDiff
Patch (100.05 KB, patch)
2011-10-24 18:32 PDT, Mark Hahnenberg 		no flags
DetailsFormatted DiffDiff
Patch (100.07 KB, patch)
2011-10-24 18:49 PDT, Mark Hahnenberg 		no flags
DetailsFormatted DiffDiff
Show Obsolete (3) View All Add attachment proposed patch, testcase, etc.
Mark Hahnenberg
Comment 1 2011-10-24 12:01:48 PDT
Created attachment 112227 [details] Patch
Mark Hahnenberg
Comment 2 2011-10-24 16:23:55 PDT
Created attachment 112272 [details] Patch
Mark Hahnenberg
Comment 3 2011-10-24 18:32:28 PDT
Created attachment 112289 [details] Patch
Mark Hahnenberg
Comment 4 2011-10-24 18:49:46 PDT
Created attachment 112291 [details] Patch
Geoffrey Garen
Comment 5 2011-10-25 17:08:27 PDT
Comment on attachment 112291 [details] Patch r=me
WebKit Review Bot
Comment 6 2011-10-25 17:56:09 PDT
Comment on attachment 112291 [details] Patch Clearing flags on attachment: 112291 Committed r98415: <http://trac.webkit.org/changeset/98415>
WebKit Review Bot
Comment 7 2011-10-25 17:56:14 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.