Summary: | [wx] Frames support | ||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Kevin Ollivier <kevino> | ||||||||||||
Component: | WebKit wx | Assignee: | Kevin Ollivier <kevino> | ||||||||||||
Status: | RESOLVED FIXED | ||||||||||||||
Severity: | Normal | CC: | robin | ||||||||||||
Priority: | P2 | Keywords: | Wx | ||||||||||||
Version: | 528+ (Nightly build) | ||||||||||||||
Hardware: | All | ||||||||||||||
OS: | All | ||||||||||||||
Attachments: |
|
Description
Kevin Ollivier
2008-05-13 20:57:29 PDT
Created attachment 21119 [details]
first pass at PltformScrollbar
Here's a first try at PlatformScrollbar for you to play with. Not sure if it's doing what it is supposed to yet, but it at least compiles.
Created attachment 21210 [details]
wxWebFrame rename to enable us to work on next steps
Comment on attachment 21210 [details]
wxWebFrame rename to enable us to work on next steps
rs=me
Can this bug be marked as RESOLVED/FIXED, or does Attachment #21119 [details] still need to be reviewed?
This still needs to be reviewed. I've finally got some extra time to work on clearing the backlog of patches, so hopefully we should have things worked out by tomorrow or so. (I would have done it yesterday, but the addition of SquirrelFish, and the recurrence of an old FontPlatformData crash which I thought we had fixed had me playing catchup for a while. ;-) Comment on attachment 21210 [details]
wxWebFrame rename to enable us to work on next steps
Clearing the review flag on this already-landed patch.
Comment on attachment 21119 [details] first pass at PltformScrollbar Thanks for the patch, Robin! r- since this patch needs ChangeLog entries. See this page for more details: http://webkit.org/coding/contributing.html Also note that it may have bit-rotted since the rename patch landed (and due to other renames in the WebKit source tree). Overall the style and formatting of the code in the patch looks good. You might consider adding your own copyright statement to the modified files (or at least the brand new file, even though it's based on an existing file). Please update the patch and resubmit! Created attachment 22689 [details]
Revised wxWebFrame patch
Revised version of the above patch implementing wxWebFrame, with ChangeLog entries.
Comment on attachment 22689 [details]
Revised wxWebFrame patch
This looks good to me, and as discussed, we really can use the leak fix. :)
Comment on attachment 22689 [details] Revised wxWebFrame patch Patch landed in r35620, thanks! :) Created attachment 38994 [details]
finishes wx port frames support
Created attachment 39001 [details]
finishes wx port frames support (revised)
Landed in r48029, thanks! :) |