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

Issue 789613007: Roll pdfium to a48ead9b7e3dc01f1bde83f69062eebd960edfdc. (Closed)

Created:
5 years, 11 months ago by Tom Sepez
Modified:
5 years, 11 months ago
Reviewers:
Lei Zhang
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 a48ead9b7e3dc01f1bde83f69062eebd960edfdc. This bring in: a48ead9 Re-enable pdfium_unittests. dd132d8 Update externs in unicodenormalization.cpp to not have array sizes. c5f520c Get rid of g_timeMap global object. f964049 Fix an array size mismatch. 286ea1d Remove run-time calculation of hash constants in pdfium. a975cfa Get rid of fifteen copies of m_sTimeMap and their initializers. 2638428 Clean up bookmark related codes. 17b5cf4 Remove static declared SymbolDictCache. 07f10ae In windows printing, convert src bitmap to dest bitmap using CompositeBitmap. d66051b Remove g_NaN global variable and its constructor. 659a56b Get rid of FX_LPCSTR cast. 3fc36e8 Revert build of pdfium_unittests. 831e138 Fix pdfium buildbot build breakage from 336991d. 336991d Create pdfium_unittests binary. 6c06469 Refactor some code in CPDF_TextPage::ProcessTextObject(). 59dffb6 Fix new third_party/BUILD.gn to use the right build configuration. 06b087a Set the freetype options missing in the recent update. b3a3230 Add the license back to ftmodule.h after the freetype 2.5.4 upgrade. a09d50d Transform font size to user space for valid text size comparison. e4fc5ce Update freetype to 2.5.4. Committed: https://crrev.com/6ae67338f424ae28d3cd1344282dfb7391ad3025 Cr-Commit-Position: refs/heads/master@{#310158}

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: 8 (2 generated)
Tom Sepez
Lei, please review. Should be just a quick "stamp".
5 years, 11 months ago (2015-01-06 18:32:40 UTC) #2
Lei Zhang
My previous roll got reverted due to failure on Win8 GN. I didn't get a ...
5 years, 11 months ago (2015-01-06 18:37:13 UTC) #3
Lei Zhang
GN bots green -> lgtm
5 years, 11 months ago (2015-01-06 20:15:33 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/789613007/1
5 years, 11 months ago (2015-01-06 20:50:52 UTC) #6
commit-bot: I haz the power
Committed patchset #1 (id:1)
5 years, 11 months ago (2015-01-06 22:22:03 UTC) #7
commit-bot: I haz the power
5 years, 11 months ago (2015-01-06 22:23:05 UTC) #8
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/6ae67338f424ae28d3cd1344282dfb7391ad3025
Cr-Commit-Position: refs/heads/master@{#310158}

Powered by Google App Engine
This is Rietveld 408576698