Remove _nodeToEntity and _nodeToThisLocal from CapturedScopeBuilder

pkg/compiler/lib/src/elements/entities.dart

Index: pkg/compiler/lib/src/elements/entities.dart
diff --git a/pkg/compiler/lib/src/elements/entities.dart b/pkg/compiler/lib/src/elements/entities.dart
index 2e89c2b47380d2b2326a943ef7138d125cffedaf..30219525f0c70be85cf119326992368d3f3c0017 100644
--- a/pkg/compiler/lib/src/elements/entities.dart
+++ b/pkg/compiler/lib/src/elements/entities.dart
@@ -234,6 +234,8 @@ abstract class ConstructorBodyEntity extends FunctionEntity {
 // TODO(johnniwinther): Should [Local] have `isAssignable` or `type`?
 abstract class Local extends Entity {
   /// The context in which this local is defined.
+  // TODO(johnniwinther): Remove this. It is only used in the AST based closure
+  // converter.
   Entity get executableContext;
 
   /// The outermost member that contains this element.