Fixing superscript and subscript baseline for tiny fonts in SVG

third_party/WebKit/Source/core/layout/svg/LayoutSVGInlineText.cpp

Index: third_party/WebKit/Source/core/layout/svg/LayoutSVGInlineText.cpp
diff --git a/third_party/WebKit/Source/core/layout/svg/LayoutSVGInlineText.cpp b/third_party/WebKit/Source/core/layout/svg/LayoutSVGInlineText.cpp
index 1366c87c9cc334ef93eb94dbcfe1594dd52ae986..039cd236e70efdbeeb540ecdb5b036ddcc22074c 100644
--- a/third_party/WebKit/Source/core/layout/svg/LayoutSVGInlineText.cpp
+++ b/third_party/WebKit/Source/core/layout/svg/LayoutSVGInlineText.cpp
@@ -398,6 +398,11 @@ void LayoutSVGInlineText::computeNewScaledFontForStyle(
       fontDescription.specifiedSize(), DoNotUseSmartMinimumForFontSize));
 
   scaledFont = Font(fontDescription);
+  // If the font size is smaller than 3, rounding ascent and descent values
+  // ignores the baseline shifts for superscript and subscript. In this case,
+  // we use subpixel ascent and descent.
+  if (scaledFont.getFontDescription().getSize().value < 3)
+    scaledFont.getFontDescription().setSubpixelAscentDescent(true);

[eae, 2016/10/17 20:14:23]

It's not clear that updateScaledFont is guaranteed to run before m_scaledFont is
used, perhaps you could set a breakpoint there or add an assert to see if that
is the case.

There is also SVGTextLayoutEngine::layoutTextOnLineOrPath that access the font
directly which might need updating.

[zakerinasab, 2016/10/18 15:51:16]

Correct. The problem is that the rounding at the beginning of platformInit()
affects the calculation of other metrics as well. The current call to
setSubpixelAscentDescent() does not update the previous calculations, so you
have to do that before the font gets initiated, which brings you to the change
needed in LayoutText.cpp which is outside SVG code and affects others too (like
Canvas). We can either go with something universal, or to remain more cautious,
we can change setSubpixelAscentDescent() to update the previous calculations if
needed. WDYT?
No benefit there as the font is already created there, unless we update the
previous metrics in setSubpixelAscentDescent(). In this case IMO this will
complete the CL.

   scaledFont.update(document.styleEngine().fontSelector());
 }