more negative test fixes

frog/evaluator.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.
 
 
 #import('file_system.dart');
 #import('lang.dart');
 
 interface JsEvaluator {
   var eval(String source);
@@ skipping 16 matching lines @@
     _markAllUsed(world.corelib);
     world.gen.writeTypes(world.coreimpl);
     world.gen.writeTypes(world.corelib);
 
     // Set these here so that we can compile the corelib without its errors
     // killing us
     options.throwOnErrors = true;
     options.throwOnFatal = true;
   }
 
+  // TODO(jimhug): Should be calling world.genMethod - but I'm scared to
+  //   make the change myself because we don't have any test coverage here.
   static void _markMethodUsed(Member m) {
     if (m == null || m.isGenerated || m.definition == null || m.isAbstract) {
       return;
     }
     new MethodGenerator(m, null).run();
   }
 
   // TODO(nweiz): use this logic for the --compile_all flag
   static void _markAllUsed(Library l) {
     if (_marked.contains(l.name)) return;
@@ skipping 81 matching lines @@
       gen.writeType(type);
       code = gen.writer.text;
     } else {
       parsed.visit(methGen);
       code = methGen.writer.text;
     }
 
     return this._jsEvaluator.eval(code);
   }
 }