Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(58)

Issue 494063003: Roll pdfium. (Closed)

Created:
6 years, 3 months ago by Tom Sepez
Modified:
6 years, 3 months ago
Reviewers:
CC:
chromium-reviews, jun_fang, Bo Xu
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Project:
chromium
Visibility:
Public.

Description

Roll pdfium. This brings in: 02e6ca4 Fix compile on mac: format string mismatch error. 1b04ea3 Set m_pLinearized to NULL after release a3c7215 Perform better input checks in early steps of parser. 1dfbe60 Use number of components from ICC profile and alternate color space 187483a Add GetValidBpc() check in CPDF_DIBSource::CreateDecoder() c2442f9 Add GetValidBpc() check in CPDF_DIBSource::GetScanLine a550e4c Revert of Use number of components from ICC profile and alternateCS (patchset #1 of c655167 Fix the issue 'SEGV on unknown address in CPDF_DataAvail::GetObjectSize' aeacba4 Fix a hang issue due to inconsistent page number in the test file 02132dc Bounds check in CFX_Font::LoadGlyphPath(). 48f26c5 Fix confusion between length in bytes and length in characters in app::response(). e9bf0e9 Don't stretch bitmaps if destination size is empty. ee6088e Check the number of component in ICCBased colorspace 4fb24f3 For pattern colorspace, its base colorspace should not be a pattern colorspace 8f2c3dc Check if user provided more than one option for pdfium_test. 93d0811 Fix bug in CPDF_PageContentGenerate constructor and destructor 8cf33bc Fixed command line parting when no output is needed. 2d55db1 Font is used after release in CPDF_TextStateData::~CPDF_TextStateData 39a97ea Fix a double-free issue in CFX_PathData::~CFX_PathData a94019d Check path point count overflow in DrawThisAppearance 91177a4 Add a null check before getting unicode text in CPDF_FormField::GetValue 4f38edb Add a null pointer check before getting the family name of the given color space in TBR=thakis@chromium.org TBR=inferno@chromium.org Committed: https://crrev.com/de62bf981511edfe8e9fffe276b507bbb06af2f0 Cr-Commit-Position: refs/heads/master@{#292006}

Patch Set 1 #

Patch Set 2 : Roll past mac build fix. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M DEPS View 1 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 7 (0 generated)
Tom Sepez
Jun, Bo, FYI this step is required to bring the pdfium used for official builds ...
6 years, 3 months ago (2014-08-26 19:59:24 UTC) #1
Tom Sepez
The CQ bit was checked by tsepez@chromium.org
6 years, 3 months ago (2014-08-26 19:59:46 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/tsepez@chromium.org/494063003/1
6 years, 3 months ago (2014-08-26 20:01:36 UTC) #3
Tom Sepez
The CQ bit was checked by tsepez@chromium.org
6 years, 3 months ago (2014-08-26 21:10:31 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/tsepez@chromium.org/494063003/20001
6 years, 3 months ago (2014-08-26 21:11:17 UTC) #5
commit-bot: I haz the power
Committed patchset #2 (20001) as 9b381807055d24702cb1090c43db3e0b75a189f0
6 years, 3 months ago (2014-08-26 22:34:56 UTC) #6
commit-bot: I haz the power
6 years, 3 months ago (2014-09-10 02:46:33 UTC) #7
Message was sent while issue was closed.
Patchset 2 (id:??) landed as
https://crrev.com/de62bf981511edfe8e9fffe276b507bbb06af2f0
Cr-Commit-Position: refs/heads/master@{#292006}

Powered by Google App Engine
This is Rietveld 408576698