New issue
Advanced search Search tips
Starred by 5 users

Issue metadata

Status: WontFix
Owner:
Closed: Apr 6
Cc:
Area: Playback , Record
Priority: Medium
Type: Idea



Sign in to add a comment

SkPicture should have an offset/origin

Project Member Reported by senorblanco@chromium.org, Aug 5 2014

Issue description

There are use cases (such as SkPictureImageFilter and Blink's
DisplayList) where the picture contains primitives which have a non-0,0
origin. They apply an offset transform at record time, and the inverse 
transform at playback time.

This seems common enough that it might be useful to be able to specify
an origin for SkPicture, so that any tiling performed on the contained
primitives starts at that origin rather than 0,0.

It could be a constructor which takes an SkIRect, or a setOrigin/setOffset
on SkPicture.

Failing that, we should add an offset to SkPictureImageFilter.
 
Project Member

Comment 1 by senorblanco@chromium.org, Aug 5 2014

Summary: SkPicture should have an offset/origin (was: SkPicture should have an offset)
Project Member

Comment 2 by senorblanco@chromium.org, Aug 5 2014

Note: for the tiling SkPictureShader use case (https://codereview.chromium.org/437393003/), it might be useful to make this an SkRect, rather than an SkIRect.
Project Member

Comment 3 by hcm@google.com, Oct 14 2014

Cc: mtkl...@google.com
Labels: -Type-Defect Type-Idea Area-Record Area-Playback
Now an idea for SkRecord I assume?
Project Member

Comment 4 by mtkl...@google.com, Oct 14 2014

Um, oddly enough, this is probably one aspect of SkPicture that SkRecord wouldn't be involved in, but spiritually sure.  "SkPicture should have an offset/origin" is still the best way to describe this.
Project Member

Comment 5 by reed@chromium.org, Nov 22 2015

Cc: fmalita@chromium.org
Pictures now have a rect-bounds, rather than just width/height. Does this resolve this issue?
Project Member

Comment 6 by senorblanco@chromium.org, Apr 6

Status: WontFix (was: Accepted)
I don't think this is a priority anymore.

Sign in to add a comment