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

Issue 23437004: [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=10928

Patch Set 1 #

Patch Set 2 : Add comment #

Unified diffs Side-by-side diffs Delta from patch set Stats (+13 lines, -8 lines) Patch
M src/pdf/SkPDFStream.h View 1 1 chunk +2 lines, -0 lines 0 comments Download
M src/pdf/SkPDFStream.cpp View 4 chunks +11 lines, -8 lines 0 comments Download

Messages

Total messages: 6 (0 generated)
vandebo (ex-Chrome)
Another try of https://codereview.chromium.org/23414007/
7 years, 3 months ago (2013-08-26 20:53:11 UTC) #1
vandebo (ex-Chrome)
+djsollen
7 years, 3 months ago (2013-08-26 20:54:27 UTC) #2
bungeman-skia
lgtm
7 years, 3 months ago (2013-08-26 21:15:31 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/23437004/4001
7 years, 3 months ago (2013-08-26 21:20:54 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=492
7 years, 3 months ago (2013-08-26 21:56:53 UTC) #5
vandebo (ex-Chrome)
7 years, 3 months ago (2013-08-26 22:52:18 UTC) #6
Message was sent while issue was closed.
Committed patchset #2 manually as r10928 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698