Updates file type selector for fileSystem API

chrome/common/extensions/api/file_system.idl

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 // File-level comment to appease parser. Eventually this will not be necessary.
 
 namespace fileSystem {
   dictionary ChooseFileOptions {
     // Type of the prompt to show. Valid types are 'openFile',
     // 'openWritableFile' or 'saveFile'.
     //
     // Both 'openFile' and 'openWritableFile' will prompt the user to open an
     // existing file, with 'openFile' returning a read-only FileEntry on
     // success. 'saveFile' will prompt the user to choose an existing file or
     // a new file, and will return a writable FileEntry.
     // Calls to chooseFile with either 'openWritableFile' or 'saveFile' will
     // fail unless the application has the fileSystemWrite permission.
     //
     // The default is 'openFile'.
     DOMString? type;
 
     // The suggested file name that will be presented to the user as the
     // default name to read or write. This is optional.
     DOMString? suggestedName;
+
+    // The optional list of accepted mime-types for this file opener, grouped
+    // by type. For example, <code>['image/*', 'text/html,.jso']</code>

[Mihai Parparita -not on Chrome, 2012/07/30 23:33:03]

Apologies if this was discussed already, but the embedding of arrays inside of
strings seems a bit hacky, and error prone for the developer. How about
something a bit more structured. Make the param into:

AcceptOption[]? accepts;

With AcceptOption being:

dictionary AcceptOption {
  // Optional, if not present, a description will be auto-generated
  DOMString? description;
  // One of mimeTypes or extensions must contain at least one element.
  DOMString[]? mimeTypes;
  DOMString[]? extensions;
}

+    // would display two type options - one accepting images - and the other
+    // accepting HTML files and those with the extension 'jso'.
+    //
+    // This field is optional. However, it is an error to not specify this
+    // field (or pass an empty array) but pass acceptsAllTypes as false.
+    DOMString[]? accepts;
+
+    // Whether to accept all file types, in addition to the options specified
+    // in the accepts argument. The default is true.
+    boolean? acceptsAllTypes;
   };
   callback GetDisplayPathCallback = void (DOMString displayPath);
   callback FileEntryCallback = void ([instanceOf=FileEntry] object fileEntry);
   callback IsWritableCallback = void (boolean isWritable);
 
   interface Functions {
     // Get the display path of a FileEntry object. The display path is based on
     // the full path of the file on the local file system, but may be made more
     // readable for display purposes.
     static void getDisplayPath([instanceOf=FileEntry] object fileEntry,
                                GetDisplayPathCallback callback);
 
     // Get a writable FileEntry from another FileEntry. This call will fail
     // if the application does not have the fileSystemWrite permission.
     static void getWritableFileEntry([instanceOf=FileEntry] object fileEntry,
                                      FileEntryCallback callback);
 
     // Gets whether this FileEntry is writable or not.
     static void isWritableFileEntry([instanceOf=FileEntry] object fileEntry,
                                     IsWritableCallback callback);
 
     // Ask the user to choose a file.
     static void chooseFile(optional ChooseFileOptions options,
                            FileEntryCallback callback);
   };
 };