PPAPI: Add new PPB_FileRef.MakeDirectory to support exclusive operation

ppapi/api/ppb_file_ref.idl

Index: ppapi/api/ppb_file_ref.idl
diff --git a/ppapi/api/ppb_file_ref.idl b/ppapi/api/ppb_file_ref.idl
index d9226471a9f23cc6e9c862369ef905c25a9731b5..bec3d804a9c8fbd1be489825ceb69cec55072b63 100644
--- a/ppapi/api/ppb_file_ref.idl
+++ b/ppapi/api/ppb_file_ref.idl
@@ -10,7 +10,8 @@
 
 label Chrome {
   M14 = 1.0,
-  M28 = 1.1
+  M28 = 1.1,
+  M34 = 1.2
 };
 
 /**
@@ -116,6 +117,27 @@ interface PPB_FileRef {
                         [in] PP_CompletionCallback callback);
 
   /**
+   * MakeDirectoryExclusive() makes a new directory in the file system as well
+   * as any parent directories if the <code>make_ancestors</code> argument is
+   * <code>PP_TRUE</code>.  It is not valid to make a directory in the external
+   * file system.
+   *
+   * @param[in] file_ref A <code>PP_Resource</code> corresponding to a file
+   * reference.
+   * @param[in] make_ancestors A <code>PP_Bool</code> set to
+   * <code>PP_TRUE</code> to make ancestor directories or <code>PP_FALSE</code>
+   * if ancestor directories are not needed.
+   *
+   * @return An int32_t containing an error code from <code>pp_errors.h</code>.
+   * Fails if the directory already exists or if ancestor directories do not
+   * exist and <code>make_ancestors</code> was passed as <code>PP_FALSE</code>.
+   */
+  [version=1.2]
+  int32_t MakeDirectoryExclusive([in] PP_Resource directory_ref,

[raymes, 2013/12/19 23:10:23]

Can we instead just add a PP_Bool to MakeDirectory() which specifies whether or
not an error should be thrown if the directory exists?

Also the comment on this function doesn't seem to explain it properly :p

[nhiroki, 2013/12/20 01:54:20]

Oh, I see. I assumed we have to add new function with new name when we want to
change an existing API in order to prevent plugins from getting broken, but now
I understand the way from other idl files.

Anyway, I'll fix this according to your comment.

+                                 [in] PP_Bool make_ancestors,
+                                 [in] PP_CompletionCallback callback);
+
+  /**
    * Touch() Updates time stamps for a file.  You must have write access to the
    * file if it exists in the external filesystem.
    *