Remove _nodeToEntity and _nodeToThisLocal from CapturedScopeBuilder

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

 // Copyright (c) 2016, 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.
 
 library entities;
 
 import 'package:front_end/src/fasta/parser/async_modifier.dart'
     show AsyncModifier;
 
 import '../common.dart';
@@ skipping 216 matching lines @@
 /// and thus implement [Local] through [LocalElement]. For non-element locals,
 /// like `this` and boxes, specialized [Local] classes are created.
 ///
 /// Type variables can introduce locals in factories and constructors
 /// but since one type variable can introduce different locals in different
 /// factories and constructors it is not itself a [Local] but instead
 /// a non-element [Local] is created through a specialized class.
 // 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.
   ///
   /// For top level, static or instance members, the member context is the
   /// element itself. For parameters, local variables and nested closures, the
   /// member context is the top level, static or instance member in which it is
   /// defined.
   MemberEntity get memberContext;
 }
@@ skipping 23 matching lines @@
   /// The total number of parameters (required or optional).
   int get totalParameters => positionalParameters + namedParameters.length;
 
   /// Returns the [CallStructure] corresponding to a call site passing all
   /// parameters both required and optional.
   CallStructure get callStructure {
     return new CallStructure(
         positionalParameters + namedParameters.length, namedParameters);
   }
 }