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

Issue 15077008: add separate impl file for SkFontMgr_fontconfig, so we don't polute (Closed)

Created:
7 years, 7 months ago by reed1
Modified:
7 years, 7 months ago
Reviewers:
djsollen
CC:
skia-review_googlegroups.com
Visibility:
Public.

Description

add separate impl file for SkFontMgr_fontconfig, so we don't polute SkFontHost_fontconfig, which should be sharable w/ android. Eventaully we will remove that FontHost file completely, and just use the SkFontMgr. R=djsollen@google.com Committed: https://code.google.com/p/skia/source/detail?r=9114

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+285 lines, -253 lines) Patch
M trunk/gyp/ports.gyp View 1 chunk +1 line, -0 lines 0 comments Download
A trunk/src/fonts/SkFontMgr_fontconfig.cpp View 1 chunk +276 lines, -0 lines 0 comments Download
M trunk/src/ports/SkFontHost_fontconfig.cpp View 2 chunks +8 lines, -253 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
reed1
7 years, 7 months ago (2013-05-13 18:38:13 UTC) #1
djsollen
lgtm
7 years, 7 months ago (2013-05-13 19:02:31 UTC) #2
reed1
7 years, 7 months ago (2013-05-13 19:44:02 UTC) #3
Message was sent while issue was closed.
Committed patchset #1 manually as r9114 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698