Change it from a WebImage to WebData to indicate that we are returning a blob of PNG data.
Created attachment 85556 [details] Patch
Comment on attachment 85556 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=85556&action=review > Source/WebKit/chromium/public/WebClipboard.h:36 > #include "WebImage.h" Can we remove this?
(In reply to comment #2) > (From update of attachment 85556 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=85556&action=review > > > Source/WebKit/chromium/public/WebClipboard.h:36 > > #include "WebImage.h" > > Can we remove this? Should be possible, see the patch that added it: https://bugs.webkit.org/attachment.cgi?id=84843&action=prettypatch
(In reply to comment #3) > (In reply to comment #2) > > (From update of attachment 85556 [details] [details]) > > View in context: https://bugs.webkit.org/attachment.cgi?id=85556&action=review > > > > > Source/WebKit/chromium/public/WebClipboard.h:36 > > > #include "WebImage.h" > > > > Can we remove this? > > Should be possible, see the patch that added it: https://bugs.webkit.org/attachment.cgi?id=84843&action=prettypatch Looks like we'll have to forward declare WebImage because writeImage uses it, which is fine.
(In reply to comment #4) > (In reply to comment #3) > > (In reply to comment #2) > > > (From update of attachment 85556 [details] [details] [details]) > > > View in context: https://bugs.webkit.org/attachment.cgi?id=85556&action=review > > > > > > > Source/WebKit/chromium/public/WebClipboard.h:36 > > > > #include "WebImage.h" > > > > > > Can we remove this? > > > > Should be possible, see the patch that added it: https://bugs.webkit.org/attachment.cgi?id=84843&action=prettypatch > > Looks like we'll have to forward declare WebImage because writeImage uses it, which is fine. I will fix this before landing. Thanks!
Committed r80907: <http://trac.webkit.org/changeset/80907>