Correcting text baseline for tiny fonts

third_party/WebKit/Source/platform/fonts/SimpleFontData.cpp

 /*
  * Copyright (C) 2005, 2008, 2010 Apple Inc. All rights reserved.
  * Copyright (C) 2006 Alexey Proskuryakov
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  *
  * 1.  Redistributions of source code must retain the above copyright
  *     notice, this list of conditions and the following disclaimer.
@@ skipping 113 matching lines @@
         isVDMXValid = true;
       WTF::Partitions::fastFree(vdmxTable);
     }
   }
 #endif
 
   float ascent;
   float descent;
 
   // Beware those who step here: This code is designed to match Win32 font
-  // metrics *exactly* (except the adjustment of ascent/descent on
-  // Linux/Android).
+  // metrics *exactly* except:
+  // - the adjustment of ascent/descent on Linux/Android
+  // - metrics.fAscent and metrics.fDesscent are not rounded to int
   if (isVDMXValid) {
     ascent = vdmxAscent;
     descent = -vdmxDescent;
   } else {
-    ascent = SkScalarRoundToInt(-metrics.fAscent);
-    descent = SkScalarRoundToInt(metrics.fDescent);
+    ascent = -metrics.fAscent;
+    descent = metrics.fDescent;
 #if OS(LINUX) || OS(ANDROID)
     // When subpixel positioning is enabled, if the descent is rounded down, the
     // descent part of the glyph may be truncated when displayed in a 'overflow:
     // hidden' container.  To avoid that, borrow 1 unit from the ascent when
     // possible.
     // FIXME: This can be removed if sub-pixel ascent/descent is supported.
     if (platformData().getFontRenderStyle().useSubpixelPositioning &&
         descent < SkScalarToFloat(metrics.fDescent) && ascent >= 1) {
       ++descent;
       --ascent;
@@ skipping 275 matching lines @@
     if (glyphs[i]) {
       pageToFill->setGlyphDataForIndex(offset + i, glyphs[i], this);
       haveGlyphs = true;
     }
   }
 
   return haveGlyphs;
 }
 
 }  // namespace blink