commit a3530ef268acbbbd84cd4c5465cdc29df5269390
author bungeman <bungeman@google.com> Fri Jun 20 10:35:00 2014 -0700
committer Commit bot <commit-bot@chromium.org> Fri Jun 20 10:35:00 2014 -0700
tree 7b66fd2feac35e318ff2e220c55e31a0387a597c
parent 4ed75287aed6371c6e4a41ffcc78c8a49c9810ed

DirectWrite to not round already subpixel metrics.

In keeping with the FreeType and Mac scaler contexts, do not artificially
round already subpixel metrics.

The subpixel flag should be the 'forceSubpixel' flag when it comes to
metrics. Existing backends will give subpixel metrics when 'subpixel' is
requested, but will not round proper metrics when 'subpixel' is not
requested.

R=reed@google.com

Author: bungeman@google.com

Review URL: https://codereview.chromium.org/344253002

src/ports/SkScalerContext_win_dw.cpp

diff --git a/src/ports/SkScalerContext_win_dw.cpp b/src/ports/SkScalerContext_win_dw.cpp
index fa21df5..f3293bf 100644
--- a/src/ports/SkScalerContext_win_dw.cpp
+++ b/src/ports/SkScalerContext_win_dw.cpp
@@ -288,10 +288,6 @@
                                        SkIntToScalar(gm.advanceWidth),
                                        SkIntToScalar(dwfm.designUnitsPerEm));
 
-    if (!this->isSubpixel()) {
-        advanceX = SkScalarRoundToScalar(advanceX);
-    }
-
     SkVector vecs[1] = { { advanceX, 0 } };
     if (DWRITE_MEASURING_MODE_GDI_CLASSIC == fMeasuringMode ||
         DWRITE_MEASURING_MODE_GDI_NATURAL == fMeasuringMode)