Add Directory.createTemp() to asynchronously create a temporary directory.

runtime/bin/directory_impl.dart

 // Copyright (c) 2011, 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.
 
 
 class _DirectoryListingIsolate extends Isolate {
 
   _DirectoryListingIsolate() : super.heavy();
 
   void main() {
@@ skipping 11 matching lines @@
 
   bool _list(String dir,
              bool recursive,
              SendPort dirPort,
              SendPort filePort,
              SendPort donePort,
              SendPort errorPort) native "Directory_List";
 }
 
 
+class _DirectoryCreateTempIsolate extends Isolate {
+
+  _DirectoryCreateTempIsolate() : super.heavy();
+
+  void main() {
+    port.receive((path, replyTo) {
+      if (path == '') path = '/tmp/darttmpdirXXXXXX';

[Mads Ager (google), 2011/11/18 09:50:45]

You should add a comment about this default behavior in the interface.

[Bill Hesse, 2011/11/21 15:48:41]

Moved to .cc file.

+
+      // Call function to get file name
+      String result = _createTemp(path);
+      var message = {'success': (result != ''), 'directoryName': result};

[Mads Ager (google), 2011/11/18 09:50:45]

Do this in the caller? Just do replyTo.send(_createTemp(path))?

[Bill Hesse, 2011/11/21 15:48:41]

Done.

+      replyTo.send(message);
+      port.close();
+    });
+  }
+
+  String _createTemp(String template) native "Directory_CreateTemp";
+}
+
+
 class _Directory implements Directory {
 
   _Directory(String this._path);
 
   bool existsSync() {
     int exists = _exists(_path);
     if (exists < 0) {
       throw new DirectoryException("Diretory exists test failed: $_path");
     }
     return (exists == 1);
   }
 
   void createSync() {
     if (!_create(_path)) {
       throw new DirectoryException("Directory creation failed: $_path");
     }
   }
 
+  void createTemp(void createdHandler(Directory tempDir)) {
+    new _DirectoryCreateTempIsolate().spawn().then((port) {
+      port.call(_path).receive((result, ignored) {
+        if (result['success']) {
+          // Create new Directory with returned file name.
+          Directory tempDir = new Directory(result['directoryName']);
+          tempDir.dirHandler = _dirHandler;

[Mads Ager (google), 2011/11/18 09:50:45]

This is subtle, why do you transfer the handlers? I think this is another
argument that what we really want is to modify the current Directory object.

[Bill Hesse, 2011/11/21 15:48:41]

All this is removed, and we do not create a new Directory object.

On 2011/11/18 09:50:45, Mads Ager wrote:

+          tempDir.fileHandler = _fileHandler;
+          tempDir.doneHandler = _doneHandler;
+          tempDir.errorHandler = _errorHandler;
+          createdHandler(tempDir);
+        } else {
+          // Report error
+        }
+      });
+    });
+  }
+
   void deleteSync() {
     if (!_delete(_path)) {
       throw new DirectoryException("Directory deletion failed: $_path");
     }
   }
 
   void list([bool recursive = false]) {
     new _DirectoryListingIsolate().spawn().then((port) {
       // Build a map of parameters to the directory listing isolate.
       Map listingParameters = new Map();
@@ skipping 85 matching lines @@
   bool _create(String path) native "Directory_Create";
   bool _delete(String path) native "Directory_Delete";
 
   var _dirHandler;
   var _fileHandler;
   var _doneHandler;
   var _errorHandler;
 
   String _path;
 }