Make DataTransferItemList indexed getter anonymous

Make DataTransferItemList indexed getter anonymous

Make DataTransferItemList indexed getter anonymous to match the latest
specification:
http://www.whatwg.org/specs/web-apps/current-work/multipage/dnd.html#datatransferitemlist

DataTransferItemList is not implemented in Firefox 23 or IE10. The
compatibility risk is low since developers usually use the [] operator
instead.

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

[arv (Not doing code reviews), 2013-08-12 15:15:48]

LGTM

[haraken, 2013-08-12 15:44:43]

LGTM. Needs an approval from an API owner.

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

https://codereview.chromium.org/22850002/diff/1/LayoutTests/fast/events/clipb...
LayoutTests/fast/events/clipboard-dataTransferItemList.html:76: function
makeClosure(_expectedData)

Nit: I think Blink doesn't use a variable name like _foo.

https://codereview.chromium.org/22850002/diff/1/LayoutTests/fast/events/clipb...
LayoutTests/fast/events/clipboard-dataTransferItemList.html:79: return function
(_data) {

Ditto.

https://codereview.chromium.org/22850002/diff/1/LayoutTests/fast/events/clipb...
LayoutTests/fast/events/clipboard-dataTransferItemList.html:142: <div
id="console"></div>

Nit: This won't be needed. Will be auto-inserted by js-test-pre.js.

[haraken, 2013-08-12 15:46:12]

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

https://codereview.chromium.org/22850002/diff/1/LayoutTests/fast/events/clipb...
LayoutTests/fast/events/clipboard-dataTransferItemList.html:142: <div
id="console"></div>
On 2013/08/12 15:44:43, haraken wrote:
> 
> Nit: This won't be needed. Will be auto-inserted by js-test-pre.js.

Sorry, I see your comment in https://codereview.chromium.org/22790003/. Ignore
my comment above.

[arv (Not doing code reviews), 2013-08-12 15:46:38]

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

https://codereview.chromium.org/22850002/diff/1/LayoutTests/fast/events/clipb...
LayoutTests/fast/events/clipboard-dataTransferItemList.html:142: <div
id="console"></div>
On 2013/08/12 15:44:43, haraken wrote:
> 
> Nit: This won't be needed. Will be auto-inserted by js-test-pre.js.

It is sometimes needed... especially when you do drag and drop because you need
it to come after all the test elements.

[do-not-use, 2013-08-13 10:49:44]

+jochen

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

lgtm

[commit-bot: I haz the power, 2013-08-13 13:59:35]

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

[commit-bot: I haz the power, 2013-08-13 15:57:16]

Change committed as 156027