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

Issue 1991143003: Cleanup unused fgas/ code. (Closed)

Created:
4 years, 7 months ago by dsinclair
Modified:
4 years, 7 months ago
Reviewers:
Tom Sepez, Lei Zhang
CC:
pdfium-reviews_googlegroups.com
Base URL:
https://pdfium.googlesource.com/pdfium.git@master
Target Ref:
refs/heads/master
Project:
pdfium
Visibility:
Public.

Description

Cleanup unused fgas/ code. Purely code removal. Committed: https://pdfium.googlesource.com/pdfium/+/bc9164115e8aeefcc06f5bf08684a4733b25e085

Patch Set 1 #

Total comments: 8
Unified diffs Side-by-side diffs Delta from patch set Stats (+3 lines, -535 lines) Patch
M BUILD.gn View 1 chunk +0 lines, -2 lines 0 comments Download
M xfa.gyp View 1 chunk +0 lines, -2 lines 0 comments Download
M xfa/fgas/crt/fgas_utils.h View 2 chunks +0 lines, -7 lines 2 comments Download
M xfa/fgas/font/fgas_stdfontmgr.h View 1 chunk +0 lines, -2 lines 6 comments Download
M xfa/fgas/layout/fgas_linebreak.h View 1 chunk +0 lines, -7 lines 0 comments Download
M xfa/fgas/layout/fgas_linebreak.cpp View 1 chunk +0 lines, -52 lines 0 comments Download
M xfa/fgas/layout/fgas_unicode.h View 2 chunks +0 lines, -19 lines 0 comments Download
M xfa/fgas/layout/fgas_unicode.cpp View 1 chunk +0 lines, -68 lines 0 comments Download
M xfa/fgas/localization/fgas_locale.h View 2 chunks +1 line, -4 lines 0 comments Download
M xfa/fgas/localization/fgas_locale.cpp View 3 chunks +2 lines, -213 lines 0 comments Download
M xfa/fgas/localization/fgas_localeimp.h View 1 chunk +0 lines, -33 lines 0 comments Download
D xfa/fgas/localization/fgas_localemgr.h View 1 chunk +0 lines, -30 lines 0 comments Download
D xfa/fgas/localization/fgas_localemgr.cpp View 1 chunk +0 lines, -96 lines 0 comments Download

Messages

Total messages: 7 (3 generated)
dsinclair
PTAL.
4 years, 7 months ago (2016-05-19 15:05:56 UTC) #2
Tom Sepez
LGTM. Red code is best code. Also, some suggestions for followup: https://codereview.chromium.org/1991143003/diff/1/xfa/fgas/crt/fgas_utils.h File xfa/fgas/crt/fgas_utils.h (right): ...
4 years, 7 months ago (2016-05-19 16:20:53 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1991143003/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1991143003/1
4 years, 7 months ago (2016-05-19 17:51:41 UTC) #5
commit-bot: I haz the power
4 years, 7 months ago (2016-05-19 17:52:09 UTC) #7
Message was sent while issue was closed.
Committed patchset #1 (id:1) as
https://pdfium.googlesource.com/pdfium/+/bc9164115e8aeefcc06f5bf08684a4733b25...

Powered by Google App Engine
This is Rietveld 408576698