Age | Commit message (Collapse) | Author |
|
Missing explicit include of <vector>
TBR=dsinclair@chromium.org
Review URL: https://codereview.chromium.org/1783433004 .
|
|
This CL moves the code from xfa/src/fgas/src up one level. It then takes the
headers in xfa/src/fgas/include and moves them to their correct folder. The
src/ and include/ directories are then removed.
In some cases, when moving from include/ there was another header with the
same name. Those headers were either folded together, or the content of the
conflicting folder moved to an anonymous namespace in the cpp file as they were
not used anywhere else.
Files with duplicate names as core/src/crt were renamed to be fgas_ instead of
fx_. (e.g. fgas_system.h, fgas_memory.h, fgas_stream.h)
R=tsepez@chromium.org
Review URL: https://codereview.chromium.org/1776303002 .
|
|
R=dsinclair@chromium.org
Review URL: https://codereview.chromium.org/1773103003 .
|
|
This CL moves xfa/src/fee/src/fee up to xfa/src/fee and moves the
xfa/src/fee/include files up to xfa/src/fee.
R=tsepez@chromium.org
Review URL: https://codereview.chromium.org/1778633003 .
|
|
R=tsepez@chromium.org
Review URL: https://codereview.chromium.org/1781683002 .
|
|
TBR=tsepez@chromium.org,thakis@chromium.org
Review URL: https://codereview.chromium.org/1778893002 .
|
|
R=thakis@chromium.org, tsepez@chromium.org
BUG=
Review URL: https://codereview.chromium.org/1774123005 .
|
|
Fixup some includes so we can still build non-xfa.
TBR=tsepez@chromium.org
Review URL: https://codereview.chromium.org/1781643002 .
|
|
This CL goes through the remaining list of list warnings and records why they
are currently blacklisted, or fixes and enables them.
R=tsepez@chromium.org
Review URL: https://codereview.chromium.org/1773733002 .
|
|
Missing include in fsdk_mgr.h
TBR=dsinclair@chromium.org
Review URL: https://codereview.chromium.org/1774303002 .
|
|
R=dsinclair@chromium.org
Review URL: https://codereview.chromium.org/1780503002 .
|
|
This reverts commit 06fae28d307fe051f0c7e890344cd487c2260910.
R=dsinclair@chromium.org
Review URL: https://codereview.chromium.org/1775023003 .
|
|
TBR=tsepez@chromium.org
Review URL: https://codereview.chromium.org/1775913003 .
|
|
This reverts commit d50095fb151688e119a7097aabce11e4b3d36244.
Reason for revert: build broken.
TBR=dsinclair@chromium.org
Review URL: https://codereview.chromium.org/1774293002 .
|
|
BUG=
R=dsinclair@chromium.org
Review URL: https://codereview.chromium.org/1774753002 .
|
|
This reverts commit 594912241cd7d409b5f58a48ccffdc1904d844b2.
R=tsepez@chromium.org
Review URL: https://codereview.chromium.org/1769273002 .
|
|
This Cl moves the xfa/src/fxgraphics/src code up to the, previously empty,
xfa/src/fxgraphics directory and removes xfa/src/fxgraphics/src.
R=tsepez@chromium.org
Review URL: https://codereview.chromium.org/1775733004 .
|
|
This CL removes the xfa/src/fxjse/src and moves all of the .h and .cpp files
up to the, previously empty, xfa/src/fxjse directory.
R=tsepez@chromium.org
Review URL: https://codereview.chromium.org/1776643002 .
|
|
This Cl moves the code in xfa/src/fxfa/src/common to the diretory which
contains the respective implementations and removes the xfa/src/fxfa/src/common
directory. It them moves all of the code in xfa/src/fxfa/src up one level and
removes the xfa/src/fxfa/src
directory.
R=tsepez@chromium.org
Review URL: https://codereview.chromium.org/1770073003 .
|
|
Fix typo in fx_fnt.h
TBR=dsinclair@chromium.org
Review URL: https://codereview.chromium.org/1777643002 .
|
|
Since it is malloc'd, memset, and such, and clang-cl
notices this. Also delete unused function found by clang-cl.
BUG=pdfium:429
R=dsinclair@chromium.org
Review URL: https://codereview.chromium.org/1777573003 .
|
|
The current cast confused runtime/casting. Update to use static_cast and enable
the lint warning.
BUG=pdfium:426
R=tsepez@chromium.org
Review URL: https://codereview.chromium.org/1770253003 .
|
|
This Cl moves the xfa/src/fwl/src code up to the previously empty parent
directory and removes xfa/src/fwl/src.
R=tsepez@chromium.org
Review URL: https://codereview.chromium.org/1770953004 .
|
|
This reverts commit 051348757b75bd527d9a8bfffeaf5231b7c24449.
Reason for revert: equivalent .gyp CL failed.
TBR=ochang@chromium.org
Review URL: https://codereview.chromium.org/1775983002 .
|
|
enum FDE_CSSDISPLAY contains about 20 values, which won't fit into
4 bits. Avoid packing and burn the memory, since I worry about the
m_dwMatchedDelcs geting truncated, too.
R=dsinclair@chromium.org
Review URL: https://codereview.chromium.org/1771193002 .
|
|
The agg driver files are split into src/ and include/ directories. This CL
moves the source files up one level and removes the src/ and include/
directories.
R=tsepez@chromium.org
Review URL: https://codereview.chromium.org/1773593007 .
|
|
This CL collapses the xfa/src/fwl/src/core/include and
xfa/src/fwl/src/basewidget/include up one level.
R=tsepez@chromium.org
Review URL: https://codereview.chromium.org/1770273002 .
|
|
It isn't sub-classed. Making GetNextChar() non-virtual
may save a virtual fn call per character if the compilers
aren't able to do so themselves.
Make members private, not protected.
BUG=
R=ochang@chromium.org
Review URL: https://codereview.chromium.org/1772973002 .
|
|
warning C4201: nonstandard extension used : nameless struct/union
R=tsepez@chromium.org
Review URL: https://codereview.chromium.org/1767263002 .
|
|
This reverts commit c582c0ddfea8ba01c8449ed7841cdb33c12e0a98.
TBR=tsepez@chromium.org
Review URL: https://codereview.chromium.org/1769263002 .
|
|
R=tsepez@chromium.org
Review URL: https://codereview.chromium.org/1772543003 .
|
|
R=ochang@chromium.org
Review URL: https://codereview.chromium.org/1763913002 .
|
|
BUG=591137
R=tsepez@chromium.org
Review URL: https://codereview.chromium.org/1769183002 .
|
|
This does not appear to ever be defined, removed.
R=tsepez@chromium.org
Review URL: https://codereview.chromium.org/1773753002 .
|
|
This reverts commit 5ad8474335798295a9c502999324635845d28ff7.
R=tsepez@chromium.org
Review URL: https://codereview.chromium.org/1772713002 .
|
|
This patch is from jinming_wang@foxitsoftware.com.
R=tsepez@chromium.org
Review URL: https://codereview.chromium.org/1770103002 .
|
|
Remove deprecated declarations
BUG=pdfium:356
R=jochen@chromium.org
Review URL: https://codereview.chromium.org/1769833002 .
|
|
R=tsepez@chromium.org
Review URL: https://codereview.chromium.org/1761233002 .
|
|
This required suppressing the non-virtual-dtor warning, since
ICU isn't clean. Getting -Werror in exchange for that seems
worthwhile.
Chromium's builds are controlled by its own files, which already
-Werror.
R=ochang@chromium.org
Review URL: https://codereview.chromium.org/1765893003 .
|
|
This reverts commit 44beca7313284a60c21b4973d42f993b8c248ec9.
Reason: broken build on chromium bots.
R=ochang@chromium.org
TBR=dsinclair@chromium.org
Review URL: https://codereview.chromium.org/1766823002 .
|
|
This reverts commit 1a35d55dd6b0d1ea1918e2d6a6c25faf599ba168.
Prevent handles from being destructed after v8 is gone.
BUG=pdfium:419
R=ochang@chromium.org
Review URL: https://codereview.chromium.org/1761073006 .
|
|
TBR=ochang@chromium.org
Review URL: https://codereview.chromium.org/1766043002 .
|
|
TBR=ochang@chromium.org
Review URL: https://codereview.chromium.org/1768693002 .
|
|
This reverts commit 3e44a7bd2b60ace1ee8bc8f48c709056f65c3dc1.
R=ochang@chromium.org
TBR=ochang@chromium.org
Review URL: https://codereview.chromium.org/1767793002 .
|
|
Kill some crummy comments along the way.
BUG=pdfium:419
R=jochen@chromium.org
Review URL: https://codereview.chromium.org/1759893002 .
|
|
R=ochang@chromium.org
Review URL: https://codereview.chromium.org/1770473002 .
|
|
See https://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_rel_ng/builds/178070/steps/compile%20%28with%20patch%29/logs/stdio
R=ochang@chromium.org
Review URL: https://codereview.chromium.org/1770453002 .
|
|
This only directs paths to use Skia. Text and bitmaps
still use antigrain.
Clipping is disabled -- still figuring out pdfium's model
for saving and restoring clips.
Deleting the Skia canvas is disabled -- there's some build problem
where the pdfium and skia libraries are built with slightly
different options that I haven't tracked down. This is also why
for the moment SkDebugf is defined locally.
With this patch and pdf_use_skia=1 a modestly complicated PDF draws
(mostly) correctly with pdfium_test.
R=dsinclair@chromium.org, tsepez@chromium.org, dsinclair
BUG=
Review URL: https://codereview.chromium.org/1761453002 .
|
|
R=jochen@chromium.org
Review URL: https://codereview.chromium.org/1762643003 .
|
|
This Cl combines the two StrToInt implementations. In doing so I had to add
some more overrides to toDecimalDigit() and add a isDecimalDigit().
BUG=pdfium:423
R=tsepez@chromium.org
Review URL: https://codereview.chromium.org/1757283002 .
|