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

Issue 946044: Implement support for transcoding WOFF fonts.

Can't Edit
Can't Publish+Mail
Start Review
Created:
14 years ago by agl
Modified:
14 years ago
Reviewers:
Yusuke Sato
CC:
chromium-reviews_chromium.org, agl
Visibility:
Public.

Description

Implement support for transcoding WOFF fonts. Not all the test utilities will work with WOFF fonts since they tend to get a lot larger after transcoding. I've updated |idempotent|, but none of the others as our test suites of fonts are still in TTF format.

Patch Set 1 #

Patch Set 2 : ... #

Patch Set 3 : ... #

Total comments: 6
Unified diffs Side-by-side diffs Delta from patch set Stats (+237 lines, -18 lines) Patch
M include/ots-memory-stream.h View 1 chunk +58 lines, -1 line 1 comment Download
M src/ots.cc View 1 2 13 chunks +173 lines, -13 lines 5 comments Download
M test/SConstruct View 1 chunk +1 line, -1 line 0 comments Download
M test/idempotent.cc View 1 chunk +5 lines, -3 lines 0 comments Download

Messages

Total messages: 3
agl
14 years ago (2010-04-27 20:34:42 UTC) #1
Yusuke Sato
Thanks a lot for the nice and quick work! Some minor comments below: http://codereview.appspot.com/946044/diff/3001/4001 File ...
14 years ago (2010-04-28 18:17:33 UTC) #2
Yusuke Sato
14 years ago (2010-04-28 20:00:27 UTC) #3
Adding chromium-reviews just in case.
Sign in to reply to this message.

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