Refactoring to prepare for kernel based jump targets

pkg/compiler/lib/src/ssa/kernel_impact.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.
 
 import 'package:kernel/ast.dart' as ir;
 
 import '../common.dart';
 import '../common/names.dart';
 import '../compiler.dart';
 import '../constants/expressions.dart';
@@ skipping 106 matching lines @@
     if (field.initializer != null) {
       visitNode(field.initializer);
       if (!field.isInstanceMember &&
           !field.isConst &&
           field.initializer is! ir.NullLiteral) {
         impactBuilder.registerFeature(Feature.LAZY_FIELD);
       }
     }
     if (field.isInstanceMember &&
         elementAdapter.isNativeClass(field.enclosingClass)) {
-      // TODO(johnniwinther): Provide the correct value for [isJsInterop].
+      // TODO(redemption): Provide the correct value for [isJsInterop].
       impactBuilder.registerNativeData(elementAdapter
           .getNativeBehaviorForFieldLoad(field, isJsInterop: false));
       impactBuilder.registerNativeData(
           elementAdapter.getNativeBehaviorForFieldStore(field));
     }
     return impactBuilder;
   }
 
   ResolutionImpact buildConstructor(ir.Constructor constructor) {
     handleSignature(constructor.function, checkReturnType: false);
@@ skipping 20 matching lines @@
             "Unexpected async marker: ${asyncMarker}");
     }
   }
 
   ResolutionImpact buildProcedure(ir.Procedure procedure) {
     handleSignature(procedure.function);
     visitNode(procedure.function.body);
     handleAsyncMarker(procedure.function.asyncMarker);
     if (procedure.isExternal &&
         !elementAdapter.isForeignLibrary(procedure.enclosingLibrary)) {
-      // TODO(johnniwinther): Provide the correct value for [isJsInterop].
+      // TODO(redemption): Provide the correct value for [isJsInterop].
       impactBuilder.registerNativeData(elementAdapter
           .getNativeBehaviorForMethod(procedure, isJsInterop: false));
     }
     return impactBuilder;
   }
 
   void visitNode(ir.Node node) => node?.accept(this);
 
   void visitNodes(Iterable<ir.Node> nodes) {
     nodes.forEach(visitNode);
@@ skipping 433 matching lines @@
     ConstructorEntity target = elementAdapter.getConstructor(node.target);
     impactBuilder.registerStaticUse(new StaticUse.superConstructorInvoke(
         target, elementAdapter.getCallStructure(node.arguments)));
   }
 
   // TODO(johnniwinther): Make this throw and visit child nodes explicitly
   // instead to ensure that we don't visit unwanted parts of the ir.
   @override
   void defaultNode(ir.Node node) => node.visitChildren(this);
 }