5 years agofix two uninitialized reads
John Abd-El-Malek [Thu, 5 Jun 2014 18:12:52 +0000]
fix two uninitialized reads


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

5 years agoUse unsigned type for iteration to avoid int overflow.
John Abd-El-Malek [Wed, 4 Jun 2014 21:42:19 +0000]
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.


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

5 years agoWrong variable assignment in Pdfium.
John Abd-El-Malek [Wed, 4 Jun 2014 17:52:59 +0000]
Wrong variable assignment in Pdfium.


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

6 years agoIn linux and mac, use Arial to replace Tahoma for displaying east europe characters
Bo Xu [Thu, 29 May 2014 18:32:56 +0000]
In linux and mac, use Arial to replace Tahoma for displaying east europe characters


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

6 years agoFix xcode build.
John Abd-El-Malek [Thu, 29 May 2014 15:01:22 +0000]
Fix xcode build.


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

6 years agoFollowup comments from https://codereview.chromium.org/298893007/
John Abd-El-Malek [Wed, 28 May 2014 20:35:54 +0000]
Followup comments from https://codereview.chromium.org/298893007/


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

6 years agoFix a security issue in FPDF_GetMetaText
Bo Xu [Wed, 28 May 2014 17:21:47 +0000]
Fix a security issue in FPDF_GetMetaText


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

6 years agoDo not use FX_BOOL for int values.
Bo Xu [Wed, 28 May 2014 16:55:07 +0000]
Do not use FX_BOOL for int values.


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

6 years agoFix licenses in headers to pass Chromium's checklicenses tool.
Lei Zhang [Wed, 28 May 2014 00:15:56 +0000]
Fix licenses in headers to pass Chromium's checklicenses tool.


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

6 years agoAdd Lei to AUTHORS
John Abd-El-Malek [Wed, 28 May 2014 00:04:52 +0000]
Add Lei to AUTHORS

6 years agoadjust output dir for msvs to fix msvs link error
Bo Xu [Tue, 27 May 2014 23:03:22 +0000]
adjust output dir for msvs to fix msvs link error

BUG=msvs build

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

6 years agoChange cc list to be pdfium-reviews@googlegroups.com
John Abd-El-Malek [Tue, 27 May 2014 21:04:03 +0000]
Change cc list to be pdfium-reviews@googlegroups.com

6 years agoUse unsigned type for decoding in _A85Decode.
Bo Xu [Tue, 27 May 2014 18:05:32 +0000]
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.


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

6 years agoFix memory leak on array size overflow.
Bo Xu [Tue, 27 May 2014 17:55:08 +0000]
Fix memory leak on array size overflow.


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

6 years agoFix build error on Windows stand alone build.
John Abd-El-Malek [Tue, 27 May 2014 01:27:11 +0000]
Fix build error on Windows stand alone build.


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

6 years agoFix build error on win x64.
John Abd-El-Malek [Mon, 26 May 2014 19:54:31 +0000]
Fix build error on win x64.

6 years agoMove linker settings from standalone.gypi to pdfium.gyp, since they're needed when...
John Abd-El-Malek [Mon, 26 May 2014 18:48:05 +0000]
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.


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

6 years agoFix warnings in android build, fix font rendering issue, fix issue 357588: wrong...
Bo Xu [Sat, 24 May 2014 19:20:17 +0000]
Fix warnings in android build, fix font rendering issue, fix issue 357588: wrong characters representation, and addjust some code indent


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

6 years agoFix a typo with PAGEMODE_UNKNOWN.
Lei Zhang [Sat, 24 May 2014 07:37:54 +0000]
Fix a typo with PAGEMODE_UNKNOWN.


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

6 years agoSwitch include paths to fpdfsdk back to be relative so that pdfium_test can be built...
John Abd-El-Malek [Sat, 24 May 2014 02:27:48 +0000]
Switch include paths to fpdfsdk back to be relative so that pdfium_test can be built from a Chromium checkout.

6 years agoRemove gcctest target which is not needed anymore.
John Abd-El-Malek [Sat, 24 May 2014 00:59:39 +0000]
Remove gcctest target which is not needed anymore.

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

6 years agoremove Microsoft SDK
John Abd-El-Malek [Sat, 24 May 2014 00:49:55 +0000]
remove Microsoft SDK

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

6 years agoAdd a .gitattributes with "* text=auto" so that new lines are
John Abd-El-Malek [Sat, 24 May 2014 00:46:18 +0000]
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".


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

6 years agoConvert all line endings to LF.
John Abd-El-Malek [Sat, 24 May 2014 00:28:10 +0000]
Convert all line endings to LF.

6 years agoFix line endings.
John Abd-El-Malek [Sat, 24 May 2014 00:13:20 +0000]
Fix line endings.

6 years agoAdd execute permission to script
John Abd-El-Malek [Fri, 23 May 2014 23:59:18 +0000]
Add execute permission to script

6 years agoCreate a standalone pdfium build.
John Abd-El-Malek [Thu, 22 May 2014 23:14:52 +0000]
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.

6 years agoRevert the changes to pdfium.gyp in 4db62f5. To completely fix all the errors on...
John Abd-El-Malek [Thu, 22 May 2014 21:22:16 +0000]
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.

6 years agoAdd Nico to AUTHORS.
John Abd-El-Malek [Wed, 21 May 2014 20:00:43 +0000]
Add Nico to AUTHORS.

6 years agoDon't remove -Wheader-hygiene for pdfium, 41f0590a9d7 fixed all violations.
Nico Weber [Wed, 21 May 2014 19:54:46 +0000]
Don't remove -Wheader-hygiene for pdfium, 41f0590a9d7 fixed all violations.


6 years agoAdd Raymes to AUTHORS
John Abd-El-Malek [Wed, 21 May 2014 19:52:45 +0000]
Add Raymes to AUTHORS

6 years agoAdd codereview.settings and OWNERS for pdfium
Raymes Khoury [Wed, 21 May 2014 19:48:54 +0000]
Add codereview.settings and OWNERS for pdfium

6 years agoFix clang warnings so we can treat warnings as errors.
John Abd-El-Malek [Wed, 21 May 2014 19:41:01 +0000]
Fix clang warnings so we can treat warnings as errors.

6 years agoAdd Matt Giuca to AUTHORS.
John Abd-El-Malek [Tue, 20 May 2014 21:36:45 +0000]
Add Matt Giuca to AUTHORS.

6 years agoFixed or silenced all warnings and turned on warnings-as-errors.
John Abd-El-Malek [Tue, 20 May 2014 21:34:30 +0000]
Fixed or silenced all warnings and turned on warnings-as-errors.

- Silence -Wheader-hygiene warning on Clang. Between fx_agg_driver.cpp
and fxjs_v8.h there are hundreds or thousands of things that depend on
"using namespace", so it is best to just suppress this warning than
fix it.
- gzguts.h: Include unistd.h on all non-Windows platforms, not just
Apple platforms. Fixes implicit include warnings on Linux.
- Added parens to silence a warning about mixing && and ||.
- Removed a check if an enum is negative. Enums can be unsigned,
generating an always-false warning. The check isn't necessary.
- Removed some statements that are just names of variables that do


Patch by Matt Giuca <mgiuca@chromium.org>

6 years agocf2hints: Fix early loop termination causing hints to be ignored.
John Abd-El-Malek [Tue, 20 May 2014 20:11:44 +0000]
cf2hints: Fix early loop termination causing hints to be ignored.

There was a comparison that checks maskEndPtr >= maskEndPtr. It has been
fixed to maskPtr >= maskEndPtr.


Patch by Matt Giuca <mgiuca@chromium.org>

6 years agoAdd some blank lines at the end of 3 files to remove the "no newlines at end of file...
John Abd-El-Malek [Tue, 20 May 2014 18:18:44 +0000]
Add some blank lines at the end of 3 files to remove the "no newlines at end of file" warning.

6 years agoAdd my @chromium account in the AUTHORS file.
John Abd-El-Malek [Tue, 20 May 2014 18:01:31 +0000]
Add my @chromium account in the AUTHORS file.

6 years agoRemove "using namespace agg" to make it clear which code is using agg, and also so...
John Abd-El-Malek [Tue, 20 May 2014 17:59:54 +0000]
Remove "using namespace agg" to make it clear which code is using agg, and also so we can turn compiler warnings into errors.

6 years agoFix warning on Mac.
John Abd-El-Malek [Tue, 20 May 2014 17:29:41 +0000]
Fix warning on Mac.

6 years agoRemove "using namespace v8" in header. This allows us to turn all warnings into error...
John Abd-El-Malek [Tue, 20 May 2014 16:52:29 +0000]
Remove "using namespace v8" in header. This allows us to turn all warnings into errors. It also makes it clearer to find usage of v8 in the library.

6 years agoRemove refernce to file that's not there in gyp.
John Abd-El-Malek [Tue, 20 May 2014 00:13:13 +0000]
Remove refernce to file that's not there in gyp.

6 years agoInitial commit.
John Abd-El-Malek [Sun, 18 May 2014 05:33:34 +0000]
Initial commit.

6 years agoInitial empty repository
Chase Phillips [Fri, 9 May 2014 21:48:50 +0000]
Initial empty repository