[webkit-reviews] review denied: [Bug 47482] Add webkitGetDataBlob/webkitSetDataBlob to event.dataTransfer : [Attachment 71825] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Oct 27 15:15:35 PDT 2010


Dmitry Titov <dimich at chromium.org> has denied Daniel Cheng
<dcheng at chromium.org>'s request for review:
Bug 47482: Add webkitGetDataBlob/webkitSetDataBlob to event.dataTransfer
https://bugs.webkit.org/show_bug.cgi?id=47482

Attachment 71825: Patch
https://bugs.webkit.org/attachment.cgi?id=71825&action=review

------- Additional Comments from Dmitry Titov <dimich at chromium.org>
I think there should be a better proposal before this kind of patch can land.

The linked post
(http://lists.whatwg.org/htdig.cgi/whatwg-whatwg.org/2010-June/026910.html)
only contains a question about what would be the preferred shape of API for
exchanging non-text data with clipboard. I think it didn't get responses
because context was unclear.

It is not exactly clear what kind of use case these APIs are going to enable,
what kind of data are we talkign about, sizes and speeds (if data is big or
produced on demand, then perhaps it should not be a synchronous API?) etc.  It
is hard to say, based on information in this bug, whether or not these APIs are
all right since it's unclear what exact data transfer scenarios they are going
to be used for.

If this is a part of a bigger modification, it could be also useful to see what
it is (since it mentions non-text data in clipboardData). In short, I think
this requires a proposal or design doc.


More information about the webkit-reviews mailing list