Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(1808)

Issue 4798069: Protect much of SkDevice and clarify usage of drawDevice. (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
13 years, 4 months ago by bungeman
Modified:
13 years, 3 months ago
Reviewers:
bsalomon, reed1
CC:
skia-review_googlegroups.com
Base URL:
http://skia.googlecode.com/svn/trunk/
Visibility:
Public.

Patch Set 1 #

Patch Set 2 : Chrome builds. #

Patch Set 3 : Hide as much as possible for now. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+97 lines, -84 lines) Patch
M include/core/SkDevice.h View 1 2 8 chunks +82 lines, -75 lines 0 comments Download
M include/pdf/SkPDFDevice.h View 1 1 chunk +6 lines, -0 lines 0 comments Download
M src/core/SkCanvas.cpp View 1 6 chunks +9 lines, -9 lines 0 comments Download

Messages

Total messages: 6
bungeman
13 years, 4 months ago (2011-08-03 14:00:20 UTC) #1
bsalomon
On 2011/08/03 14:00:20, bungeman wrote: LGTM
13 years, 4 months ago (2011-08-03 14:07:41 UTC) #2
bungeman
This is as much as can be hidden at the moment in SkDevice with Skia ...
13 years, 3 months ago (2011-08-08 18:31:19 UTC) #3
reed1
LGTM
13 years, 3 months ago (2011-08-08 18:34:20 UTC) #4
bsalomon
lgtm
13 years, 3 months ago (2011-08-08 18:57:02 UTC) #5
bungeman
13 years, 3 months ago (2011-08-08 21:21:43 UTC) #6
Committed revision 2066.
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b