diff options
| author | Ryan Harrison <rharrison@chromium.org> | 2017-09-26 15:39:10 -0400 |
|---|---|---|
| committer | Chromium commit bot <commit-bot@chromium.org> | 2017-09-26 19:58:33 +0000 |
| commit | e2df5b7305df66efbd81232d911615af60624ae3 (patch) | |
| tree | 5c5aff0ae260792790cfb1ed2f3f15863c0c0d63 /BUILD.gn | |
| parent | 7d04f1b0ab4848f1d10983b7a7b1444ac93dec70 (diff) | |
| download | pdfium-e2df5b7305df66efbd81232d911615af60624ae3.tar.xz | |
Move LZW decoder out of fx_gif
CGifLZWDecoder has been moved out into its own file, name changed to
CFX_LZWDecoder, member variable names updated, creation pattern
changed, and unit tests added.
Wrt the creation pattern, there is no longer a constructor and 2
initialization methods that need to be called. Instead all of the
initialization is done as part of the constructor. A wrapper has been
added for generating a std::unique_ptr<CFX_LZWDecoder>, so that params
can be validated.
BUG=pdfium:900,pdfium:901,pdfium:903,pdfium:904
Change-Id: Idcbe773f7fb18b08e64d5a89bfd87d4801332c53
Reviewed-on: https://pdfium-review.googlesource.com/14814
Reviewed-by: Tom Sepez <tsepez@chromium.org>
Commit-Queue: Ryan Harrison <rharrison@chromium.org>
Diffstat (limited to 'BUILD.gn')
| -rw-r--r-- | BUILD.gn | 3 |
1 files changed, 3 insertions, 0 deletions
@@ -777,6 +777,8 @@ static_library("fxcodec") { "core/fxcodec/codec/fx_codec_progress.cpp", "core/fxcodec/lbmp/fx_bmp.cpp", "core/fxcodec/lbmp/fx_bmp.h", + "core/fxcodec/lgif/cfx_lzwdecoder.cpp", + "core/fxcodec/lgif/cfx_lzwdecoder.h", "core/fxcodec/lgif/cgifcontext.cpp", "core/fxcodec/lgif/cgifcontext.h", "core/fxcodec/lgif/fx_gif.cpp", @@ -1948,6 +1950,7 @@ test("pdfium_unittests") { include_dirs = [] if (pdf_enable_xfa) { sources += [ + "core/fxcodec/lgif/cfx_lzwdecoder_unittest.cpp", "core/fxcrt/css/cfx_cssdeclaration_unittest.cpp", "core/fxcrt/css/cfx_cssstylesheet_unittest.cpp", "core/fxcrt/css/cfx_cssvaluelistparser_unittest.cpp", |