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

Issues Search
    My Issues | Starred     Open | Closed | All

Issue 1275393003: Fix for 510931, merge to m44 (Closed)

Created:
5 years, 4 months ago by joshua.litt
Modified:
5 years, 4 months ago
CC:
reviews_skia.org
Base URL:
https://skia.googlesource.com/skia.git@m44
Target Ref:
refs/heads/chrome/m44
Project:
skia
Visibility:
Public.

Description

Fix for 510931, merge to m44 BUG=510931 NOTREECHECKS=true NOTRY=true NOPRESUBMIT=true Committed: https://skia.googlesource.com/skia/+/4d5906855ab3aa195841d92b7810c02c57b5b50a

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+769 lines, -154 lines) Patch
A gm/textblobrandomfont.cpp View 1 chunk +139 lines, -0 lines 0 comments Download
M gyp/utils.gypi View 1 chunk +2 lines, -0 lines 0 comments Download
M include/core/SkTypeface.h View 1 chunk +1 line, -0 lines 0 comments Download
M src/core/SkScalerContext.h View 2 chunks +4 lines, -0 lines 0 comments Download
A + src/fonts/SkRandomScalerContext.h View 2 chunks +12 lines, -6 lines 0 comments Download
A src/fonts/SkRandomScalerContext.cpp View 1 chunk +258 lines, -0 lines 0 comments Download
M src/gpu/GrAtlasTextContext.h View 5 chunks +20 lines, -6 lines 0 comments Download
M src/gpu/GrAtlasTextContext.cpp View 17 chunks +50 lines, -54 lines 0 comments Download
M src/gpu/GrBatchAtlas.h View 4 chunks +14 lines, -8 lines 0 comments Download
M src/gpu/GrBatchAtlas.cpp View 10 chunks +21 lines, -19 lines 0 comments Download
M src/gpu/GrBatchFontCache.h View 3 chunks +26 lines, -5 lines 0 comments Download
M src/gpu/GrBatchFontCache.cpp View 3 chunks +12 lines, -13 lines 0 comments Download
M src/gpu/GrFontScaler.h View 2 chunks +9 lines, -8 lines 0 comments Download
M src/gpu/GrFontScaler.cpp View 8 chunks +37 lines, -35 lines 0 comments Download
A tests/TextBlobCacheTest.cpp View 1 chunk +164 lines, -0 lines 0 comments Download

Messages

Total messages: 6 (2 generated)
joshualitt
ptal
5 years, 4 months ago (2015-08-07 17:27:05 UTC) #2
robertphillips
sure why not - lgtm
5 years, 4 months ago (2015-08-07 17:57:52 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1275393003/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1275393003/1
5 years, 4 months ago (2015-08-08 17:44:08 UTC) #5
commit-bot: I haz the power
5 years, 4 months ago (2015-08-08 17:44:22 UTC) #6
Message was sent while issue was closed. 
Committed patchset #1 (id:1) as
https://skia.googlesource.com/skia/+/4d5906855ab3aa195841d92b7810c02c57b5b50a

Powered by Google App Engine
This is Rietveld 408576698