Summary: | Push parts of form submission code down into Frame.cpp | ||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Eric Seidel (no email) <eric> | ||||||||||
Component: | WebKit Misc. | Assignee: | Eric Seidel (no email) <eric> | ||||||||||
Status: | RESOLVED FIXED | ||||||||||||
Severity: | Normal | ||||||||||||
Priority: | P4 | ||||||||||||
Version: | 420+ | ||||||||||||
Hardware: | Mac | ||||||||||||
OS: | OS X 10.4 | ||||||||||||
Attachments: |
|
Description
Eric Seidel (no email)
2006-03-17 01:44:11 PST
Created attachment 7145 [details]
push parts of MacFrame into Frame, GraphicsContextMac into GraphicsContext
4th attempt to post this patch.
Comment on attachment 7145 [details]
push parts of MacFrame into Frame, GraphicsContextMac into GraphicsContext
I'd think you could just rename drawFloatImage to drawImage.
+ ~GraphicsContextPrivate();
Seems unnnecessary. The default destructor will do the job.
+ int width = ((s == BSTop || s == BSBottom) ? y2 - y1 : x2 - x1 );
Space before that last parenthesis looks strange to me. Maybe remove the outer parentheses altogether?
Adding these values directly to Frame rather than to FramePrivate seems like it might be moving in the wrong direction. Not sure. Are we planning to get rid of FramePrivate?
I think recordFormValue should take a PassRefPtr rather than a raw pointer.
Seems OK to land as-is. r=me
Created attachment 7181 [details]
Same patch, updated after rename changes.
Created attachment 7182 [details]
Better patch, with darin's suggested changes.
Created attachment 7183 [details]
Better patch, now also moving all Frame members to FramePrivate
Comment on attachment 7183 [details]
Better patch, now also moving all Frame members to FramePrivate
r=me
|