fixes #209, remove tree mutation from CodeChecker

lib/src/testing.dart

Index: lib/src/testing.dart
diff --git a/lib/src/testing.dart b/lib/src/testing.dart
index a5ae8b04ae6012215c036a3fcc3741eccf6d664b..590298ee75645270505619afe6b31ddf63a7b638 100644
--- a/lib/src/testing.dart
+++ b/lib/src/testing.dart
@@ -241,8 +241,7 @@ class _ErrorMarkerVisitor extends UnifyingAstVisitor {
       while (comment.next != null) {
         comment = comment.next;
       }
-      if (comment.end == token.offset &&
-          _realParent(node).beginToken != token) {
+      if (comment.end == token.offset && node.parent.beginToken != token) {
         var commentText = '$comment';
         var start = commentText.lastIndexOf('/*');
         var end = commentText.lastIndexOf('*/');
@@ -256,13 +255,6 @@ class _ErrorMarkerVisitor extends UnifyingAstVisitor {
     }
     return super.visitNode(node);
   }
-
-  /// Get the node's parent, ignoring fake conversion nodes.
-  AstNode _realParent(AstNode node) {
-    var p = node.parent;
-    while (p is Conversion) p = p.parent;

[Jennifer Messerly, 2015/06/08 23:51:37]

after renamed Conversion -> CoercionInfo, I noticed this code is now dead

-    return p;
-  }
 }
 
 /// Describes an expected message that should be produced by the checker.