[webkit-reviews] review granted: [Bug 32674] Fix link failure on LTCG builds on Windows for chromium port : [Attachment 45092] Patch

[bugzilla-daemon at webkit.org]

Eric Seidel <eric at webkit.org> has granted Marc-Antoine Ruel
<maruel at chromium.org>'s request for review:
Bug 32674: Fix link failure on LTCG builds on Windows for chromium port
https://bugs.webkit.org/show_bug.cgi?id=32674

Attachment 45092: Patch
https://bugs.webkit.org/attachment.cgi?id=45092&action=review

------- Additional Comments from Eric Seidel <eric at webkit.org>
Do we have a build-bot which uses this (or that will), otherwise this will
certainly break again.

This is kinda an ugly fix, but that's because any of the AllInOne filese are
kinda hacks.  What is a LTCG build?  Why should we be prefixing the SVGNames
which conflict with HTMLNames instead of the reverse?

It seems that at least some of these changes need comments.  Like the naming of
mapSvgAttributeToCSSProperty should have a comment.  maybe we should just share
that code using StyledElement.h/StyledElement.cpp instead of renaming our copy?
 Or does the copy actually do something different in the SVG case?

This looks fine, but would be nice to hear answers to above before landing.

Better to leave the "Reviewed by" line as the default "Reviewed by
NOBODY(OOPS!)" because our scripts like svn-apply, the commit-queue and normal
bugzilla-tool land-diff/land-patches know how to fix that up with the proper
reviewer from the bug anyway.