You need to
before you can comment on or make changes to this bug.
[Chromium] Implement canvas.toDataURL("image/webp")
Created an attachment (id=111129) [details]
Created an attachment (id=111214) [details]
Created an attachment (id=111573) [details]
(From update of attachment 111573 [details])
View in context: https://bugs.webkit.org/attachment.cgi?id=111573&action=review
> + return "data:,";
Should "data:," be a constant shared by all the uses in this function?
Indeed, could be shared yes. In a future patch, I plan to factor it out and
make this function return a boolean.
Created an attachment (id=113629) [details]
Patch for landing.
(From update of attachment 113629 [details])
Clearing flags on attachment: 113629
Committed r99319: <http://trac.webkit.org/changeset/99319>
All reviewed patches have been landed. Closing bug.
(In reply to comment #5)
> In a future patch, I plan to factor it out and make this function return a boolean.