Clean up code comments based on the changes in response to sra and johnniwinther's CL comments. (oo…

pkg/compiler/lib/src/js_model/closure_visitors.dart

 // Copyright (c) 2017, 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 'package:kernel/ast.dart' as ir;
 
 import '../closure.dart';
 import '../elements/entities.dart';
 import '../kernel/element_map.dart';
 import 'closure.dart';
 
 /// This builder walks the code to determine what variables are captured/free at
 /// various points to build ClosureScope that can respond to queries
 /// about how a particular variable is being used at any point in the code.
 class ClosureScopeBuilder extends ir.Visitor {
-  /// A map of each visited call node with the associated information about what
-  /// variables are captured/used. Each ir.Node key corresponds to a scope that
-  /// was encountered while visiting a closure (initially called through
-  /// [translateLazyIntializer] or [translateConstructorOrProcedure]).
-  // TODODODOOOOOO
+  /// A map of each for loop statement and the corresponding variable
+  /// information about what variables are captured/used in the this for loop
+  /// declaration.
   final Map<ir.Statement, LoopClosureScope> _loopClosureScopeMap;
 
   /// Map entities to their corresponding scope information (such as what
-  /// variables are captured/used). The distinction between this map and
-  /// [_closureInfoMap] is that [_closureInfoMap] stores this data for closures
-  /// specifically, whereas [_scopeInfoMap] stores this information for entities
-  /// that are *not* closures (this information is used by the locals handler).
-  /// The union of these two maps represents all the scopes encountered.
+  /// variables are captured/used).
   final Map<Entity, ScopeInfo> _scopeInfoMap;
 
   /// A map of the nodes that we have flagged as necessary to generate closure
   /// classes for in a later stage. We map that node to information ascertained
   /// about variable usage in the surrounding scope.
   final Map<ir.TreeNode /* ir.Field | ir.FunctionNode */, ScopeInfo>
       _closuresToGenerate;
 
   /// The local variables that have been declared in the current scope.
   List<ir.VariableDeclaration> _scopeVariables;
@@ skipping 273 matching lines @@
               nodeToConvert.isInstanceMember)) {
         return new ThisLocal(_kernelToElementMap.getConstructor(nodeToConvert));
       } else if (nodeToConvert is ir.Procedure &&
           nodeToConvert.isInstanceMember) {
         return new ThisLocal(_kernelToElementMap.getMethod(nodeToConvert));
       }
     }
     return null;
   }
 }