Move all SVG renderer to WebCore/rendering/svg
Created attachment 78283 [details] Patch
Please don't review the patch above. Just testing the build. Will clear the review flag when all build bots are ready.
Comment on attachment 78283 [details] Patch Hm. I'll try to clean up the code according to the webkit style and try uploading this patch later, when less changes occur to the build systems. When ever this will be :-/
Why? Is this a new policy?
(In reply to comment #4) > Why? Is this a new policy? Talked on webkit-dev about it. I think it makes much more sense to move code to the type specific folders. I think even the renderer for mathml should move to rendering/mathml and if WML has renderers too, they should move to rendering/wml. Makes the structure of rendering/ much more readable.
Created attachment 78343 [details] Patch
Attachment 78343 [details] did not build on qt: Build output: http://queues.webkit.org/results/7371076
Created attachment 78348 [details] Patch
It looks like the EWS bots are not sync to trunk. The patch applies and builds with trunk. Checked it locally.
Comment on attachment 78348 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=78348&action=review r=me. > Source/WebCore/GNUmakefile.am:2922 > + Source/WebCore/rendering/svg/RenderForeignObject.cpp \ > + Source/WebCore/rendering/svg/RenderForeignObject.h \ This should be renamed RenderSVGForeignObject, do you plan to do this in a follow-up patch?
(In reply to comment #10) > (From update of attachment 78348 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=78348&action=review > > r=me. > > > Source/WebCore/GNUmakefile.am:2922 > > + Source/WebCore/rendering/svg/RenderForeignObject.cpp \ > > + Source/WebCore/rendering/svg/RenderForeignObject.h \ > > This should be renamed RenderSVGForeignObject, do you plan to do this in a follow-up patch? Yes, I'll prepare a follow-up for review. Thanks.
Comment on attachment 78348 [details] Patch Clearing flags on attachment: 78348 Committed r75350: <http://trac.webkit.org/changeset/75350>
All reviewed patches have been landed. Closing bug.