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 7d9928eb7456edceb02499f1f9987ba1c4a8c938..3f913e205ef66c959ab4f6d44ea00354dbf7694e 100644
--- a/chrome/common/extensions/api/file_system.idl
+++ b/chrome/common/extensions/api/file_system.idl
@@ -18,6 +18,27 @@ namespace fileSystem {
     //
     // 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? suggestedPath;

[benwells, 2012/07/12 06:10:57]

Cool :) Feel free to do in a separate patch, or in this one.

Should be suggestedFilename, not Path. We've avoided making paths a part of the
API.

[thorogood, 2012/07/13 04:17:27]

Done in other patch.

+
+    // The optional list of accepted mime-types for this file opener, grouped
+    // by type.
+    //
+    // For example, ['image/*', 'application/json,.jso'] would display two type

[benwells, 2012/07/12 06:10:57]

This is not nitting on the text, just me trying to understand. 

My understanding is that the syntax is 
- an array of filters
- each filter is a string, being made up of a comma separated list of types
- each type is either
  - an accepted MIME type filter
  - an accepted extension

Is that right?

[thorogood, 2012/07/13 04:17:27]

Yes, that's right. The aim here is to allow for all sorts of different
configurations.

Ideally, I'd allow the API to also accept a single string argument, which would
be treated as an array of length one, for a "simple case". I'm not sure if this
is really practical or worth it though.

+    // options - one accepting images - and the other accepting json 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;
+
+    // Does this file picker accept all file types, in addition to the options
+    // specified in the accepts argument?

[benwells, 2012/07/12 06:10:57]

Nit: don't phrase as a question.

[thorogood, 2012/07/13 04:17:27]

Done.

+    //
+    // The default is true.
+    boolean? acceptsAllTypes;
   };
   callback GetDisplayPathCallback = void (DOMString displayPath);
   callback FileEntryCallback = void ([instanceOf=fileEntry] object fileEntry);