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

Issue 23414007: [PDF] Fix printing crashes caused by font streams that don't support getMemoryBase(). (Closed)

Created:
7 years, 3 months ago by vandebo (ex-Chrome)
Modified:
7 years, 3 months ago
CC:
skia-review_googlegroups.com
Visibility:
Public.

Description

[PDF] Fix printing crashes caused by font streams that don't support getMemoryBase(). An apparently recently change started putting web fonts into SkStream objects that don't support getMemoryBase(). This change uses writeStream to avoid needing to call getMemoryBase(). BUG=chromium:274440 R=bungeman@google.com Committed: https://code.google.com/p/skia/source/detail?r=10918

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M src/pdf/SkPDFStream.cpp View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 8 (0 generated)
vandebo (ex-Chrome)
PTAL
7 years, 3 months ago (2013-08-26 16:52:46 UTC) #1
reed1
adding Ben for streamy stuff
7 years, 3 months ago (2013-08-26 17:03:55 UTC) #2
bungeman-skia
lgtm Having an extra buffer/copy along the way is somewhat annoying here. It would be ...
7 years, 3 months ago (2013-08-26 17:23:48 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://skia-tree-status.appspot.com/cq/vandebo@chromium.org/23414007/1
7 years, 3 months ago (2013-08-26 17:27:30 UTC) #4
commit-bot: I haz the power
Retried try job too often on Build-Mac10.7-Clang-x86-Release-Trybot for step(s) BuildMost http://173.255.115.253:10117/buildstatus?builder=Build-Mac10.7-Clang-x86-Release-Trybot&number=482
7 years, 3 months ago (2013-08-26 17:40:50 UTC) #5
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://skia-tree-status.appspot.com/cq/vandebo@chromium.org/23414007/1
7 years, 3 months ago (2013-08-26 17:42:08 UTC) #6
commit-bot: I haz the power
Retried try job too often on Build-Mac10.7-Clang-x86-Release-Trybot for step(s) BuildMost http://173.255.115.253:10117/buildstatus?builder=Build-Mac10.7-Clang-x86-Release-Trybot&number=485
7 years, 3 months ago (2013-08-26 17:55:02 UTC) #7
vandebo (ex-Chrome)
7 years, 3 months ago (2013-08-26 18:02:05 UTC) #8
Message was sent while issue was closed.
Committed patchset #1 manually as r10918 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698