Fix some minor static type issues in dart:io implementation.

runtime/bin/file_impl.dart

Index: runtime/bin/file_impl.dart
diff --git a/runtime/bin/file_impl.dart b/runtime/bin/file_impl.dart
index 479cea36b0a2fd12c872def397baff59ed31c39c..70af2ac491dfa79bcf107b8026aa19f1d133e496 100644
--- a/runtime/bin/file_impl.dart
+++ b/runtime/bin/file_impl.dart
@@ -489,7 +489,7 @@ class _File implements File {
     });
   }
 
-  RandomAccessFile openSync([FileMode mode = FileMode.READ]) {
+  RandomAccessFile openSync(FileMode mode) {

[Mads Ager (google), 2012/03/14 13:34:08]

Why?

I can't believe that this does not break other code. If you really want to make
this change you need to find all users and update the API as well.

[Bill Hesse, 2012/03/14 13:46:49]

The File interface already declares openSync this way.
This is just an implementation (in _File) with declaration disagreeing with the
interface.  All users of this go through the File interface, which has the
non-optional argument already. 

On 2012/03/14 13:34:08, Mads Ager wrote:

[Mads Ager (google), 2012/03/14 14:03:03]

OK, could you check that all users of openSync provide a mode. If they don't
this change will still break users.

     if (_asyncUsed) {
       throw new FileIOException(
           "Mixed use of synchronous and asynchronous API");
@@ -572,7 +572,7 @@ class _File implements File {
       throw new FileIOException(
           "Mixed use of synchronous and asynchronous API");
     }
-    var opened = openSync();
+    var opened = openSync(FileMode.READ);
     var length = opened.lengthSync();
     var result = new ByteArray(length);
     var read = opened.readListSync(result, 0, length);
@@ -655,7 +655,7 @@ class _File implements File {
 
   String get name() => _name;
 
-  void set onError(void handler(String error)) {
+  void set onError(void handler(Exception error)) {
     _onError = handler;
   }