Summary: | KCanvas rework - Part I | ||||||||||||||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Nikolas Zimmermann <zimmermann> | ||||||||||||||||||||||||
Component: | SVG | Assignee: | Nikolas Zimmermann <zimmermann> | ||||||||||||||||||||||||
Status: | RESOLVED FIXED | ||||||||||||||||||||||||||
Severity: | Normal | CC: | eric, mjs | ||||||||||||||||||||||||
Priority: | P2 | ||||||||||||||||||||||||||
Version: | 420+ | ||||||||||||||||||||||||||
Hardware: | Mac | ||||||||||||||||||||||||||
OS: | OS X 10.4 | ||||||||||||||||||||||||||
Attachments: |
|
Description
Nikolas Zimmermann
2006-10-27 11:10:51 PDT
Created attachment 11245 [details]
Chunk #1: kcanvas-changes.diff, Initial patch
Created attachment 11246 [details]
Chunk #2: kcanvas-qt-changes.diff, Initial patch
Created attachment 11247 [details]
Chunk #3: kcanvas-quartz-changes.diff, Initial patch
Created attachment 11248 [details]
Chunk #4: platform-changes.diff, Initial patch
Created attachment 11249 [details]
Chunk #5: platform-qt-changes.diff, Initial patch
Created attachment 11250 [details]
Chunk #6: platform-quartz-changes.diff, Initial patch
Created attachment 11251 [details]
Chunk #7: ksvg-changes.diff, Initial patch
Created attachment 11252 [details]
Chunk #8: rendering-changes.diff, Initial patch
Created attachment 11253 [details]
Chunk #9: build-system-changes.diff, Initial patch
Sorry for the patch flood, though I used a consisten naming scheme: Chunk #N - patch-name.diff, Initial Patch any new chunk will be named "Updated patch I" / "Updated patch II" etc.. :-) In case new patches are needed :-) Thanks a lot for looking into this. Niko Created attachment 11277 [details]
Final patch including ChangeLogs
No LayoutTests broken. No new leaks introduced.
Comment on attachment 11253 [details]
Chunk #9: build-system-changes.diff, Initial patch
If we're moving things out of platform and into graphics, we need to do it for all platforms. I'd also want Darin and/or Maciej to take a look at it before this gets landed.
Created attachment 11291 [details]
Updated final patch
Incorporated Darin's comments from private mail exchange.
Location and file names changed.
Landed by Oliver in r17647. |