Age | Commit message (Collapse) | Author |
|
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 .
|
|
R=dsinclair@chromium.org
Review URL: https://codereview.chromium.org/1780503002 .
|
|
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 .
|
|
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 .
|
|
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 .
|
|
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 .
|
|
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 .
|
|
This does not appear to ever be defined, removed.
R=tsepez@chromium.org
Review URL: https://codereview.chromium.org/1773753002 .
|
|
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=jochen@chromium.org
Review URL: https://codereview.chromium.org/1762643003 .
|
|
BUG=pdfium:404
R=jun_fang@foxitsoftware.com, tsepez@chromium.org
Review URL: https://codereview.chromium.org/1742903002 .
|
|
R=tsepez@chromium.org
Review URL: https://codereview.chromium.org/1764473002 .
|
|
This CL fixes and enables:
* readability/namespace
* readability/multiline_string
* readability/multiline_comment
* readability/inheritance
* readability/function
* readability/braces
R=tsepez@chromium.org
Review URL: https://codereview.chromium.org/1747123002 .
|
|
Helps with transparency. Also remove default args and
use override consistently in the affected methods.
R=dsinclair@chromium.org
Review URL: https://codereview.chromium.org/1751093002 .
|
|
This is never defined in the current code base. Remove.
R=tsepez@chromium.org
Review URL: https://codereview.chromium.org/1751913002 .
|
|
This flag is always defined, remove conditional code and flag.
R=tsepez@chromium.org
Review URL: https://codereview.chromium.org/1752503004 .
|
|
The _XFA_EMB flag is never defined, remove conditionals.
R=tsepez@chromium.org
Review URL: https://codereview.chromium.org/1751033002 .
|
|
R=dsinclair@chromium.org
Review URL: https://codereview.chromium.org/1745243002 .
|
|
R=tsepez@chromium.org
Review URL: https://codereview.chromium.org/1745023003 .
|
|
R=tsepez@chromium.org
Review URL: https://codereview.chromium.org/1746053003 .
|
|
Change a couple int32_t's to size_t in order to fix warnings on OSX.
R=tsepez@chromium.org
Review URL: https://codereview.chromium.org/1747793002 .
|
|
R=tsepez@chromium.org
Review URL: https://codereview.chromium.org/1749703002 .
|
|
The _XFA_LAYOUTITEM_ProcessCACHE_ define is always set and the
_XFA_LAYOUTITEM_MAPCACHE_ define is never set.
R=tsepez@chromium.org
Review URL: https://codereview.chromium.org/1750623002 .
|
|
This code is dead, remove it.
R=tsepez@chromium.org
Review URL: https://codereview.chromium.org/1746793002 .
|
|
R=tsepez@chromium.org
Review URL: https://codereview.chromium.org/1749793002 .
|
|
R=tsepez@chromium.org
Review URL: https://codereview.chromium.org/1749783002 .
|
|
This value is never set and never read. Removing it fixes a series of warnings
when building on OSX as g_XFAElementPropertyIndex does not initialize the
value.
R=tsepez@chromium.org
Review URL: https://codereview.chromium.org/1744893002 .
|
|
R=tsepez@chromium.org
Review URL: https://codereview.chromium.org/1746803002 .
|
|
This CL removes duplicate forward declarations, removes extraneous declarations
and adds a couple of missing declarations.
R=thestig@chromium.org
Review URL: https://codereview.chromium.org/1741003002 .
|
|
TBR=thestig@chromium.org
Review URL: https://codereview.chromium.org/1727273007 .
|
|
Grepping for FX_BOOL and |==| on the same line gives
a very strong clue that the expression won't be out
of range of the |bool| type iteself.
R=thestig@chromium.org
Review URL: https://codereview.chromium.org/1736323003 .
|
|
They buy us nothing and obfuscate the logic.
R=thestig@chromium.org
Review URL: https://codereview.chromium.org/1740223002 .
|
|
Do some cleanups along the way.
R=tsepez@chromium.org
Review URL: https://codereview.chromium.org/1732223002 .
|
|
This does not appear to ever be defined. There are a few methods which end up
being empty after this change, removed those and their callers as well.
R=tsepez@chromium.org
Review URL: https://codereview.chromium.org/1736133002 .
|
|
Remove the define and the #ifdef checks.
R=tsepez@chromium.org
Review URL: https://codereview.chromium.org/1741543002 .
|
|
The fde_memory.h include wasn't needed. This CL converts the
FDE_NewWith and FDE_DeleteWith calls to their FXTARGET_
counterparts.
R=thestig@chromium.org
Review URL: https://codereview.chromium.org/1736053002 .
|
|
R=thestig@chromium.org
Review URL: https://codereview.chromium.org/1740863002 .
|
|
R=dsinclair@chromium.org
Review URL: https://codereview.chromium.org/1740643002 .
|
|
R=thestig@chromium.org
Review URL: https://codereview.chromium.org/1735353002 .
|
|
R=dsinclair@chromium.org
Review URL: https://codereview.chromium.org/1735333002 .
|
|
And other nearby dead code.
R=dsinclair@chromium.org
Review URL: https://codereview.chromium.org/1740613002 .
|
|
Add default ctor so we don't have to zero these out.
Also, make CFX_VTemplate simpler.
Also, remove the Set() method in favor of assignment.
Also, remove CFX_FloatPoint define.
Also, remove unused (and wrong) CFX_VTemplate methods.
R=thestig@chromium.org
Review URL: https://codereview.chromium.org/1733523003 .
|
|
R=thestig@chromium.org
Review URL: https://codereview.chromium.org/1740783002 .
|