[fasta] Add and correct even more offsets

pkg/front_end/lib/src/fasta/kernel/kernel_library_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_library_builder;
 
 import 'package:kernel/ast.dart';
 
 import 'package:kernel/clone.dart' show CloneVisitor;
 
@@ skipping 151 matching lines @@
   void addProcedure(
       List<MetadataBuilder> metadata,
       int modifiers,
       KernelTypeBuilder returnType,
       String name,
       List<TypeVariableBuilder> typeVariables,
       List<FormalParameterBuilder> formals,
       AsyncMarker asyncModifier,
       ProcedureKind kind,
       int charOffset,
+      int charOpenParenOffset,
       int charEndOffset,
       String nativeMethodName,
       {bool isTopLevel}) {
     // Nested declaration began in `OutlineBuilder.beginMethod` or
     // `OutlineBuilder.beginTopLevelMethod`.
     endNestedDeclaration().resolveTypes(typeVariables, this);
     ProcedureBuilder procedure;
     if (!isTopLevel && isConstructorName(name, currentDeclaration.name)) {
       name = computeConstructorName(name);
       procedure = new KernelConstructorBuilder(
@@ skipping 12 matching lines @@
           metadata,
           modifiers,
           returnType,
           name,
           typeVariables,
           formals,
           asyncModifier,
           kind,
           this,
           charOffset,
+          charOpenParenOffset,
           charEndOffset,
           nativeMethodName);
     }
     addBuilder(name, procedure, charOffset);
     if (nativeMethodName != null) {
       addNativeMethod(procedure);
     }
   }
 
   void addFactoryMethod(
       List<MetadataBuilder> metadata,
       int modifiers,
       ConstructorReferenceBuilder constructorName,
       List<FormalParameterBuilder> formals,
       AsyncMarker asyncModifier,
       ConstructorReferenceBuilder redirectionTarget,
       int charOffset,
+      int charOpenParenOffset,
       int charEndOffset,
       String nativeMethodName) {
     // Nested declaration began in `OutlineBuilder.beginFactoryMethod`.
     DeclarationBuilder<KernelTypeBuilder> factoryDeclaration =
         endNestedDeclaration();
     String name = constructorName.name;
     if (isConstructorName(name, currentDeclaration.name)) {
       name = computeConstructorName(name);
     }
     assert(constructorName.suffix == null);
     KernelProcedureBuilder procedure = new KernelProcedureBuilder(
         metadata,
         staticMask | modifiers,
         null,
         name,
         <TypeVariableBuilder>[],
         formals,
         asyncModifier,
         ProcedureKind.Factory,
         this,
         charOffset,
+        charOpenParenOffset,
         charEndOffset,
         nativeMethodName,
         redirectionTarget);
     currentDeclaration.addFactoryDeclaration(procedure, factoryDeclaration);
     addBuilder(name, procedure, charOffset);
     if (nativeMethodName != null) {
       addNativeMethod(procedure);
     }
   }
 
@@ skipping 181 matching lines @@
   }
 }
 
 bool isConstructorName(String name, String className) {
   if (name.startsWith(className)) {
     if (name.length == className.length) return true;
     if (name.startsWith(".", className.length)) return true;
   }
   return false;
 }