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

Issue 302143005: Roll pdfium. (Closed)

Created:
6 years, 6 months ago by jam
Modified:
6 years, 6 months ago
Reviewers:
Lei Zhang
CC:
chromium-reviews
Visibility:
Public.

Description

Roll pdfium. This brings in dbd4c06 In linux and mac, use Arial to replace Tahoma for displaying east europe characters 64ab40e Fix xcode build. 961a34a Followup comments from https://codereview.chromium.org/298893007/ 40c4fb8 Fix a security issue in FPDF_GetMetaText by Bo Xu - 2 days ago 17211ae Do not use FX_BOOL for int values. 6387aff Fix licenses in headers to pass Chromium's checklicenses tool. 33729cd Add Lei to AUTHORS 9ea3da2 adjust output dir for msvs to fix msvs link error 2296d76 Change cc list to be pdfium-reviews@googlegroups.com db140c8 Use unsigned type for decoding in _A85Decode. 282e53d Fix memory leak on array size overflow. 2611932 Fix build error on Windows stand alone build Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=273930

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: 5 (0 generated)
jam
6 years, 6 months ago (2014-05-30 17:46:58 UTC) #1
Lei Zhang
lgtm
6 years, 6 months ago (2014-05-30 18:00:13 UTC) #2
jam
The CQ bit was checked by jam@chromium.org
6 years, 6 months ago (2014-05-30 19:15:23 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/jam@chromium.org/302143005/1
6 years, 6 months ago (2014-05-30 19:18:23 UTC) #4
commit-bot: I haz the power
6 years, 6 months ago (2014-05-30 23:22:30 UTC) #5
Message was sent while issue was closed.
Change committed as 273930

Powered by Google App Engine
This is Rietveld 408576698