|Summary:||[OpenVG] Add support for decoding and drawing images|
|Product:||WebKit||Reporter:||Jakob Petsovits <jpetsovits>|
|Component:||Platform||Assignee:||Jakob Petsovits <jpetsovits>|
|Severity:||Normal||CC:||elongbug, eric, krit, staikos|
|Version:||528+ (Nightly build)|
|Bug Depends on:|
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 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.