Bug 16860

Summary: a bit of cleanup after the Activation optimization
Product: WebKit Reporter: Darin Adler <darin>
Component: JavaScriptCoreAssignee: Darin Adler <darin>
Status: RESOLVED FIXED    
Severity: Normal    
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Mac   
OS: OS X 10.4   
Attachments:
Description Flags
patch
none
patch (revised with a tiny change to pass all tests) eric: review+

Darin Adler
Reported 2008-01-13 09:17:40 PST
Here's a bit of clean-up fro the just-landed Activation optimization.
Attachments
patch (28.27 KB, patch)
2008-01-13 09:34 PST, Darin Adler 		no flags
DetailsFormatted DiffDiff
patch (revised with a tiny change to pass all tests) (28.30 KB, patch)
2008-01-13 11:18 PST, Darin Adler 		eric: review+
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Darin Adler
Comment 1 2008-01-13 09:34:01 PST
Created attachment 18422 [details] patch
Darin Adler
Comment 2 2008-01-13 11:18:41 PST
Created attachment 18424 [details] patch (revised with a tiny change to pass all tests)
Eric Seidel (no email)
Comment 3 2008-01-25 13:09:24 PST
Comment on attachment 18424 [details] patch (revised with a tiny change to pass all tests) LGTM
Darin Adler
Comment 4 2008-01-25 13:10:35 PST
Comment on attachment 18424 [details] patch (revised with a tiny change to pass all tests) I think this patch no longer cleanly applies because of the new activation stack. I'll find out when I get some time.
Darin Adler
Comment 5 2008-01-26 10:58:31 PST
Committed revision 29810.
Note You need to log in before you can comment on or make changes to this bug.