Age | Commit message (Collapse) | Author |
|
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 .
|
|
This CL will enable the chromium-style checks when building. It is currently
disabled by default as there are a lot of warnings.
I only added this to GYP because there is no standalone GN build, so I have
no way to test.
R=tsepez@chromium.org
Review URL: https://codereview.chromium.org/1760213002 .
|
|
This CL fixes up chromium-style warnings in fpdf_object.h Specifically, the
warnings fixed were:
* Complex constructor has an inlined body.
* Complex destructor has an inline body.
* virtual methods with non-empty bodies shouldn't be declared inline.
R=tsepez@chromium.org
Review URL: https://codereview.chromium.org/1764793002 .
|
|
Temporary measure until FX fixes these (or I do).
This is blocking rolling XFA into chrome.
R=dsinclair@chromium.org
Review URL: https://codereview.chromium.org/1764503003 .
|
|
R=dsinclair@chromium.org
Review URL: https://codereview.chromium.org/1761993002 .
|
|
BUG=pdfium:420
R=tsepez@chromium.org
Review URL: https://codereview.chromium.org/1757373002 .
|
|
Add these tests before fixing bug 419.
R=dsinclair@chromium.org
Review URL: https://codereview.chromium.org/1761923002 .
|
|
Despite what the c++11 spec says, std::list::size() is
still O(n), not O(1).
R=dsinclair@chromium.org, ochang@chromium.org
Review URL: https://codereview.chromium.org/1763443003 .
|
|
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 .
|
|
Currently, there is a check that an object number is <= 0x1000000. If that
check fails, we end up putting the parser into a bad state and fail to load
documents. The object does not need to be in the XRef table, or referenced
from the document, just be in the document.
This Cl removes the size check and updates the various atoi calls to use a
uint32_t instead of an int32_t so we don't end up getting strange values when
converting from a string.
BUG=455199
R=tsepez@chromium.org
Review URL: https://codereview.chromium.org/1755273002 .
|
|
BUG=pdfium:421
R=tsepez@chromium.org
Review URL: https://codereview.chromium.org/1758643002 .
|
|
This is to fix an issue when win32k lockdown is enabled on PDF content
which can cause application crashes. If GDI is disabled then there's
little benefit to loading the GDI+ library.
BUG=583038
Patch by forshaw@
R=tsepez@chromium.org
Review URL: https://codereview.chromium.org/1748163006 .
|
|
Due to implicit conversion if the names buffer is empty then nullptr will
be passed to GetNameFromTT which blindly dereferences it leading to a NPD
if an invalid font file is read.
BUG=583037
Patch by forshaw@
R=tsepez@chromium.org
Review URL: https://codereview.chromium.org/1763493003 .
|
|
Part of the work required to ship XFA with chrome.
BUG=pdfium:396
R=dsinclair@chromium.org
Review URL: https://codereview.chromium.org/1763493002 .
|
|
The CFX_GrowOnlyPool class is never used, removed.
R=tsepez@chromium.org
Review URL: https://codereview.chromium.org/1757043002 .
|
|
R=tsepez@chromium.org
Review URL: https://codereview.chromium.org/1751753002 .
|
|
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 .
|
|
I assumed there was a *@chromium.org entry in there, so never looked.
R=tsepez@chromium.org
Review URL: https://codereview.chromium.org/1754763002 .
|
|
and making bits of the code simpler.
R=tsepez@chromium.org
Review URL: https://codereview.chromium.org/1742273002 .
|
|
This does not appear to ever be defined, remove.
R=tsepez@chromium.org
Review URL: https://codereview.chromium.org/1753723002 .
|
|
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 .
|
|
* FIELDFLAG_DONOTSPELLCHECK is defined on line 20.
* 1 << 25 is used by FIELDFLAG_RICHTEXT already.
R=tsepez@chromium.org
Review URL: https://codereview.chromium.org/1745263002 .
|
|
R=dsinclair@chromium.org
Review URL: https://codereview.chromium.org/1752543002 .
|