CRX FileSystem Pepper private API

ppapi/api/private/ppb_ext_crx_file_system_private.idl

+/* Copyright (c) 2013 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.
+ */
+
+[generate_thunk]
+
+/**
+ * This file contains the <code>PPB_Ext_CrxFileSystem_Private</code> interface.
+ */
+label Chrome {
+  M28 = 0.1
+};
+
+/* <code>PPB_Ext_CrxFileSystem_Private</code> interface */
+interface PPB_Ext_CrxFileSystem_Private {

[yzshen1, 2013/04/30 19:31:19]

IMO, you could remove Ext from the name because Crx already indicates that this
is for extensions.
Besides, my original plan is to reserve PPB_Ext_ and pp::ext:: for only those
Pepper equivalents of extension JS APIs.

But I am fine if you would like to leave it as it is and revisit it later if
necessary.

[victorhsieh, 2013/04/30 22:04:32]

Yeah, I saw you have Ext there so I just name it that way.  I remember you said
you will fit this into your ext work later, so let's rename it together at that
time.

+  /**
+   * Create() creates a CRX filesystem resource.  It may fail when called from

[yzshen1, 2013/04/30 19:31:19]

Is "may" precise here?

[victorhsieh, 2013/04/30 22:04:32]

It's referenced from pp::FileSystem.  I think it's "may" because the operation
can fail on later Open.

+   * non-extension context.
+   */
+  PP_Resource Create([in] PP_Instance insance);
+
+  /**
+   * Open() opens the CRX file system for the current extension.  It will fail
+   * when called from non-extension context.
+   *
+   * @return An int32_t containing an error code from <code>pp_errors.h</code>.
+   */
+  int32_t Open([in] PP_Resource crxfs, [in] PP_CompletionCallback callback);
+};