Fix lookahead beyond bounds in CJS_PublicMethods::MakeFormatDate().

BUG=396255
R=jun_fang@foxitsoftware.com

Review URL: https://codereview.chromium.org/407243003

Fix FPDF_RenderPage 2x zoom in when bBackgroundAlphaNeeded is true

BUG=179413
R=thestig@chromium.org

Review URL: https://codereview.chromium.org/410073002

Added options to save pages into BMP and EMF on Windows.

BUG=179413
R=bo_xu@foxitsoftware.com

Review URL: https://codereview.chromium.org/408403002

Adjust null object check

BUG=None
R=thakis@chromium.org

Review URL: https://codereview.chromium.org/396173003

Fix remaining -Wdelete-non-virtual-dtor violations on Windows.

Follow-up to https://codereview.chromium.org/370853002/

BUG=none
R=bo_xu@foxitsoftware.com

Review URL: https://codereview.chromium.org/406683005

Build with exceptions enabled on Windows too.

This should be set consistently on all platforms. Ideally, we wouldn't
need exceptions, but for now they're used.

BUG=none (noticed while looking at chromium:82385)
R=jam@chromium.org

Review URL: https://codereview.chromium.org/404803005

Fix the potential integer overflow from "offset + size".

BUG=382667
R=jschuh@chromium.org, jun_fang@foxitsoftware.com

Review URL: https://codereview.chromium.org/390983007

Fix bounds checking in CJS_PublicMethods::MakeRegularDate().

The function is looking ahead N characters at both its "format" and "value"
strings without validating that accesses are in bounds.  Add those validations.

There are also duplicate checks in the else-branches which re-test the inverse
of the if-branch.  These are removed for simplicity.

I also tidied some stray whitespace in the function while I was at it.

BUG=393831
R=jun_fang@foxitsoftware.com

Review URL: https://codereview.chromium.org/395303004

Fix pdfium building issue on VS2010

BUG=pdfium_23
R=palmer@chromium.org

Review URL: https://codereview.chromium.org/399233002

pdfium: Fix all -Wdelete-non-virtual-dtor violations on Mac.

Calling `delete` on an object of a type that has virtual functions but
not a virtual destructor is questionable: Since the object has virtual functions,
it likely has subclasses, so if it's deleted through the base pointer and the
destructor isn't virtual, the subclass destructor won't be called.

In most cases, the classes getting deleted can just be marked final to tell
the compiler that it can't possibly have subclasses (this also enables the
compiler to generate better code).

Two classes didn't have any sub- or superclasses but virtual functions -
this doesn't make sense, so make all methods of these classes non-virtual.
(Also delete an unused function on one of the two classes.)

In one case, a class actually did have a subclass that needs to be deleted
virtually, so mark one destructor as virtual.

BUG=none
R=bo_xu@foxitsoftware.com

Review URL: https://codereview.chromium.org/370853002

Fix garbled text when loading linearized pdf document

BUG=260112, 249006, 275281, 354966, 365302, 236952
R=thestig@chromium.org

Review URL: https://codereview.chromium.org/400083002

Fix building pdfium for x64 with ninja under windows.

This adds the necessary directives to the standalone gyp file.

R=jschuh@chromium.org, jam@chromium.org
BUG=22

Patch from Michael Doppler <m.doppler@gmail.com>.

Review URL: https://codereview.chromium.org/360273002

Hook up the default v8::Platform implementation to pdfium

It remains to call the PumpMessageLoop() method at a regular interval,
however, since nothing posts to the loop yet, that shouldn't be a
problem.

BUG=25
R=jam@chromium.org

Review URL: https://codereview.chromium.org/374123002

Disable warning 4267 in pdfium until it is fixed.

BUG=376399
TBR=thestig@chromium.org

Review URL: https://codereview.chromium.org/398163006

Remove unused variable orig_size.

Original patch by Andrey Khalyavin <halyavin@google.com>

BUG=N/A
R=bo_xu@foxitsoftware.com

Review URL: https://codereview.chromium.org/384143002

Do not disable warnings-as-errors for pdfium.

It currently doesn't have any build warnings, and this way the
chromium build is guaranteed to stay warning-free after pdfium rolls.

BUG=none
R=jam@chromium.org

Review URL: https://codereview.chromium.org/373643002

Add a newline at the end of fpdfeditpage.cpp.

Fixes a warning.

BUG=
TBR=bo_xu@foxitsoftware.com

Review URL: https://codereview.chromium.org/395293002

Remove uninitialized const global g_GbFontNameMap.

C++11 makes uninitialized const PODs an error, because they contain
uninitialized memory (they're uninitialized that can never be initialized
(because they're const).

In this case, the memory was only used by _GetSubFontName() if the lang
parameter was 1, but _GetSubFontName() is only called from one place, with
a lang parameter of 0. So remove _GetSubFontName()'s lang parameter too.

(Using bsearch for searching an array that always has exactly 2 entries is
overkill too, but I'm trying to keep the diff small.)

No intended behavior change. Fixes this error on the clang/win bot:
..\..\third_party\pdfium\core\src\fxge\win32\fx_win32_device.cpp(207,20) :  error(clang): default initialization of an object of const type 'const _FontNameMap [1]'
const _FontNameMap g_GbFontNameMap[1];
                   ^

BUG=chromium:82385
R=bo_xu@foxitsoftware.com

Review URL: https://codereview.chromium.org/369343003

Error handling for invalid component number in CPDF_ICCBasedCS::v_Load

BUG=386728
R=palmer@chromium.org

Review URL: https://codereview.chromium.org/397803002

Fix uninitialized coords in _DrawCoonPatchMeshes

BUG=391470
R=palmer@chromium.org

Review URL: https://codereview.chromium.org/384593002

Fix uninitialized nresults in GetRGB

BUG=387809
R=palmer@chromium.org

Review URL: https://codereview.chromium.org/379283003

Fix uninitialized RGB in DrawShading

BUG=386730
R=palmer@chromium.org

Review URL: https://codereview.chromium.org/379923012

Fix uninitialized Storage in _LUTeval16

BUG=387826
R=palmer@chromium.org

Review URL: https://codereview.chromium.org/387273002

Add support to extract viewer preference

This change adds the support to extract "NumCopies", "PrintPageRange", "Duplex" viewer preferences for printing.

BUG=169120
R=bo_xu@foxitsoftware.com, vitalybuka@chromium.org

Review URL: https://codereview.chromium.org/345123002

Fix an out-of-boundary issue for wide string

BUG=381521
R=palmer@chromium.org

Review URL: https://codereview.chromium.org/383563002

Fix uninitialized Storage

BUG=387843
R=palmer@chromium.org

Review URL: https://codereview.chromium.org/380293002

Fix uninitialized okeybuf

BUG=387011
R=palmer@chromium.org

Review URL: https://codereview.chromium.org/382603003

Fix uninitialized triangle

BUG=387835
R=palmer@chromium.org

Review URL: https://codereview.chromium.org/381173002

Zero out temporary arrays before use in PDF encryption.

BUG=387834
R=palmer@chromium.org

Review URL: https://codereview.chromium.org/386663003

Fix a null object bug

BUG=387975
R=thakis@chromium.org

Review URL: https://codereview.chromium.org/379273002

Fixes for re-enabling more MSVC level 4 warnings: pdfium/ edition (try 2)

This contains fixes for the following sorts of issues:
* Nonstandard extension: unnamed struct/union

This also changes the signature of FPDF_FillRect() to enable the cleanest
possible changes in https://codereview.chromium.org/372273005 .

BUG=81439
TEST=none

Original patch by Peter Kasting <pkasting@chromium.org>
Original code review: https://codereview.chromium.org/376003003/

R=pkasting@chromium.org

Review URL: https://codereview.chromium.org/377353002

Fix a null object check

BUG=390615
R=thakis@chromium.org

Review URL: https://codereview.chromium.org/378293005

Fix BUILD.gn for commit 3522876d.

TBR=palmer@chromium.org

Review URL: https://codereview.chromium.org/378073002

Revert commit 95c018e2d15ecc3a5bd2a9aab18638316a6d3d46

Fixes for re-enabling more MSVC level 4 warnings: pdfium/ edition

BUG=81439
TBR=pkasting@chromium.org

Review URL: https://codereview.chromium.org/376043003

Fixes for re-enabling more MSVC level 4 warnings: pdfium/ edition

This contains fixes for the following sorts of issues:
* Nonstandard extension: unnamed struct/union

This also changes the signature of FPDF_FillRect() to enable the cleanest
possible changes in https://codereview.chromium.org/372273005 .

BUG=81439
TEST=none

Original patch by Peter Kasting <pkasting@chromium.org>
Original code review: https://codereview.chromium.org/376003003/

R=pkasting@chromium.org

Review URL: https://codereview.chromium.org/372383004

Remove custom memory manager

BUG=
R=palmer@chromium.org

Review URL: https://codereview.chromium.org/372473003

Fix for UMR in CXML_Parser::GetCharRef.

BUG=387822
R=jun_fang@foxitsoftware.com

Review URL: https://codereview.chromium.org/367383002

Fix integer overflow in fx_basic_[bw]string.cpp.

BUG=382601
R=jun_fang@foxitsoftware.com

Review URL: https://codereview.chromium.org/336003004

Fix a crasher issue due to invalid 'bits per component' in PDF

BUG=382820
R=palmer@chromium.org

Review URL: https://codereview.chromium.org/349853003

fix a crash issue in _CMapLookupCallback

BUG=382242
R=palmer@chromium.org

Review URL: https://codereview.chromium.org/341333004

Fix uninitialized nresults and RGB

BUG=387785
R=palmer@chromium.org

Review URL: https://codereview.chromium.org/362403002

Fix uninitialized offset

BUG=387783
R=palmer@chromium.org

Review URL: https://codereview.chromium.org/368783004

Fix uninitialized RGB in GetRGB

BUG=386729
R=palmer@chromium.org

Review URL: https://codereview.chromium.org/366803003

Fix uninitialized RGB in translating scanline

BUG=387841
R=palmer@chromium.org

Review URL: https://codereview.chromium.org/354363003

Integer overflow in fx_basic.h and fx_memory.h

BUG=382656
R=palmer@chromium.org

Review URL: https://codereview.chromium.org/334573007

Fix uninitialized RGB in LoadPalette

BUG=387506
R=palmer@chromium.org

Review URL: https://codereview.chromium.org/361103002

Fix uninitialized font_offset

BUG=387840
R=palmer@chromium.org

Review URL: https://codereview.chromium.org/354413002

Fix syntax error in BUILD.gn

Fix checklicenses in chromium

Remove "this==NULL" and adjust corresponding callers

BUG=
R=thakis@chromium.org

Review URL: https://codereview.chromium.org/361553002

Fix standalone build.

BUG=20
R=jschuh@chromium.org

Review URL: https://codereview.chromium.org/349033008

Import Chromium base/numerics to resolve integer overflow.

We'll use this for integer overflows going forward.

BUG=382606
R=jam@chromium.org

Review URL: https://codereview.chromium.org/349363005

Explicitly use binary mode when opening files in the pdfium_test sample.

This patch adds the 'b' (binary mode) flag to the fopen calls.

Before the fix the PPM images written with --write_images were corrupted
under Windows because on this platform in text mode all occurrences of
the byte 0xa (line feed) are replaced with the bytes 0x0d (carriage
return) and 0x0a (line feed).

On UNIX platforms the 'b' flag is supported but ignored, because no
translation of the line feed character takes place.

BUG=18
R=jam@chromium.org

Original patch by Michael Doppler <m.doppler@gmail.com>

Review URL: https://codereview.chromium.org/343303004

Fix and integer overflow issue in SDK's QuickSort

BUG=382639
R=mdempsky@chromium.org

Review URL: https://codereview.chromium.org/354673002

Revert "Import Chromium base/numerics to resolve integer overflow."

This reverts commit d9713f05fdcecab8428d39034c6b84cd0bbd2920.

This is breaking compile.

BUG=382241
R=palmer@chromium.org

Review URL: https://codereview.chromium.org/331123002

Fix a crash in CFXMEM_FixedMgr::AllocLarge

BUG=382243
R=palmer@chromium.org

Review URL: https://codereview.chromium.org/333213002

Stack-buffer-overflow in IccLib_Translate

BUG=382240
R=palmer@chromium.org

Review URL: https://codereview.chromium.org/332143002

Import Chromium base/numerics to resolve integer overflow.

We'll use this for integer overflows going forward.

BUG=382606
R=bo_xu@foxitsoftware.com, jschuh@chromium.org

Review URL: https://codereview.chromium.org/341533007

Fix JS_WIDESTRING to work with clang-cl

MSVC lexes L#macro_arg as a single wide string literal token, but
Clang and other C/C++ compliant lexers do not.  There was already
a workaround to use implicit string concatenation for GCC, but there's
a simpler solution of token pasting the L onto the stringized macro
argument with 'L###macro_arg'.  This works with Clang, GCC, and MSVC.

R=jun_fang@foxitsoftware.com, jam@chromium.org
BUG=82385

Original patch by Reid Kleckner <rnk@chromium.org>

Review URL: https://codereview.chromium.org/345643002

resolve symbol collsion with zlib 1.2.8

R=jam@chromium.org

Review URL: https://codereview.chromium.org/330023004

Add FPDFPage_SetRotation function

R=jam@chromium.org

Review URL: https://codereview.chromium.org/336563004

Enable security removal in FPDF_Doc_Save

R=jam@chromium.org

Review URL: https://codereview.chromium.org/333753002

Bring back the Skia files from the old repo.

BUG=11

BUG=379656
R=palmer@chromium.org

Review URL: https://codereview.chromium.org/320223003

free, rather than delete, malloc'd object.

BUG=381031

Original patch by Chris Palmer <palmer@chromium.org>

Review URL: https://codereview.chromium.org/322453002

Disable pointer-to-int conversion warning.

In this case the warning is benign.

Original patch by Brett Wilson <brettw@chromium.org>

R=jam@chromium.org

Review URL: https://codereview.chromium.org/319033002

Fix a crash when parsing truetype fonts without any tables.

BUG=chromium:381011
R=bo_xu@foxitsoftware.com

Review URL: https://codereview.chromium.org/317203003

Use none SSE functions when data is not 16 byte aligned

BUG=
R=jabdelmalek@google.com

Review URL: https://codereview.chromium.org/318593002

Add GN build file for pdfium, rename GYP target.

The GYP target is now called "pdfium" rather than "fpdfsdk".

Original pach by Brett Wilson <brettw@chromium.org>

fix two uninitialized reads
https://code.google.com/p/pdfium/issues/detail?id=9
https://code.google.com/p/pdfium/issues/detail?id=10

BUG=
R=bo_xu@foxitsoftware.com

Review URL: https://codereview.chromium.org/312273002

Use unsigned type for iteration to avoid int overflow.

If src_len in PDF_DecodeText is larger than 2^31,
2 * max_chars will overflow and the function will produce
an incorrect result.

BUG=none
R=bo_xu@foxitsoftware.com

Review URL: https://codereview.chromium.org/306923006

Wrong variable assignment in Pdfium.

CID=115579

Original patch by Finnur Thorarinsson <finnur@chromium.org>

In linux and mac, use Arial to replace Tahoma for displaying east europe characters

BUG=

Review URL: https://codereview.chromium.org/301263002

Fix xcode build.

R=thakis@chromium.org

Review URL: https://codereview.chromium.org/300993002

Followup comments from https://codereview.chromium.org/298893007/

R=thakis@chromium.org

Review URL: https://codereview.chromium.org/302653009

Fix a security issue in FPDF_GetMetaText

BUG=security

Review URL: https://codereview.chromium.org/307653002

Do not use FX_BOOL for int values.

BUG=none
R=bo_xu@foxitsoftware.com

Review URL: https://codereview.chromium.org/300063015

Fix licenses in headers to pass Chromium's checklicenses tool.

BUG=chromium:374943
R=jam@chromium.org

Review URL: https://codereview.chromium.org/303723004

Add Lei to AUTHORS

adjust output dir for msvs to fix msvs link error

BUG=msvs build
R=jam@chromium.org

Review URL: https://codereview.chromium.org/295323004

Change cc list to be pdfium-reviews@googlegroups.com

Use unsigned type for decoding in _A85Decode.

The local variable res can overflow over 2^31-1 and so
we must use unsigned type for decoding.

BUG=none
R=bo_xu@foxitsoftware.com

Review URL: https://codereview.chromium.org/288983009

Fix memory leak on array size overflow.

BUG=none
R=bo_xu@foxitsoftware.com

Review URL: https://codereview.chromium.org/300023002

Fix build error on Windows stand alone build.

R=scottmg@chromium.org

Review URL: https://codereview.chromium.org/299353008

Fix build error on win x64.

Move linker settings from standalone.gypi to pdfium.gyp, since they're needed when linking pdfium_test as part of other projects' gyp files as well where standalone.gypi isn't used.

R=scottmg@chromium.org

Review URL: https://codereview.chromium.org/302453004

Fix warnings in android build, fix font rendering issue, fix issue 357588: wrong characters representation, and addjust some code indent

BUG=
R=jam@chromium.org

Review URL: https://codereview.chromium.org/294353002

Fix a typo with PAGEMODE_UNKNOWN.

R=jam@chromium.org

Review URL: https://codereview.chromium.org/297993002

Switch include paths to fpdfsdk back to be relative so that pdfium_test can be built from a Chromium checkout.

Remove gcctest target which is not needed anymore.

Original patch by Andrey Khalyavin <halyavin@chromium.org>

remove Microsoft SDK

Original patch by Lucas Nihlen <luken@chromium.org>

Add a .gitattributes with "* text=auto" so that new lines are
automatically converted to the native platform's format on checkout but
are committed as "LF".

BUG=375516

Original patch from Julien Tinnes <jln@chromium.org>

Convert all line endings to LF.

Fix line endings.

Add execute permission to script

Create a standalone pdfium build.

I moved pdfium_test from chromium's repo to pdfium's. It's now buildable as a sample following the instructions on the wiki.

Revert the changes to pdfium.gyp in 4db62f5. To completely fix all the errors on all platforms we'd have to make changes to third party libraries. Some of those libraries are already shipped in chrome, and if we are going to want to share them we don't want them to diverge.

Add Nico to AUTHORS.

Don't remove -Wheader-hygiene for pdfium, 41f0590a9d7 fixed all violations.

BUG=375558