Updates file type selector for fileSystem API

chrome/common/extensions/api/file_system.idl

Index: chrome/common/extensions/api/file_system.idl
diff --git a/chrome/common/extensions/api/file_system.idl b/chrome/common/extensions/api/file_system.idl
index 51a93a6c39f3cea39c6afa53db43e625276a93b2..6a261f035c02b346e555df0d9bd370e75cf6cee5 100644
--- a/chrome/common/extensions/api/file_system.idl
+++ b/chrome/common/extensions/api/file_system.idl
@@ -22,6 +22,19 @@ namespace fileSystem {
     // 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);