Move DirectoryReader::ReadEntries to FileRef::ReadDirectoryEntries

ppapi/api/ppb_file_ref.idl

 /* Copyright (c) 2012 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.
  */
 
 /**
  * This file defines the API to create a file reference or "weak pointer" to a
  * file in a file system.
  */
 
@@ skipping 167 matching lines @@
    * populated with information about the file or directory.
    * @param[in] callback A <code>PP_CompletionCallback</code> to be called upon
    * completion of Query().
    *
    * @return An int32_t containing an error code from <code>pp_errors.h</code>.
    */
   [version=1.1]
   int32_t Query([in] PP_Resource file_ref,
                 [out] PP_FileInfo info,
                 [in] PP_CompletionCallback callback);
+
+  /* Reads all entries in a directory.

[dmichael (off chromium), 2013/05/02 19:43:54]

nit: please use doxygen comment style: /**
(see others)

[hamaji, 2013/05/02 20:14:49]

Done.

+   *
+   * @param[in] file_ref A <code>PP_Resource</code> corresponding to a directory
+   * reference.
+   * @param[in] output An output array which will receive
+   * <code>PP_DirectoryEntry</code> objects on success.
+   * @param[in] callback A <code>PP_CompletionCallback</code> to run on
+   * completion.
+   *
+   * @return An int32_t containing an error code from <code>pp_errors.h</code>.
+   */
+  [version=1.1]

[teravest, 2013/05/02 17:32:10]

You'll want to make this version 1.2=M29 unless this gets in really soon.

[hamaji, 2013/05/02 18:49:40]

We want this real soon and I'd like to land this before the branch cut.

+  int32_t ReadEntries([in] PP_Resource file_ref,
+                      [in] PP_ArrayOutput output,
+                      [in] PP_CompletionCallback callback);

[dmichael (off chromium), 2013/05/02 19:43:54]

This is a little bit tricky...  We would usually rev the interface for adding a
function. Maybe we can get away with this if it ships in 28 with Query. The risk
is that some app (e.g. in NaCl) decides to start using it because it showed up
in the SDK, but runs against a version of Chrome that doesn't actually have
support yet. Given that the SDK will probably not have this any sooner than Dev
channel/Canary...  we can probably take that chance this time.

But if we miss 28, we'll have to make FileRef go to version 1.2 for this
function.

[hamaji, 2013/05/02 20:14:49]

OK, I'll do my best to land this soon.

 };