Add support for DataTransferItemList.remove() method

Add support for DataTransferItemList.remove() method

Add support for DataTransferItemList.remove() method as per the latest
specification:
http://www.whatwg.org/specs/web-apps/current-work/multipage/dnd.html#dom-datatransferitemlist-remove

Firefox 23 and IE10 do not support DataTransferItemList yet.

BUG=271424
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=156015

[arv (Not doing code reviews), 2013-08-13 13:19:47]

LGTM

[haraken, 2013-08-13 13:25:44]

LGTM. Needs an approval from an API owner.

https://codereview.chromium.org/23051003/diff/1/LayoutTests/fast/events/clipb...
File LayoutTests/fast/events/clipboard-dataTransferItemList-remove.html (right):

https://codereview.chromium.org/23051003/diff/1/LayoutTests/fast/events/clipb...
LayoutTests/fast/events/clipboard-dataTransferItemList-remove.html:15:
event.preventDefault();

Nit: Why do you need preventDefault() in copy() but don't need preventDefault()
in paste() ?

[jochen (gone - plz use gerrit), 2013-08-13 13:36:10]

lgtm

[do-not-use, 2013-08-13 13:51:21]

https://codereview.chromium.org/23051003/diff/1/LayoutTests/fast/events/clipb...
File LayoutTests/fast/events/clipboard-dataTransferItemList-remove.html (right):

https://codereview.chromium.org/23051003/diff/1/LayoutTests/fast/events/clipb...
LayoutTests/fast/events/clipboard-dataTransferItemList-remove.html:15:
event.preventDefault();
On 2013/08/13 13:25:44, haraken wrote:
> 
> Nit: Why do you need preventDefault() in copy() but don't need
preventDefault()
> in paste() ?

I need preventDefault() in copy() because I am editing the
clipboard/DataTransferItemList.

The spec (http://dev.w3.org/2006/webapi/clipops/clipops.html#copy-event) says:
"If there is no selection, the clipboard is not modified except if the default
action is prevented and the script has added entries in the
DataTransferItemList, for example by calling the setData() method."

[haraken, 2013-08-13 14:05:11]

On 2013/08/13 13:51:21, Christophe Dumez wrote:
>
https://codereview.chromium.org/23051003/diff/1/LayoutTests/fast/events/clipb...
> File LayoutTests/fast/events/clipboard-dataTransferItemList-remove.html
(right):
> 
>
https://codereview.chromium.org/23051003/diff/1/LayoutTests/fast/events/clipb...
> LayoutTests/fast/events/clipboard-dataTransferItemList-remove.html:15:
> event.preventDefault();
> On 2013/08/13 13:25:44, haraken wrote:
> > 
> > Nit: Why do you need preventDefault() in copy() but don't need
> preventDefault()
> > in paste() ?
> 
> I need preventDefault() in copy() because I am editing the
> clipboard/DataTransferItemList.
> 
> The spec (http://dev.w3.org/2006/webapi/clipops/clipops.html#copy-event) says:
> "If there is no selection, the clipboard is not modified except if the default
> action is prevented and the script has added entries in the
> DataTransferItemList, for example by calling the setData() method."

Thanks, makes sense.

[commit-bot: I haz the power, 2013-08-13 14:08:24]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/ch.dumez@sisa.samsung.com/23051003/1

[commit-bot: I haz the power, 2013-08-13 14:37:33]

Change committed as 156015