Bug 28216 - [CAIRO] getImageData fails on a given rect bigger than the image rect
Summary: [CAIRO] getImageData fails on a given rect bigger than the image rect
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKitGTK (show other bugs)
Version: 525.x (Safari 3.1)
Hardware: PC OS X 10.5
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-08-12 03:27 PDT by Dirk Schulze
Modified: 2009-08-13 10:52 PDT (History)
1 user (show)

See Also:


Attachments
Cairo fix of getImageData (1.29 KB, patch)
2009-08-12 03:27 PDT, Dirk Schulze
eric: review-
Details | Formatted Diff | Diff
Cairo fix of getImageData (1.58 KB, patch)
2009-08-12 11:08 PDT, Dirk Schulze
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Dirk Schulze 2009-08-12 03:27:06 PDT
Created attachment 34650 [details]
Cairo fix of getImageData

[CAIRO] getImageData fails on a given rect bigger than the image rect.

I found this bug during my work on SVG feGaussianBlur. Currently only canvas could test it, but we don't allow the it there to make a bigger rect then the image rect.
The problem is, that the destination image and source image were mixed up on creating the imageData. This patch just follows to the behavior of the other ports.
Comment 1 Eric Seidel (no email) 2009-08-12 10:18:19 PDT
Comment on attachment 34650 [details]
Cairo fix of getImageData

Test case?  Or at least an explanation of why one is impossible.
Comment 2 Dirk Schulze 2009-08-12 11:08:01 PDT
Created attachment 34677 [details]
Cairo fix of getImageData

a better Changelog...
Comment 3 Eric Seidel (no email) 2009-08-13 09:53:06 PDT
Comment on attachment 34677 [details]
Cairo fix of getImageData

LGTM.  Not marking cq+ since you're a committer.  You can mark it yourself if you'd like the queue to land this for you.
Comment 4 Eric Seidel (no email) 2009-08-13 10:17:27 PDT
Exception: Unknown committer: krit@webkit.org

Are you not a committer?  Or do we just not have you in:
WebKitTools/Scripts/modules/committers.py yet?

If not, feel free to add yourself to the committer list.  No review needed.
Comment 5 Dirk Schulze 2009-08-13 10:39:49 PDT
(In reply to comment #4)
> Exception: Unknown committer: krit@webkit.org
> 
> Are you not a committer?  Or do we just not have you in:
> WebKitTools/Scripts/modules/committers.py yet?
> 
> If not, feel free to add yourself to the committer list.  No review needed.

Added myself to the list.
Comment 6 Eric Seidel (no email) 2009-08-13 10:52:33 PDT
Comment on attachment 34677 [details]
Cairo fix of getImageData

Clearing flags on attachment: 34677

Committing to http://svn.webkit.org/repository/webkit/trunk ...
	M	WebCore/ChangeLog
	M	WebCore/platform/graphics/cairo/ImageBufferCairo.cpp
Committed r47211
	M	WebCore/ChangeLog
	M	WebCore/platform/graphics/cairo/ImageBufferCairo.cpp
r47211 = 43323f18d890228b2275a1ae4d4f9d211d2c001d (trunk)
No changes between current HEAD and refs/remotes/trunk
Resetting to the latest refs/remotes/trunk
http://trac.webkit.org/changeset/47211
Comment 7 Eric Seidel (no email) 2009-08-13 10:52:37 PDT
All reviewed patches have been landed.  Closing bug.