Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(4158)

Issue 1847046: Add support for TTC font files in SkFontHost_Freetype....

Can't Edit
Can't Publish+Mail
Start Review
Created:
14 years, 3 months ago by jshin
Modified:
14 years, 3 months ago
Reviewers:
agl
CC:
skia-review_googlegroups.com, evan
Base URL:
http://skia.googlecode.com/svn/trunk/
Visibility:
Public.

Description

Add support for TTC font files in SkFontHost_Freetype. This is a companion change to a Chrome CL ( http://codereview.chromium.org/2870073/show ) BUG=http://crbug.com/50389 TEST=See the Chromium CL mentioned above.

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+4 lines, -1 line) Patch
M src/ports/SkFontHost_FreeType.cpp View 1 chunk +4 lines, -1 line 0 comments Download

Messages

Total messages: 5
jshin
14 years, 3 months ago (2010-07-28 07:55:29 UTC) #1
jshin
14 years, 3 months ago (2010-07-28 18:25:42 UTC) #2
agl
LGTM
14 years, 3 months ago (2010-07-28 18:49:42 UTC) #3
jshin
Thank you for the review. Can you check this in? The chrome-side change was just ...
14 years, 3 months ago (2010-08-06 18:05:57 UTC) #4
agl
14 years, 3 months ago (2010-08-06 18:10:31 UTC) #5
On Fri, Aug 6, 2010 at 2:05 PM,  <jshin@chromium.org> wrote:
> Thank you for the review. Can you check this in? The chrome-side change
> was just checked in (implementing GetFileName only to retrieve the face
> index). Then, I'll update skia revision on Chrome-side (after running
> webkit tests).

Landed as r593.


AGL
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b