[fasta] Add and correct even more offsets

pkg/front_end/lib/src/fasta/kernel/kernel_procedure_builder.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 fasta.kernel_procedure_builder;
 
 import 'package:kernel/ast.dart'
     show
         Arguments,
         AsyncMarker,
@@ skipping 157 matching lines @@
     } else {
       annotation = new StaticInvocation(constructor.target, arguments)
         ..isConst = true;
     }
     target.addAnnotation(annotation);
   }
 }
 
 class KernelProcedureBuilder extends KernelFunctionBuilder {
   final Procedure procedure;
+  final int charOpenParenOffset;
 
   AsyncMarker actualAsyncModifier;
 
   final ConstructorReferenceBuilder redirectionTarget;
 
   KernelProcedureBuilder(
       List<MetadataBuilder> metadata,
       int modifiers,
       KernelTypeBuilder returnType,
       String name,
       List<TypeVariableBuilder> typeVariables,
       List<FormalParameterBuilder> formals,
       this.actualAsyncModifier,
       ProcedureKind kind,
       KernelLibraryBuilder compilationUnit,
       int charOffset,
+      this.charOpenParenOffset,
       int charEndOffset,
       [String nativeMethodName,
       this.redirectionTarget])
       : procedure = new Procedure(null, kind, null,
             fileUri: compilationUnit?.relativeFileUri)
           ..fileOffset = charOffset
           ..fileEndOffset = charEndOffset,
         super(metadata, modifiers, returnType, name, typeVariables, formals,
             compilationUnit, charOffset, nativeMethodName);
 
@@ skipping 17 matching lines @@
       // No parent, it's an enum.
       function.asyncMarker = actualAsyncModifier;
     }
   }
 
   Procedure build(LibraryBuilder library) {
     // TODO(ahe): I think we may call this twice on parts. Investigate.
     if (procedure.name == null) {
       procedure.function = buildFunction(library);
       procedure.function.parent = procedure;
+      procedure.function.fileOffset = charOpenParenOffset;
+      procedure.function.fileEndOffset = procedure.fileEndOffset;
       procedure.isAbstract = isAbstract;
       procedure.isStatic = isStatic;
       procedure.isExternal = isExternal;
       procedure.isConst = isConst;
       procedure.name = new Name(name, library.target);
     }
     return procedure;
   }
 
   Procedure get target => procedure;
 }
 
 // TODO(ahe): Move this to own file?
 class KernelConstructorBuilder extends KernelFunctionBuilder {
   final Constructor constructor;
+  final int charOpenParenOffset;
 
   bool hasMovedSuperInitializer = false;
 
   SuperInitializer superInitializer;
 
   RedirectingInitializer redirectingInitializer;
 
   KernelConstructorBuilder(
       List<MetadataBuilder> metadata,
       int modifiers,
       KernelTypeBuilder returnType,
       String name,
       List<TypeVariableBuilder> typeVariables,
       List<FormalParameterBuilder> formals,
       KernelLibraryBuilder compilationUnit,
       int charOffset,
+      this.charOpenParenOffset,
       int charEndOffset,
       [String nativeMethodName])
       : constructor = new Constructor(null)
           ..fileOffset = charOffset
           ..fileEndOffset = charEndOffset,
         super(metadata, modifiers, returnType, name, typeVariables, formals,
             compilationUnit, charOffset, nativeMethodName);
 
   bool get isInstanceMember => false;
 
   bool get isConstructor => true;
 
   AsyncMarker get asyncModifier => AsyncMarker.Sync;
 
   ProcedureKind get kind => null;
 
   Constructor build(LibraryBuilder library) {
     if (constructor.name == null) {
       constructor.function = buildFunction(library);
       constructor.function.parent = constructor;
+      constructor.function.fileOffset = charOpenParenOffset;
+      constructor.function.fileEndOffset = constructor.fileEndOffset;
       constructor.isConst = isConst;
       constructor.isExternal = isExternal;
       constructor.name = new Name(name, library.target);
     }
     return constructor;
   }
 
   FunctionNode buildFunction(LibraryBuilder library) {
     // TODO(ahe): Should complain if another type is explicitly set.
     return super.buildFunction(library)..returnType = const VoidType();
@@ skipping 56 matching lines @@
         }
       }
       initializers.add(initializer..parent = constructor);
       initializers.add(superInitializer);
       return;
     }
     initializers.add(initializer);
     initializer.parent = constructor;
   }
 }