Bug 36072

Summary: [OpenVG] Add support for decoding and drawing images
Product: WebKit Reporter: Jakob Petsovits <jpetsovits>
Component: PlatformAssignee: Jakob Petsovits <jpetsovits>
Severity: Normal CC: elongbug, eric, krit, staikos
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Other   
OS: All   
Bug Depends on:    
Bug Blocks: 33987    
Description Flags
Patch staikos: review+

Description Jakob Petsovits 2010-03-12 14:16:06 PST
Image support is still missing in the OpenVG backend (as is ImageBuffer and lots of features, but those are subject to future bugs). As you might guess, this patch has a shot at that shortcoming. It's currently duplicating memory - both the decoded buffer and the VGImage are storing the same pixel data, which is bad. But the patch is large enough already, so I'll wait with the "better" version until this one has been landed. Image pattern drawing for arbitrary-size images is also not quite trivial and will come in a follow-up patch as well.
Comment 1 Jakob Petsovits 2010-03-12 14:18:02 PST
Created attachment 50633 [details]
Comment 2 Jakob Petsovits 2010-03-29 08:56:41 PDT
Created attachment 51918 [details]
Comment 3 Jakob Petsovits 2010-03-29 08:58:37 PDT
The above fixes a couple of issues noticed by Dirk, who suggests that Olliej or someone else should take a look at this patch as well. Still waiting for review, thanks!
Comment 4 George Staikos 2010-04-06 19:53:14 PDT
Looks good, and I know you have some additional patches to follow this which I hope will show up quickly. :)
Comment 5 Eric Seidel (no email) 2010-04-06 23:44:59 PDT
Attachment 51918 [details] was posted by a committer and has review+, assigning to Jakob Petsovits for commit.
Comment 6 Eric Seidel (no email) 2010-05-17 00:55:42 PDT
Looks like this has been in pending-commit for over a month.  Did this get landed already?  Should it just be commit-queued?
Comment 7 Jakob Petsovits 2010-05-17 11:23:46 PDT
Was not landed yet. I'll land it today, and will try harder to get the rest of OpenVG patches upstream as well.
Comment 8 Jakob Petsovits 2010-05-17 12:11:45 PDT
Committed r59619: <http://trac.webkit.org/changeset/59619>