Use native fields for process pid and fix bug that allowed killing

tests/standalone/io/process_kill_unstarted_test.dart

Index: tests/standalone/io/process_kill_unstarted_test.dart
diff --git a/tests/standalone/io/process_kill_unstarted_test.dart b/tests/standalone/io/process_kill_unstarted_test.dart
new file mode 100644
index 0000000000000000000000000000000000000000..01261855b4c73875ff705d964fffad5a34860924
--- /dev/null
+++ b/tests/standalone/io/process_kill_unstarted_test.dart
@@ -0,0 +1,20 @@
+// 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.
+//
+// Process test program to test that an unstarted process cannot be killed.
+
+#library("ProcessKillUnstartedTest");
+#import("dart:io");
+
+main() {
+  var p = Process.start('________', []);
+  try {

[Søren Gjesse, 2012/09/21 10:44:16]

Why not use Expect.throws here?

[Mads Ager (google), 2012/09/21 11:07:39]

Because I forgot we had that. ;-) Done! Thanks!

+    p.kill();
+    Expect.fail("Exception expected");
+  } on ProcessException catch (e) {
+    // Do nothing.
+  }
+  p.onError = (e) => Expect.isTrue(e is ProcessException);
+  p.onStart = () => Expect.fail("Process not expected to start");
+}