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

Issues Search
    My Issues | Starred     Open | Closed | All

Issue 1566583002: Merge to XFA: Remove header files that only have includes. (Closed)

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

Description

Merge to XFA: Remove header files that only have includes. Also do some cleanups in affected files. TBR=tsepez@chromium.org Review URL: https://codereview.chromium.org/1552583002 . (cherry picked from commit d794d34b5deb6ad691c19af758090f9ce46015a3) Committed: https://pdfium.googlesource.com/pdfium/+/c2fb35f2bbcf6c8fcf37380000544ae39d9a8cec

Patch Set 1 #

Patch Set 2 : Fix XFA #

Unified diffs Side-by-side diffs Delta from patch set Stats (+253 lines, -637 lines) Patch
M BUILD.gn View 4 chunks +0 lines, -5 lines 0 comments Download
D core/include/fpdfapi/fpdfapi.h View 1 chunk +0 lines, -16 lines 0 comments Download
M fpdfsdk/include/formfiller/FFL_FormFiller.h View 1 chunk +3 lines, -2 lines 0 comments Download
M fpdfsdk/include/formfiller/FFL_IFormFiller.h View 1 chunk +6 lines, -1 line 0 comments Download
D fpdfsdk/include/formfiller/FormFiller.h View 1 chunk +0 lines, -17 lines 0 comments Download
M fpdfsdk/include/fsdk_define.h View 1 2 chunks +1 line, -2 lines 0 comments Download
D fpdfsdk/include/fxedit/fxet_stub.h View 1 chunk +0 lines, -16 lines 0 comments Download
D fpdfsdk/include/pdfwindow/IPDFWindow.h View 1 chunk +0 lines, -28 lines 0 comments Download
D fpdfsdk/include/pdfwindow/PDFWindow.h View 1 chunk +0 lines, -17 lines 0 comments Download
M fpdfsdk/include/pdfwindow/PWL_Edit.h View 1 chunk +3 lines, -4 lines 0 comments Download
M fpdfsdk/src/formfiller/FFL_CBA_Fontmap.cpp View 2 chunks +5 lines, -3 lines 0 comments Download
M fpdfsdk/src/formfiller/FFL_CheckBox.cpp View 2 chunks +4 lines, -5 lines 0 comments Download
M fpdfsdk/src/formfiller/FFL_ComboBox.cpp View 4 chunks +31 lines, -39 lines 0 comments Download
M fpdfsdk/src/formfiller/FFL_FormFiller.cpp View 3 chunks +22 lines, -22 lines 0 comments Download
M fpdfsdk/src/formfiller/FFL_IFormFiller.cpp View 2 chunks +6 lines, -2 lines 0 comments Download
M fpdfsdk/src/formfiller/FFL_ListBox.cpp View 2 chunks +7 lines, -6 lines 0 comments Download
M fpdfsdk/src/formfiller/FFL_PushButton.cpp View 1 chunk +2 lines, -4 lines 0 comments Download
M fpdfsdk/src/formfiller/FFL_RadioButton.cpp View 1 chunk +3 lines, -4 lines 0 comments Download
M fpdfsdk/src/formfiller/FFL_TextField.cpp View 2 chunks +5 lines, -4 lines 0 comments Download
M fpdfsdk/src/fsdk_baseform.cpp View 2 chunks +3 lines, -3 lines 0 comments Download
M fpdfsdk/src/fxedit/fxet_ap.cpp View 2 chunks +2 lines, -2 lines 0 comments Download
M fpdfsdk/src/fxedit/fxet_edit.cpp View 30 chunks +33 lines, -123 lines 0 comments Download
M fpdfsdk/src/fxedit/fxet_list.cpp View 7 chunks +4 lines, -18 lines 0 comments Download
M fpdfsdk/src/fxedit/fxet_module.cpp View 2 chunks +0 lines, -5 lines 0 comments Download
M fpdfsdk/src/fxedit/fxet_pageobjs.cpp View 6 chunks +8 lines, -17 lines 0 comments Download
M fpdfsdk/src/pdfwindow/PWL_Button.cpp View 1 chunk +0 lines, -5 lines 0 comments Download
M fpdfsdk/src/pdfwindow/PWL_Caret.cpp View 4 chunks +2 lines, -8 lines 0 comments Download
M fpdfsdk/src/pdfwindow/PWL_ComboBox.cpp View 4 chunks +2 lines, -7 lines 0 comments Download
M fpdfsdk/src/pdfwindow/PWL_Edit.cpp View 3 chunks +22 lines, -27 lines 0 comments Download
M fpdfsdk/src/pdfwindow/PWL_EditCtrl.cpp View 1 chunk +3 lines, -4 lines 0 comments Download
M fpdfsdk/src/pdfwindow/PWL_FontMap.cpp View 4 chunks +4 lines, -14 lines 0 comments Download
M fpdfsdk/src/pdfwindow/PWL_Icon.cpp View 4 chunks +7 lines, -15 lines 0 comments Download
M fpdfsdk/src/pdfwindow/PWL_IconList.cpp View 3 chunks +2 lines, -7 lines 0 comments Download
M fpdfsdk/src/pdfwindow/PWL_Label.cpp View 1 chunk +0 lines, -3 lines 0 comments Download
M fpdfsdk/src/pdfwindow/PWL_ListBox.cpp View 3 chunks +3 lines, -8 lines 0 comments Download
M fpdfsdk/src/pdfwindow/PWL_ListCtrl.cpp View 1 chunk +0 lines, -3 lines 0 comments Download
M fpdfsdk/src/pdfwindow/PWL_Note.cpp View 1 13 chunks +5 lines, -35 lines 0 comments Download
M fpdfsdk/src/pdfwindow/PWL_ScrollBar.cpp View 3 chunks +4 lines, -3 lines 0 comments Download
M fpdfsdk/src/pdfwindow/PWL_Signature.cpp View 3 chunks +2 lines, -30 lines 0 comments Download
M fpdfsdk/src/pdfwindow/PWL_SpecialButton.cpp View 3 chunks +0 lines, -7 lines 0 comments Download
M fpdfsdk/src/pdfwindow/PWL_Utils.cpp View 4 chunks +38 lines, -64 lines 0 comments Download
M fpdfsdk/src/pdfwindow/PWL_Wnd.cpp View 8 chunks +9 lines, -26 lines 0 comments Download
M pdfium.gyp View 4 chunks +0 lines, -5 lines 0 comments Download
M xfa/include/foxitxfa.h View 1 1 chunk +2 lines, -1 line 0 comments Download

Messages

Total messages: 3 (1 generated)
Lei Zhang
TBR. There's some minor XFA fixes in patch set 2.
4 years, 11 months ago (2016-01-06 00:44:40 UTC) #1
Lei Zhang
4 years, 11 months ago (2016-01-06 00:47:01 UTC) #3
Message was sent while issue was closed. 
Committed patchset #2 (id:20001) manually as
c2fb35f2bbcf6c8fcf37380000544ae39d9a8cec (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698