All the elements in the array are nsIFile objects, so it makes sense to use nsCOMArray and thus not have to worry as much about refcounting or releasing each element in the array when the stream goes away.
Created attachment 209912 [details] [diff] [review] Patch
Comment on attachment 209912 [details] [diff] [review] Patch nice, thanks! r+sr=darin
Patch checked in with a change to remove the temporary nsIFile* used when adding each file to mArray (the result of review comments on similar code in bug 309296), so marking FIXED.