After renaming RenderSVGTransformableContainer -> LegacyRenderSVGTransformableContainer (bug 237023), RenderSVGTransformableContainer can be re-introduced for LBSE.
Created attachment 452887 [details] Patch
Created attachment 452893 [details] Patch, v2
Comment on attachment 452893 [details] Patch, v2 View in context: https://bugs.webkit.org/attachment.cgi?id=452893&action=review > Source/WebCore/rendering/svg/RenderSVGTransformableContainer.h:3 > + * Copyright (C) 2009 Google, Inc. Does this need Igalia 2022 copyright? > Source/WebCore/rendering/svg/RenderSVGTransformableContainer.h:30 > +class RenderSVGTransformableContainer final : public RenderSVGContainer { Is this really final? If so why do we have a method that is override and not final? > Source/WebCore/rendering/svg/RenderSVGTransformableContainer.h:46 > + FloatPoint extraContainerTranslation() const; extra sounds not so nice, additional? > Source/WebCore/rendering/svg/RenderSVGTransformableContainer.h:50 > + void updateFromStyle() final; Probably better to separate the methods and member vars below.
<rdar://problem/89607181>
(In reply to Rob Buis from comment #3) > Comment on attachment 452893 [details] > Patch, v2 > > View in context: > https://bugs.webkit.org/attachment.cgi?id=452893&action=review > > > Source/WebCore/rendering/svg/RenderSVGTransformableContainer.h:3 > > + * Copyright (C) 2009 Google, Inc. > > Does this need Igalia 2022 copyright? Fixed, also 2020 and 2021. > > > Source/WebCore/rendering/svg/RenderSVGTransformableContainer.h:30 > > +class RenderSVGTransformableContainer final : public RenderSVGContainer { > > Is this really final? If so why do we have a method that is override and not > final? Good question in general. Do we have rules/preferences for that? Marking a class as final means that you cannot inherit from it anymore, a stricter constraint than imposing final/override qualifiers per function. Does that mean we automatically should rewrite s/override/final/ on all methods, because the class is marked as final? Here it makes sense, just wondering about the general preferences. Thanks. > > > Source/WebCore/rendering/svg/RenderSVGTransformableContainer.h:46 > > + FloatPoint extraContainerTranslation() const; > > extra sounds not so nice, additional? Fixed. > > > Source/WebCore/rendering/svg/RenderSVGTransformableContainer.h:50 > > + void updateFromStyle() final; > > Probably better to separate the methods and member vars below. Fixed.
Created attachment 453833 [details] Patch, v3
Committed r290880 (248111@trunk): <https://commits.webkit.org/248111@trunk>