PPAPI: Add new PPB_FileRef.MakeDirectory to support exclusive operation

ppapi/cpp/file_ref.h

Index: ppapi/cpp/file_ref.h
diff --git a/ppapi/cpp/file_ref.h b/ppapi/cpp/file_ref.h
index 452d2a61321866ba44ec2f70d3cf6b9db78417eb..362dd2a619eda8fd28d493e5369d3ffbd28da5c7 100644
--- a/ppapi/cpp/file_ref.h
+++ b/ppapi/cpp/file_ref.h
@@ -115,6 +115,22 @@ class FileRef : public Resource {
   /// Succeeds if the directory already exists.
   int32_t MakeDirectoryIncludingAncestors(const CompletionCallback& cc);
 
+  /// MakeDirectoryExclusive() makes a new directory in the file system as well
+  /// as any parent directories. It is not valid to make a directory in the
+  /// external file system.
+  ///
+  /// @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.
+  /// @param[in] cc A <code>CompletionCallback</code> to be called upon
+  /// completion of MakeDirectoryIncludingAncestors().
+  ///
+  /// @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>.
+  int32_t MakeDirectoryExclusive(PP_Bool make_ancestors,

[teravest, 2013/12/19 17:18:20]

It's sad that this looks different than
MakeDirectory/MakeDirectoryIncludingAncestors above.

What do you think of
MakeDirectoryExclusive/MakeDirectoryExclusiveIncludingAncestors?

It's a long name, but at least it would be consistent.

Otherwise, this should use "bool" instead of "PP_Bool".

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

It seems a bit weird that we made a whole new function here instead of just
adding a bool to MakeDirectory =\ Not sure the reasons for that.

My preference would be to just have MakeDirectory(bool make ancestors, bool
exclusive, const CompletionCallback& cc); The existing functions can be
deprecated and just call through that one. What do you guys think?

+                                 const CompletionCallback& cc);
+
   /// Touch() Updates time stamps for a file.  You must have write access to the
   /// file if it exists in the external filesystem.
   ///