Update sync_file_system.idl to use a regular array instead of an EntryArray

Update sync_file_system.idl to use a regular array instead of an EntryArray

After Blink r155749, the File API's readEntries() now returns a Entry[]
instead of an EntryArray type, to match the specification.

This patch updates the sync_file_system extension API accordingly to
fix breakage.

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=216649

[do-not-use, 2013-08-08 15:35:33]

This fixes the following browser test:
SyncFileSystemApiTest.GetFileStatuses

[not at google - send to devlin, 2013-08-08 15:46:39]

lgtm because this fixes a test, but we probably want better than object[] there.

+kinuko to follow this up.

[do-not-use, 2013-08-08 15:49:17]

https://codereview.chromium.org/22639003/diff/1/chrome/common/extensions/api/...
File chrome/common/extensions/api/sync_file_system.idl (right):

https://codereview.chromium.org/22639003/diff/1/chrome/common/extensions/api/...
chrome/common/extensions/api/sync_file_system.idl:173: static void
getFileStatuses(object[] fileEntries,
Yes, the singular version above (getFileStatus) is using 'object', which is why
I used object[] here. I am really not familiar with this part of the code so
sorry if this is not the right / best type.

[not at google - send to devlin, 2013-08-08 15:56:03]

Yes just submit. We can worry about what it should be later.

[kinuko, 2013-08-08 16:19:49]

Thanks for the quick fix!  Yes we can take care of it after it's submitted.

[do-not-use, 2013-08-08 18:15:17]

I updated the CL to unskip the browser test in the same patch (fmalita@ skipped
it to allow Blink to roll).

I am waiting for Blink to roll before triggering the commit queue.

[f(malita), 2013-08-08 18:25:34]

On 2013/08/08 18:15:17, Christophe Dumez wrote:
> I updated the CL to unskip the browser test in the same patch (fmalita@
skipped
> it to allow Blink to roll).
> 
> I am waiting for Blink to roll before triggering the commit queue.

Thanks, Christophe!

I landed a speculative roll (155754) since the bots were taking too long and
we're falling behind. Let's wait a bit more with this to see if the roll sticks.

[commit-bot: I haz the power, 2013-08-08 22:00:30]

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

[commit-bot: I haz the power, 2013-08-09 10:15:47]

Change committed as 216649