Bug 254376 – Random fixes from CPPChecker done in Blink

• Component: New Bugs
• Status: RESOLVED WONTFIX

[Ahmad Saleem]

Hi Team,

While going through Blink's commit, I came across following fixes identified by CPPChecker and done by Blink.

Blink Commit - https://src.chromium.org/viewvc/blink?view=revision&revision=165481

There are some which we can import or merge, just raising to get input.

Similar to this was delete: https://searchfox.org/wubkat/source/Source/WebCore/rendering/style/RenderStyle.h#416

Deleted - https://searchfox.org/wubkat/source/Source/WebCore/rendering/svg/LegacyRenderSVGShape.h#134 but we have three uses of it.

This can be done safely - tried it in Local Build as well - https://searchfox.org/wubkat/source/Source/WebCore/rendering/svg/SVGTextQuery.cpp#198

While all others were ++it changing to it++ and changing == to !, which our style checker already take care of.

I think from above, we can merge 'SVGTextQuery' only but just wanted to get input.

Thanks!

[Radar WebKit Bug Importer]

<rdar://problem/107444097>

[Ahmad Saleem]

SVGTextQuery change is no longer applicable - https://github.com/WebKit/WebKit/blob/2eed0466d7087a67e3b65404c23d3e1622e6ae15/Source/WebCore/rendering/svg/SVGTextQuery.cpp#L166

This function has changed due to recent optimizations. Only now to look into RenderStyle change otherwise we will be able to close it.

[Ahmad Saleem]

m_textAlignLast <- I am unable to find it and the link where I found it earlier mentioned in Comment 0 is not working due to 'searchfox.org' not having revision specific link (could be my fault).

Marking this as 'RESOLVED WONTFIX'.