Expose FileList in Worker contexts.

Expose FileList in Worker contexts.

The File API spec exposes FileList in both Window and Worker contexts,

 http://dev.w3.org/2006/webapi/FileAPI/#filelist-section

Follow up and have the implementation also provide the latter.

R=michaeln@chromium.org,jsbell@chromium.org
BUG=339441
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=177456

[sof, 2014-06-30 10:33:37]

Please take a look.

[jsbell, 2014-07-01 16:37:18]

lgtm but I'm not intimately familiar with the file tests and I'm not in OWNERS

https://codereview.chromium.org/349633005/diff/1/LayoutTests/fast/files/worke...
File LayoutTests/fast/files/workers/resources/worker-read-file-async.js (right):

https://codereview.chromium.org/349633005/diff/1/LayoutTests/fast/files/worke...
LayoutTests/fast/files/workers/resources/worker-read-file-async.js:4: {
Nit: place this on the same line as the function() for function expressions

https://codereview.chromium.org/349633005/diff/1/LayoutTests/fast/files/worke...
LayoutTests/fast/files/workers/resources/worker-read-file-async.js:14: var tests
= { };
We seem to more consistently write {}

https://codereview.chromium.org/349633005/diff/1/LayoutTests/fast/files/worke...
LayoutTests/fast/files/workers/resources/worker-read-file-async.js:15: for (var
i = 0; i < files.length; i++) {
Note: Can optionally skip the brace for a single line body.

https://codereview.chromium.org/349633005/diff/1/LayoutTests/fast/files/worke...
LayoutTests/fast/files/workers/resources/worker-read-file-async.js:22: }
Nit: ; after assignment statement

[sof, 2014-07-02 12:29:26]

On 2014/07/01 16:37:18, jsbell wrote:
> lgtm but I'm not intimately familiar with the file tests and I'm not in OWNERS
> 

Thanks for the review; ps#2 follows up on the comments made.

Sorry, I thought you were in Source/core/OWNERS. michaeln@: does exposing
FileList to Workers look acceptable to you? (they're already cloneable.)

[michaeln, 2014-07-02 23:38:06]

oh!! r/s lgtm2

[sof, 2014-07-03 05:14:59]

The CQ bit was checked by sigbjornf@opera.com

[commit-bot: I haz the power, 2014-07-03 05:16:22]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/sigbjornf@opera.com/349633005/20001

[commit-bot: I haz the power, 2014-07-03 07:38:05]

Change committed as 177456