AuroraMiddleware/skia2
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Revert of Bump up point where we switch to distance fields for large glyphs (patchset #1 id:1 of https://codereview.chromium.org/1184153004/)

Browse Source 
Reason for revert:
Seeing some Nexus 4 perf regressions in individual tests in Chromium that may be due to this change. This doesn't appear to be the correct fix for the bug in any case.

Original issue's description:
> Bump up point where we switch to distance fields for large glyphs
>
> This is hopefully a temporary fix. It's unclear why distance fields
> are so much slower on the N4 (and N7).
>
> BUG=chromium:467569
>
> Committed: https://skia.googlesource.com/skia/+/0fce1fb02d93e66d42528f322f8aa4ca64ff0fb2

TBR=joshualitt@google.com,bsalomon@google.com
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=chromium:467569

Review URL: https://codereview.chromium.org/1178713005
This commit is contained in:
jvanverth 2015-06-17 13:06:48 -07:00 committed by Commit bot
parent 77f85adadb
commit 5a9e2994c9
1 changed files with 1 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/gpu/GrAtlasTextContext.cpp
View File

@ -57,8 +57,6 @@ static const int kMediumDFFontLimit = 72;
static const int kLargeDFFontSize = 162;
static const int kLargeDFFontLimit = 2 * kLargeDFFontSize;
static const int kLargeGlyphLimit = 192;
SkDEBUGCODE(static const int kExpectedDistanceAdjustTableSize = 8;)
static const int kDistanceAdjustLumShift = 5;
@ -453,7 +451,7 @@ inline bool GrAtlasTextContext::canDrawAsDistanceFields(const SkPaint& skPaint,
}
if (!fEnableDFRendering && !skPaint.isDistanceFieldTextTEMP() &&
scaledTextSize < kLargeGlyphLimit) {
scaledTextSize < kLargeDFFontSize) {
return false;
}