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

Unified Diff: src/core/SkGlyphCache.cpp

Issue 227593010: Move distance field generation to the glyph cache (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: Move GrFontScaler back to public interface Created 6 years, 8 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/core/SkGlyphCache.h ('k') | src/gpu/GrDistanceFieldTextContext.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/core/SkGlyphCache.cpp
diff --git a/src/core/SkGlyphCache.cpp b/src/core/SkGlyphCache.cpp
old mode 100644
new mode 100755
index 2b1b62b839641d2825795ff42472ffe8519c030c..7b57e2f55ab16ef3df197d0901f9126305ce570c
--- a/src/core/SkGlyphCache.cpp
+++ b/src/core/SkGlyphCache.cpp
@@ -9,6 +9,7 @@
#include "SkGlyphCache.h"
#include "SkGlyphCache_Globals.h"
+#include "SkDistanceFieldGen.h"
#include "SkGraphics.h"
#include "SkPaint.h"
#include "SkPath.h"
@@ -328,12 +329,12 @@ SkGlyph* SkGlyphCache::lookupMetrics(uint32_t id, MetricsType mtype) {
const void* SkGlyphCache::findImage(const SkGlyph& glyph) {
if (glyph.fWidth > 0 && glyph.fWidth < kMaxGlyphWidth) {
- if (glyph.fImage == NULL) {
+ if (NULL == glyph.fImage) {
size_t size = glyph.computeImageSize();
const_cast<SkGlyph&>(glyph).fImage = fGlyphAlloc.alloc(size,
SkChunkAlloc::kReturnNil_AllocFailType);
// check that alloc() actually succeeded
- if (glyph.fImage) {
+ if (NULL != glyph.fImage) {
fScalerContext->getImage(glyph);
// TODO: the scaler may have changed the maskformat during
// getImage (e.g. from AA or LCD to BW) which means we may have
@@ -358,6 +359,45 @@ const SkPath* SkGlyphCache::findPath(const SkGlyph& glyph) {
return glyph.fPath;
}
+const void* SkGlyphCache::findDistanceField(const SkGlyph& glyph) {
+ if (glyph.fWidth > 0 && glyph.fWidth < kMaxGlyphWidth) {
+ if (NULL == glyph.fDistanceField) {
+ size_t size = SkComputeDistanceFieldSize(glyph.fWidth, glyph.fHeight);
+ if (size == 0) {
+ return NULL;
+ }
+ const void* image = this->findImage(glyph);
+ // now generate the distance field
+ if (NULL != image) {
+ const_cast<SkGlyph&>(glyph).fDistanceField = fGlyphAlloc.alloc(size,
+ SkChunkAlloc::kReturnNil_AllocFailType);
+ if (NULL != glyph.fDistanceField) {
+ SkMask::Format maskFormat = static_cast<SkMask::Format>(glyph.fMaskFormat);
+ if (SkMask::kA8_Format == maskFormat) {
+ // make the distance field from the image
+ SkGenerateDistanceFieldFromA8Image((unsigned char*)glyph.fDistanceField,
+ (unsigned char*)glyph.fImage,
+ glyph.fWidth, glyph.fHeight,
+ glyph.rowBytes());
+ fMemoryUsed += size;
+ } else if (SkMask::kBW_Format == maskFormat) {
+ // make the distance field from the image
+ SkGenerateDistanceFieldFromBWImage((unsigned char*)glyph.fDistanceField,
+ (unsigned char*)glyph.fImage,
+ glyph.fWidth, glyph.fHeight,
+ glyph.rowBytes());
+ fMemoryUsed += size;
+ } else {
+ fGlyphAlloc.unalloc(glyph.fDistanceField);
+ const_cast<SkGlyph&>(glyph).fDistanceField = NULL;
+ }
+ }
+ }
+ }
+ }
+ return glyph.fDistanceField;
+}
+
///////////////////////////////////////////////////////////////////////////////
bool SkGlyphCache::getAuxProcData(void (*proc)(void*), void** dataPtr) const {
@@ -649,6 +689,9 @@ void SkGlyphCache::validate() const {
if (glyph->fImage) {
SkASSERT(fGlyphAlloc.contains(glyph->fImage));
}
+ if (glyph->fDistanceField) {
+ SkASSERT(fGlyphAlloc.contains(glyph->fDistanceField));
+ }
}
#endif
}
« no previous file with comments | « src/core/SkGlyphCache.h ('k') | src/gpu/GrDistanceFieldTextContext.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698