Add index consistency check to source map validation tests.

tests/compiler/dart2js/source_map_validator_helper.dart

 // Copyright (c) 2014, 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 'dart:io';
 import 'dart:convert';
 import 'dart:async';
 
 import 'package:path/path.dart' as path;
 import 'package:expect/expect.dart';
 import 'package:source_maps/source_maps.dart';
 
 validateSourceMap(Uri targetUri) {
   Uri mapUri = getMapUri(targetUri);
   SingleMapping sourceMap = getSourceMap(mapUri);
   checkFileReferences(targetUri, mapUri, sourceMap);
+  checkIndexReferences(targetUri, mapUri, sourceMap);
   checkRedundancy(sourceMap);
 }
 
+checkIndexReferences(Uri targetUri, Uri mapUri, SingleMapping sourceMap) {
+  List<String> target =
+      new File.fromUri(targetUri).readAsStringSync().split('\n');
+  int urlsLength = sourceMap.urls.length;
+  List<List<String>> sources = new List(urlsLength);
+  print('Reading sources');
+  for (int i = 0; i < urlsLength; i++) {
+    sources[i] = new File.fromUri(mapUri.resolve(sourceMap.urls[i])).
+        readAsStringSync().split('\n');
+  }
+
+  sourceMap.lines.forEach((TargetLineEntry line) {
+    Expect.isTrue(line.line < target.length);

[Johnni Winther, 2014/04/10 12:55:01]

Check that line.line >= 1 (or is it 0?)

[zarah, 2014/04/10 13:46:19]

Done.

+    for (TargetEntry entry in line.entries) {
+      int urlIndex = entry.sourceUrlId;
+
+      // TODO(zarah): Entry columns sometimes point one or more characters too
+      // far. Incomment this check when this is fixed.
+      //
+      // Expect.isTrue(entry.column < target[line.line].length);

[Johnni Winther, 2014/04/10 12:55:01]

Check entry.column >= 1 (or 0?)

[zarah, 2014/04/10 13:46:19]

Done.

+      Expect.isTrue(urlIndex == null || urlIndex < urlsLength);

[Johnni Winther, 2014/04/10 12:55:01]

Check urlIndex >= 0.

[zarah, 2014/04/10 13:46:19]

Done.

+      Expect.isTrue(entry.sourceLine == null ||
+          entry.sourceLine < sources[urlIndex].length);

[Johnni Winther, 2014/04/10 12:55:01]

Check entry.sourceLine >= 1 (or 1?).

[zarah, 2014/04/10 13:46:19]

Done.

+      Expect.isTrue(entry.sourceColumn == null ||
+          entry.sourceColumn < sources[urlIndex][entry.sourceLine].length);

[Johnni Winther, 2014/04/10 12:55:01]

Ditto.

[zarah, 2014/04/10 13:46:19]

Done.

+      Expect.isTrue(entry.sourceNameId == null ||
+          entry.sourceNameId < sourceMap.names.length);

[Johnni Winther, 2014/04/10 12:55:01]

Ditto.

[zarah, 2014/04/10 13:46:19]

Done.

+    }
+  });
+}
+
 checkFileReferences(Uri targetUri, Uri mapUri, SingleMapping sourceMap) {
   Expect.equals(targetUri, mapUri.resolve(sourceMap.targetUrl));
   print('Checking sources');
   sourceMap.urls.forEach((String url) {
     Expect.isTrue(new File.fromUri(mapUri.resolve(url)).existsSync());
   });
 }
 
 checkRedundancy(SingleMapping sourceMap) {
   sourceMap.lines.forEach((TargetLineEntry line) {
@@ skipping 50 matching lines @@
   });
 }
 
 Future<Directory> createTempDir() {
   return Directory.systemTemp
       .createTemp('sourceMap_test-')
       .then((Directory dir) {
     return dir;
   });
 }