Refactor SourceFile, SourceFileProvider and SourceLocation.

pkg/compiler/lib/src/io/source_map_builder.dart

 // Copyright (c) 2012, 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 dart2js.source_map_builder;
 
 import '../util/util.dart';
 import '../util/uri_extras.dart' show relativize;
 import 'line_column_provider.dart';
-import 'source_information.dart' show SourceFileLocation;
+import 'source_information.dart' show SourceLocation;
 
 class SourceMapBuilder {
   static const int VLQ_BASE_SHIFT = 5;
   static const int VLQ_BASE_MASK = (1 << 5) - 1;
   static const int VLQ_CONTINUATION_BIT = 1 << 5;
   static const int VLQ_CONTINUATION_MASK = 1 << 5;
   static const String BASE64_DIGITS = 'ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmn'
                                       'opqrstuvwxyz0123456789+/';
 
-  final Uri uri;
-  final Uri fileUri;
+  /// The URI of the source map file.
+  final Uri sourceMapUri;
+  /// The URI of the target language file.
+  final Uri targetFileUri;
 
   LineColumnProvider lineColumnProvider;
   List<SourceMapEntry> entries;
 
-  Map<String, int> sourceUrlMap;
-  List<String> sourceUrlList;
+  Map<Uri, int> sourceUriMap;
+  List<Uri> sourceUriList;
   Map<String, int> sourceNameMap;
   List<String> sourceNameList;
 
   int previousTargetLine;
   int previousTargetColumn;
-  int previousSourceUrlIndex;
+  int previousSourceUriIndex;
   int previousSourceLine;
   int previousSourceColumn;
   int previousSourceNameIndex;
   bool firstEntryInLine;
 
-  SourceMapBuilder(this.uri, this.fileUri, this.lineColumnProvider) {
+  SourceMapBuilder(this.sourceMapUri,
+                   this.targetFileUri,
+                   this.lineColumnProvider) {
     entries = new List<SourceMapEntry>();
 
-    sourceUrlMap = new Map<String, int>();
-    sourceUrlList = new List<String>();
+    sourceUriMap = new Map<Uri, int>();
+    sourceUriList = new List<Uri>();
     sourceNameMap = new Map<String, int>();
     sourceNameList = new List<String>();
 
     previousTargetLine = 0;
     previousTargetColumn = 0;
-    previousSourceUrlIndex = 0;
+    previousSourceUriIndex = 0;
     previousSourceLine = 0;
     previousSourceColumn = 0;
     previousSourceNameIndex = 0;
     firstEntryInLine = true;
   }
 
   resetPreviousSourceLocation() {
-    previousSourceUrlIndex = 0;
+    previousSourceUriIndex = 0;
     previousSourceLine = 0;
     previousSourceColumn = 0;
     previousSourceNameIndex = 0;
   }
 
-  updatePreviousSourceLocation(SourceFileLocation sourceLocation) {
-    previousSourceLine = sourceLocation.getLine();
-    previousSourceColumn = sourceLocation.getColumn();
-    String sourceUrl = sourceLocation.getSourceUrl();
-    previousSourceUrlIndex = indexOf(sourceUrlList, sourceUrl, sourceUrlMap);
-    String sourceName = sourceLocation.getSourceName();
+  updatePreviousSourceLocation(SourceLocation sourceLocation) {
+    previousSourceLine = sourceLocation.line;
+    previousSourceColumn = sourceLocation.column;
+    Uri sourceUri = sourceLocation.sourceUri;
+    previousSourceUriIndex = indexOf(sourceUriList, sourceUri, sourceUriMap);
+    String sourceName = sourceLocation.sourceName;
     if (sourceName != null) {
       previousSourceNameIndex =
           indexOf(sourceNameList, sourceName, sourceNameMap);
     }
   }
 
-  bool sameAsPreviousLocation(SourceFileLocation sourceLocation) {
+  bool sameAsPreviousLocation(SourceLocation sourceLocation) {
     if (sourceLocation == null) {
       return true;
     }
-    int sourceUrlIndex =
-        indexOf(sourceUrlList, sourceLocation.getSourceUrl(), sourceUrlMap);
+    int sourceUriIndex =
+        indexOf(sourceUriList, sourceLocation.sourceUri, sourceUriMap);
     return
-       sourceUrlIndex == previousSourceUrlIndex &&
-       sourceLocation.getLine() == previousSourceLine &&
-       sourceLocation.getColumn() == previousSourceColumn;
+       sourceUriIndex == previousSourceUriIndex &&
+       sourceLocation.line == previousSourceLine &&
+       sourceLocation.column == previousSourceColumn;
   }
 
-  void addMapping(int targetOffset, SourceFileLocation sourceLocation) {
+  void addMapping(int targetOffset, SourceLocation sourceLocation) {
 
     bool sameLine(int position, otherPosition) {
       return lineColumnProvider.getLine(position) ==
                  lineColumnProvider.getLine(otherPosition);
     }
 
     if (!entries.isEmpty && sameLine(targetOffset, entries.last.targetOffset)) {
       if (sameAsPreviousLocation(sourceLocation)) {
         // The entry points to the same source location as the previous entry in
         // the same line, hence it is not needed for the source map.
@@ skipping 27 matching lines @@
 
   String build() {
     resetPreviousSourceLocation();
     StringBuffer mappingsBuffer = new StringBuffer();
     entries.forEach((SourceMapEntry entry) {
       writeEntry(entry, mappingsBuffer);
     });
     StringBuffer buffer = new StringBuffer();
     buffer.write('{\n');
     buffer.write('  "version": 3,\n');
-    if (uri != null && fileUri != null) {
-      buffer.write('  "file": "${relativize(uri, fileUri, false)}",\n');
+    if (sourceMapUri != null && targetFileUri != null) {
+      buffer.write(
+          '  "file": "${relativize(sourceMapUri, targetFileUri, false)}",\n');
     }
     buffer.write('  "sourceRoot": "",\n');
     buffer.write('  "sources": ');
-    if (uri != null) {
-      sourceUrlList =
-          sourceUrlList.map((url) => relativize(uri, Uri.parse(url), false))
+    List<String> relativeSourceUriList = <String>[];
+    if (sourceMapUri != null) {
+      relativeSourceUriList =

[floitsch, 2015/02/13 13:46:03]

maybe:
relativeSourceUriList = sourceUriList
    .map(...)
    .toList();

[Johnni Winther, 2015/02/13 13:55:49]

Done.

+          sourceUriList.map((u) => relativize(sourceMapUri, u, false))
               .toList();
     }
-    printStringListOn(sourceUrlList, buffer);
+    printStringListOn(relativeSourceUriList, buffer);
     buffer.write(',\n');
     buffer.write('  "names": ');
     printStringListOn(sourceNameList, buffer);
     buffer.write(',\n');
     buffer.write('  "mappings": "');
     buffer.write(mappingsBuffer);
     buffer.write('"\n}\n');
     return buffer.toString();
   }
 
@@ skipping 14 matching lines @@
     if (!firstEntryInLine) {
       output.write(',');
     }
     firstEntryInLine = false;
 
     encodeVLQ(output, targetColumn - previousTargetColumn);
     previousTargetColumn = targetColumn;
 
     if (entry.sourceLocation == null) return;
 
-    String sourceUrl = entry.sourceLocation.getSourceUrl();
-    int sourceLine = entry.sourceLocation.getLine();
-    int sourceColumn = entry.sourceLocation.getColumn();
-    String sourceName = entry.sourceLocation.getSourceName();
+    Uri sourceUri = entry.sourceLocation.sourceUri;
+    int sourceLine = entry.sourceLocation.line;
+    int sourceColumn = entry.sourceLocation.column;
+    String sourceName = entry.sourceLocation.sourceName;
 
-    int sourceUrlIndex = indexOf(sourceUrlList, sourceUrl, sourceUrlMap);
-    encodeVLQ(output, sourceUrlIndex - previousSourceUrlIndex);
+    int sourceUriIndex = indexOf(sourceUriList, sourceUri, sourceUriMap);
+    encodeVLQ(output, sourceUriIndex - previousSourceUriIndex);
     encodeVLQ(output, sourceLine - previousSourceLine);
     encodeVLQ(output, sourceColumn - previousSourceColumn);
 
     if (sourceName != null) {
       int sourceNameIndex = indexOf(sourceNameList, sourceName, sourceNameMap);
       encodeVLQ(output, sourceNameIndex - previousSourceNameIndex);
     }
 
     // Update previous source location to ensure the next indices are relative
     // to those if [entry.sourceLocation].
     updatePreviousSourceLocation(entry.sourceLocation);
   }
 
-  int indexOf(List<String> list, String value, Map<String, int> map) {
+  int indexOf(List list, value, Map<dynamic, int> map) {
     return map.putIfAbsent(value, () {
       int index = list.length;
       list.add(value);
       return index;
     });
   }
 
   static void encodeVLQ(StringBuffer output, int value) {
     int signBit = 0;
     if (value < 0) {
       signBit = 1;
       value = -value;
     }
     value = (value << 1) | signBit;
     do {
       int digit = value & VLQ_BASE_MASK;
       value >>= VLQ_BASE_SHIFT;
       if (value > 0) {
         digit |= VLQ_CONTINUATION_BIT;
       }
       output.write(BASE64_DIGITS[digit]);
     } while (value > 0);
   }
 }
 
 class SourceMapEntry {
-  SourceFileLocation sourceLocation;
+  SourceLocation sourceLocation;
   int targetOffset;
 
   SourceMapEntry(this.sourceLocation, this.targetOffset);
 }