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

Issue 1059673003: Roll PDFium to 9bf1f76771e04c20cb8a0f1c3d1938bdb965882e. (Closed)

Created:
5 years, 8 months ago by Lei Zhang
Modified:
5 years, 8 months ago
Reviewers:
Tom Sepez
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 PDFium to 9bf1f76771e04c20cb8a0f1c3d1938bdb965882e. 9bf1f76 Update the path of header file to avoid a compiling error in chrome. 6ce87f3 Refactor PDFium python test utilities. 18d971d No-op change to poke the bots after being out of sequence. 9cf3d5e Fix a compiling error in fpdf_parser_parser_embeddertest.cpp e26ddf5 Add embedder test comparing with the last patch. 672bd17 Fix uninitialized face in fx_ge_font.cpp 7e75ed5 Update windows suppressions and unify across platforms a1b7af0 Check direction of property access before blindly getting. 370d356 Roll DEPS to pick up new test expectations after commit e3dd159. BUG=459580 TBR=tsepez@chromium.org Committed: https://crrev.com/cd408039e5f0c342c6568ccb79848c789664c22b Cr-Commit-Position: refs/heads/master@{#323861}

Patch Set 1 #

Patch Set 2 : #

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: 12 (4 generated)
Lei Zhang
TBR
5 years, 8 months ago (2015-04-03 20:49:44 UTC) #1
Tom Sepez
RS LGTM
5 years, 8 months ago (2015-04-03 20:49:51 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1059673003/1
5 years, 8 months ago (2015-04-03 21:05:28 UTC) #4
Tom Sepez
On 2015/04/03 21:05:28, I haz the power (commit-bot) wrote: > CQ is trying da patch. ...
5 years, 8 months ago (2015-04-03 21:33:25 UTC) #6
Lei Zhang
On 2015/04/03 21:33:25, Tom Sepez wrote: > On 2015/04/03 21:05:28, I haz the power (commit-bot) ...
5 years, 8 months ago (2015-04-03 22:55:55 UTC) #7
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1059673003/20001
5 years, 8 months ago (2015-04-03 22:56:54 UTC) #10
commit-bot: I haz the power
Committed patchset #2 (id:20001)
5 years, 8 months ago (2015-04-04 00:51:53 UTC) #11
commit-bot: I haz the power
5 years, 8 months ago (2015-04-04 00:52:43 UTC) #12
Message was sent while issue was closed.
Patchset 2 (id:??) landed as
https://crrev.com/cd408039e5f0c342c6568ccb79848c789664c22b
Cr-Commit-Position: refs/heads/master@{#323861}

Powered by Google App Engine
This is Rietveld 408576698