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

Issue 6532056: In render_pictures tiled rendering, draw a separate PNG for each tile. (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
11 years, 9 months ago by Leon
Modified:
11 years, 9 months ago
Reviewers:
tfarina1, EricB, reed1
CC:
skia-review_googlegroups.com
Base URL:
https://skia.googlecode.com/svn/trunk
Visibility:
Public.

Description

In render_pictures tiled rendering, draw a separate PNG for each tile. Since the passed in picture may represent an image which is too large to be represented on the GPU, never create such a large canvas. Instead, after drawing to each tile, create a file showing just that tile. Committed: https://code.google.com/p/skia/source/detail?r=5603

Patch Set 1 #

Total comments: 2
Unified diffs Side-by-side diffs Delta from patch set Stats (+66 lines, -53 lines) Patch
M tools/PictureBenchmark.cpp View 1 chunk +1 line, -1 line 0 comments Download
M tools/PictureRenderer.h View 7 chunks +25 lines, -12 lines 0 comments Download
M tools/PictureRenderer.cpp View 6 chunks +31 lines, -24 lines 2 comments Download
M tools/render_pictures_main.cpp View 2 chunks +9 lines, -16 lines 0 comments Download

Messages

Total messages: 4
Leon
11 years, 9 months ago (2012-09-19 19:32:51 UTC) #1
EricB
LGTM
11 years, 9 months ago (2012-09-20 11:56:26 UTC) #2
tfarina1
https://codereview.appspot.com/6532056/diff/1/tools/PictureRenderer.cpp File tools/PictureRenderer.cpp (right): https://codereview.appspot.com/6532056/diff/1/tools/PictureRenderer.cpp#newcode102 tools/PictureRenderer.cpp:102: bool PictureRenderer::write(SkCanvas* canvas, SkString path) const { should we ...
11 years, 9 months ago (2012-09-20 15:47:48 UTC) #3
Leon
11 years, 9 months ago (2012-09-20 15:56:20 UTC) #4
https://codereview.appspot.com/6532056/diff/1/tools/PictureRenderer.cpp
File tools/PictureRenderer.cpp (right):

https://codereview.appspot.com/6532056/diff/1/tools/PictureRenderer.cpp#newco...
tools/PictureRenderer.cpp:102: bool PictureRenderer::write(SkCanvas* canvas,
SkString path) const {
On 2012/09/20 15:47:48, tfarina1 wrote:
> should we pass |path| by const-ref?

We could, but if you look down below, you'll see that I actually modify it. In
that case I would need to make the copy manually.
Sign in to reply to this message.

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