Add synchronous directory listing to dart:io Directory.

runtime/bin/directory.h

 // Copyright (c) 2012, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 #ifndef BIN_DIRECTORY_H_
 #define BIN_DIRECTORY_H_
 
 #include "bin/builtin.h"
 #include "bin/dartutils.h"
 #include "platform/globals.h"
 #include "platform/thread.h"
 
 class DirectoryListing {
  public:
+  virtual bool HandleDirectory(char* dir_name) = 0;
+  virtual bool HandleFile(char* file_name) = 0;
+  virtual bool HandleError(const char* dir_name) = 0;
+
+  virtual ~DirectoryListing() {}
+};
+
+
+class DirectoryAsyncListing : public DirectoryListing {

[Mads Ager (google), 2013/01/04 10:00:25]

Can we make this AsyncDirectoryListing? I think that reads better.

[Bill Hesse, 2013/01/04 13:49:27]

Done.

+ public:
   enum Response {
     kListDirectory = 0,
     kListFile = 1,
     kListError = 2,
     kListDone = 3
   };
 
-  explicit DirectoryListing(Dart_Port response_port)
+  explicit DirectoryAsyncListing(Dart_Port response_port)
       : response_port_(response_port) {}
-  bool HandleDirectory(char* dir_name);
-  bool HandleFile(char* file_name);
-  bool HandleError(const char* dir_name);
+  virtual bool HandleDirectory(char* dir_name);
+  virtual bool HandleFile(char* file_name);
+  virtual bool HandleError(const char* dir_name);
 
+  virtual ~DirectoryAsyncListing() {}

[Mads Ager (google), 2013/01/04 10:00:25]

Destructor above methods, please.

http://google-styleguide.googlecode.com/svn/trunk/cppguide.xml#Declaration_Order

[Bill Hesse, 2013/01/04 13:49:27]

Done.

  private:
   CObjectArray* NewResponse(Response response, char* arg);
   Dart_Port response_port_;
 
-  DISALLOW_IMPLICIT_CONSTRUCTORS(DirectoryListing);
+  DISALLOW_IMPLICIT_CONSTRUCTORS(DirectoryAsyncListing);
 };
 
 
+class DirectorySyncListing: public DirectoryListing {

[Mads Ager (google), 2013/01/04 10:00:25]

SyncDirectoryListing?

[Bill Hesse, 2013/01/04 13:49:27]

Done.

+ public:
+  explicit DirectorySyncListing(Dart_Handle results)
+      : results_(results) {
+    add_ = DartUtils::NewString("add");

[Mads Ager (google), 2013/01/04 10:00:25]

add_string_?

[Bill Hesse, 2013/01/04 13:49:27]

Done.

+    directory_class_ =
+        DartUtils::GetDartClass(DartUtils::kIOLibURL, "Directory");
+    file_class_ =
+        DartUtils::GetDartClass(DartUtils::kIOLibURL, "File");
+  }
+  virtual bool HandleDirectory(char* dir_name);
+  virtual bool HandleFile(char* file_name);
+  virtual bool HandleError(const char* dir_name);
+
+  virtual ~DirectorySyncListing() {}

[Mads Ager (google), 2013/01/04 10:00:25]

Destructor before methods, please.

[Bill Hesse, 2013/01/04 13:49:27]

Done.

+ private:
+  Dart_Handle results_;
+  Dart_Handle add_;
+  Dart_Handle directory_class_;
+  Dart_Handle file_class_;
+
+  DISALLOW_IMPLICIT_CONSTRUCTORS(DirectorySyncListing);
+};
+
+
 class Directory {
  public:
   enum ExistsResult {
     UNKNOWN,
     EXISTS,
     DOES_NOT_EXIST
   };
 
   // This enum must be kept in sync with the request values in
   // directory_impl.dart.
@@ skipping 22 matching lines @@
   static int service_ports_size_;
   static Dart_Port* service_ports_;
   static int service_ports_index_;
 
   DISALLOW_ALLOCATION();
   DISALLOW_IMPLICIT_CONSTRUCTORS(Directory);
 };
 
 
 #endif  // BIN_DIRECTORY_H_