Bug 279125 – Simplify `isSVGRenderer()` in `RenderObject.h`

• Component: Layout and Rendering
• Status: RESOLVED FIXED

[Ahmad Saleem]

Hi Team,

Just noticed following in RenderObject.h:

WebKit Source: https://searchfox.org/wubkat/rev/b36cbce69fddb7da33823f316bd8ead5bebee970/Source/WebCore/rendering/RenderObject.h#602

Current:

bool isSVGRenderer() const { return isRenderOrLegacyRenderSVGRoot() || isLegacyRenderSVGModelObject() || isRenderSVGModelObject() || isRenderSVGBlock() || isRenderSVGInline(); }

Where:

isLegacyRenderSVGModelObject() || isRenderSVGModelObject() -> can be replaced with (isRenderOrLegacyRenderSVGModelObject())

Which in turn will make `isSVGRenderer` as:

bool isSVGRenderer() const { return isRenderOrLegacyRenderSVGRoot() || isRenderOrLegacyRenderSVGModelObject() || isRenderSVGBlock() || isRenderSVGInline(); }

___

Just raising so we can do this clean-up.

Thanks!

[Fady Farag]

Pull request: https://github.com/WebKit/WebKit/pull/33147

[EWS]

Committed 283279@main (78a12607073a): <https://commits.webkit.org/283279@main>

Reviewed commits have been landed. Closing PR #33147 and removing active labels.

[Radar WebKit Bug Importer]

<rdar://problem/135442411>