Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(321)

Unified Diff: src/gpu/GrStencilAndCoverTextContext.h

Issue 563283004: Use per-typeface sets of glyphs for nvpr text (Closed) Base URL: https://skia.googlesource.com/skia.git@upload_glyphmemorypath
Patch Set: Fix builds Created 6 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « src/gpu/GrPathRendering.cpp ('k') | src/gpu/GrStencilAndCoverTextContext.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/gpu/GrStencilAndCoverTextContext.h
diff --git a/src/gpu/GrStencilAndCoverTextContext.h b/src/gpu/GrStencilAndCoverTextContext.h
index 08d7526b7e390d20e70ef86a75645abad70da2f8..5ba4a7027b370d49296f7ef2785cf19b972a1c18 100644
--- a/src/gpu/GrStencilAndCoverTextContext.h
+++ b/src/gpu/GrStencilAndCoverTextContext.h
@@ -15,6 +15,7 @@
class GrTextStrike;
class GrPath;
+class GrPathRange;
/*
* This class implements text rendering using stencil and cover path rendering
@@ -37,15 +38,31 @@ public:
virtual bool canDraw(const SkPaint& paint) SK_OVERRIDE;
private:
- class GlyphPathRange;
static const int kGlyphBufferSize = 1024;
- enum DeviceSpaceGlyphsBehavior {
- kUseIfNeeded_DeviceSpaceGlyphsBehavior,
- kDoNotUse_DeviceSpaceGlyphsBehavior,
+ enum RenderMode {
+ /**
+ * This is the render mode used by drawText(), which is mainly used by
+ * the Skia unit tests. It tries match the other text backends exactly,
+ * with the exception of not implementing LCD text, and doing anti-
+ * aliasing with the built-in MSAA.
+ */
+ kMaxAccuracy_RenderMode,
+
+ /**
+ * This is the render mode used by drawPosText(). It ignores hinting and
+ * LCD text, even if the client provided positions for hinted glyphs,
+ * and renders from a canonically-sized, generic set of paths for the
+ * given typeface. In the future we should work out a system for the
+ * client to know it should not provide hinted glyph positions. This
+ * render mode also tries to use GPU stroking for fake bold, even when
+ * SK_USE_FREETYPE_EMBOLDEN is set.
+ */
+ kMaxPerformance_RenderMode,
};
+
void init(const GrPaint&, const SkPaint&, size_t textByteLength,
- DeviceSpaceGlyphsBehavior, SkScalar textTranslateY = 0);
+ RenderMode, SkScalar textTranslateY = 0);
void initGlyphs(SkGlyphCache* cache);
void appendGlyph(uint16_t glyphID, float x);
void appendGlyph(uint16_t glyphID, float x, float y);
@@ -55,9 +72,8 @@ private:
GrDrawState::AutoRestoreEffects fStateRestore;
SkScalar fTextRatio;
float fTextInverseRatio;
- SkStrokeRec fStroke;
SkGlyphCache* fGlyphCache;
- GlyphPathRange* fGlyphs;
+ GrPathRange* fGlyphs;
uint32_t fIndexBuffer[kGlyphBufferSize];
float fTransformBuffer[2 * kGlyphBufferSize];
GrDrawTarget::PathTransformType fTransformType;
« no previous file with comments | « src/gpu/GrPathRendering.cpp ('k') | src/gpu/GrStencilAndCoverTextContext.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698