DirectWrite to use aliased if ClearType is empty.

src/ports/SkScalerContext_win_dw.cpp

 /*
  * Copyright 2011 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 #include "SkTypes.h"
 #undef GetGlyphIndices
 
@@ skipping 380 matching lines @@
         vecs[0].fX = SkScalarRoundToScalar(advanceX);
         fG_inv.mapVectors(vecs, SK_ARRAY_COUNT(vecs));
     } else {
         fSkXform.mapVectors(vecs, SK_ARRAY_COUNT(vecs));
     }
 
     glyph->fAdvanceX = SkScalarToFixed(vecs[0].fX);
     glyph->fAdvanceY = SkScalarToFixed(vecs[0].fY);
 }
 
-void SkScalerContext_DW::generateMetrics(SkGlyph* glyph) {
-    glyph->fWidth = 0;
-
-    this->generateAdvance(glyph);
-
+void SkScalerContext_DW::getBoundingBox(SkGlyph* glyph,
+                                        DWRITE_RENDERING_MODE renderingMode,
+                                        DWRITE_TEXTURE_TYPE textureType,
+                                        RECT* bbox)
+{
     //Measure raster size.
     fXform.dx = SkFixedToFloat(glyph->getSubXFixed());
     fXform.dy = SkFixedToFloat(glyph->getSubYFixed());
 
     FLOAT advance = 0;
 
     UINT16 glyphId = glyph->getGlyphID();
 
     DWRITE_GLYPH_OFFSET offset;
     offset.advanceOffset = 0.0f;
     offset.ascenderOffset = 0.0f;
 
     DWRITE_GLYPH_RUN run;
     run.glyphCount = 1;
     run.glyphAdvances = &advance;
     run.fontFace = fTypeface->fDWriteFontFace.get();
     run.fontEmSize = SkScalarToFloat(fTextSizeRender);
     run.bidiLevel = 0;
     run.glyphIndices = &glyphId;
     run.isSideways = FALSE;
     run.glyphOffsets = &offset;
 
     SkTScopedComPtr<IDWriteGlyphRunAnalysis> glyphRunAnalysis;
     HRVM(fTypeface->fFactory->CreateGlyphRunAnalysis(
              &run,
              1.0f, // pixelsPerDip,
              &fXform,
-             fRenderingMode,
+             renderingMode,
              fMeasuringMode,
              0.0f, // baselineOriginX,
              0.0f, // baselineOriginY,
              &glyphRunAnalysis),
          "Could not create glyph run analysis.");
 
+    HRVM(glyphRunAnalysis->GetAlphaTextureBounds(textureType, bbox),
+         "Could not get texture bounds.");
+}
+
+void SkScalerContext_DW::generateMetrics(SkGlyph* glyph) {
+    glyph->fWidth = 0;
+
+    this->generateAdvance(glyph);
+
     RECT bbox;
-    HRVM(glyphRunAnalysis->GetAlphaTextureBounds(fTextureType, &bbox),
-         "Could not get texture bounds.");
+    this->getBoundingBox(glyph, fRenderingMode, fTextureType, &bbox);
+
+    if (bbox.left == bbox.right || bbox.top == bbox.bottom) {
+        if (DWRITE_TEXTURE_CLEARTYPE_3x1 == fTextureType) {
+            this->getBoundingBox(glyph,
+                                 DWRITE_RENDERING_MODE_ALIASED,
+                                 DWRITE_TEXTURE_ALIASED_1x1,
+                                 &bbox);
+            if (bbox.left != bbox.right && bbox.top != bbox.bottom) {

[reed1, 2014/08/27 22:07:33]

I'd add a comment inside the if...

// We concede that, from here out for this strike, we are force ourselves to
return BW glyphs.
// If it turns out that DW version alone is sufficient to know that LCD requests
will fail, we could
//    move the test into our constructor.
// If instead, we discover that this failure is not global per strike, then
overwriting fTextureType
//    here is overkill, and we should find a way to do this per-glyph (perhaps
checking for failure
//    in generateImage).

+                fTextureType = DWRITE_TEXTURE_ALIASED_1x1;
+                fRenderingMode = DWRITE_RENDERING_MODE_ALIASED;
+            }
+        }
+    }
 
     glyph->fWidth = SkToU16(bbox.right - bbox.left);
     glyph->fHeight = SkToU16(bbox.bottom - bbox.top);
     glyph->fLeft = SkToS16(bbox.left);
     glyph->fTop = SkToS16(bbox.top);
 }
 
 void SkScalerContext_DW::generateFontMetrics(SkPaint::FontMetrics* metrics) {
     if (NULL == metrics) {
         return;
@@ skipping 259 matching lines @@
                                        NULL, //advances
                                        NULL, //offsets
                                        1, //num glyphs
                                        FALSE, //sideways
                                        FALSE, //rtl
                                        geometryToPath.get()),
          "Could not create glyph outline.");
 
     path->transform(fSkXform);
 }