Address comments from CL 23304012.

dart/sdk/lib/_internal/compiler/implementation/js_backend/namer.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.
 
 part of js_backend;
 
 /**
  * Assigns JavaScript identifiers to Dart variables, class-names and members.
  */
 class Namer implements ClosureNamer {
@@ skipping 788 matching lines @@
   String getNameOfClass(ClassElement cls) => getNameX(cls);
 
   String getNameOfField(VariableElement field) => getNameX(field);
 
   String getNameOfInstanceMember(Element member) => getNameX(member);
 
   String getNameOfGlobalField(VariableElement field) => getNameX(field);
 
   String getNameOfGlobalFunction(FunctionElement element) => getNameX(element);
 
-  /// Returns true if [element] is stored on CURRENT_ISOLATE ('$').
+  /// Returns true if [element] is stored on CURRENT_ISOLATE ('$').  We intend
+  /// to store only mutable static state in [CURRENT_ISOLATE], constants are
+  /// stored in 'C', and functions, accessors, classes, etc. are stored in one
+  /// of the other objects in [reservedGlobalObjectNames].
   bool isPropertyOfCurrentIsolate(Element element) {
+    // TODO(ahe): Make sure this method's documentation is always true and
+    // remove the word "intend".
     return
+        // TODO(ahe): Re-write these tests to be positive (so it only returns
+        // true for static/top-level mutable fields). Right now, a number of
+        // other elements, such as bound closures also live in CURRENT_ISOLATE.
         !element.isAccessor() &&
         !element.isClass() &&
         !element.isConstructor() &&
         !element.isFunction() &&
         !element.isLibrary();
   }
 
-  /// Returns [CURRENT_ISOLATE] or one of [reservedGlobalObjectNames].  We
-  /// intend to store only mutable static state in [CURRENT_ISOLATE], constants
-  /// are stored in 'C', and functions, accessors, classes, etc. are stored in
-  /// one of the other objects in [reservedGlobalObjectNames].
-  // TODO(ahe): Make sure the above is always true and remove the word "intend".
+  /// Returns [CURRENT_ISOLATE] or one of [reservedGlobalObjectNames].
   String globalObjectFor(Element element) {
     if (isPropertyOfCurrentIsolate(element)) return CURRENT_ISOLATE;
     LibraryElement library = element.getLibrary();
     if (library == compiler.interceptorsLibrary) return 'J';
     if (library.isInternalLibrary) return 'H';
     if (library.isPlatformLibrary) {
       if ('${library.canonicalUri}' == 'dart:html') return 'W';
       return 'P';
     }
     return userGlobalObjects[
@@ skipping 535 matching lines @@
         if (!first) {
           sb.write('_');
         }
         sb.write('_');
         visit(link.head);
         first = true;
       }
     }
   }
 }