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

Issue 1946263002: Roll src/third_party/pdfium/ fd670fdef..696fc8b70 (6 commits). (Closed)

Created:
4 years, 7 months ago by dsinclair
Modified:
4 years, 7 months ago
Reviewers:
Tom Sepez, Nico
CC:
chromium-reviews
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Roll src/third_party/pdfium/ fd670fdef..696fc8b70 (6 commits). https://pdfium.googlesource.com/pdfium.git/+log/fd670fdef898..696fc8b70c19 $ git log fd670fdef..696fc8b70 --date=short --no-merges --format='%ad %ae %s' 2016-05-04 dsinclair Revert of Remove unneeded CPVT classes. (patchset #7 id:140001 of https://codereview.chromium.org/1919283008/ ) 2016-05-04 tsepez Replace IFX_MemoryAllocator::Release() with delete. 2016-05-04 tsepez Remove unused CFX_CPLTree<> template 2016-05-03 tsepez Cleanup XFA-Specific memory allocators. 2016-05-03 dsinclair Remove FWL_WGTHITTEST_* defines in favour of enum class. 2016-05-03 weili Revert of Relax a couple checks to allow certain non-standard PDF files. (patchset #1 id:1 of https://codereview.chromium.org/1926823002/ ) BUG=chromium:608901 Committed: https://crrev.com/2660f610c7a6fd5ab69b180bb717111c8b1c1b96 Cr-Commit-Position: refs/heads/master@{#391537}

Patch Set 1 #

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

Messages

Total messages: 10 (5 generated)
dsinclair
PTAL.
4 years, 7 months ago (2016-05-04 15:02:20 UTC) #2
Nico
lgtm Most pdfium rolls (recent example: https://chromium.googlesource.com/chromium/src/+/a7b49810b22ebaed37a9290809dd6f21ae6bf453) have a spiffy link to all the changes ...
4 years, 7 months ago (2016-05-04 15:52:26 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1946263002/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1946263002/1
4 years, 7 months ago (2016-05-04 16:07:08 UTC) #6
commit-bot: I haz the power
Committed patchset #1 (id:1)
4 years, 7 months ago (2016-05-04 16:41:14 UTC) #8
commit-bot: I haz the power
4 years, 7 months ago (2016-05-04 16:42:11 UTC) #10
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/2660f610c7a6fd5ab69b180bb717111c8b1c1b96
Cr-Commit-Position: refs/heads/master@{#391537}

Powered by Google App Engine
This is Rietveld 408576698