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

Issue 1982763002: Roll src/third_party/pdfium/ f6d9a61af..06f457216 (14 commits). (Closed)

Created:
4 years, 7 months ago by Nico
Modified:
4 years, 7 months ago
Reviewers:
Lei Zhang
CC:
chromium-reviews, Tom Sepez, hans, Wei Li, dsinclair, _cary, Oliver Chang
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/ f6d9a61af..06f457216 (14 commits). https://pdfium.googlesource.com/pdfium.git/+log/f6d9a61af218..06f457216f45 $ git log f6d9a61af..06f457216 --date=short --no-merges --format='%ad %ae %s' 2016-05-16 hans Fix binding of null pointer dereference to reference type 2016-05-16 tsepez Remove some c_str() calls from StringCs in xfa docs. 2016-05-16 tsepez Replace FX_WSTRC(L"xyz").c_str() with just L"xyz" 2016-05-16 dsinclair Convert border style defines to an enum class. 2016-05-16 dsinclair Readme update to make need for pdfium_is_standalone clear. 2016-05-13 tsepez Make CFX_ByteString(const CFX_ByteStringC&) explicit. 2016-05-13 tsepez Make CFX_WideString(const CFX_WideString&) explicit. 2016-05-13 weili Fix GCC warnings to be ready for GCC standalone build 2016-05-13 tsepez ReplaceSelections() methods never called. 2016-05-13 tsepez Templatize CFX_{Byte,Wide}StringC 2016-05-13 thestig Make the Skia GN build work. 2016-05-13 thestig Move some classes out of doc_vt.cpp into their own files. 2016-05-13 caryclark allow Skia to be the default graphics engine 2016-05-12 thestig Fix some misc nits. BUG=none TBR=thestig Committed: https://crrev.com/dc008ca85ec84fbe20a7a6ca79d3af05aea1d9ac Cr-Commit-Position: refs/heads/master@{#393927}

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)
Nico
4 years, 7 months ago (2016-05-16 19:27:31 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1982763002/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1982763002/1
4 years, 7 months ago (2016-05-16 19:28:09 UTC) #5
Lei Zhang
lgtm
4 years, 7 months ago (2016-05-16 20:19:25 UTC) #6
commit-bot: I haz the power
Committed patchset #1 (id:1)
4 years, 7 months ago (2016-05-16 21:16:35 UTC) #8
commit-bot: I haz the power
4 years, 7 months ago (2016-05-16 21:18:03 UTC) #10
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/dc008ca85ec84fbe20a7a6ca79d3af05aea1d9ac
Cr-Commit-Position: refs/heads/master@{#393927}

Powered by Google App Engine
This is Rietveld 408576698