[turbolizer] Use locations rather than ranges everywhere

tools/turbolizer/schedule-view.js

 // Copyright 2015 the V8 project authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 "use strict";
 
 class ScheduleView extends TextView {
-  constructor(id, broker, nodePositionMap) {
+  constructor(id, broker) {
     super(id, broker, null, false);
     let view = this;
     let BLOCK_STYLE = {
       css: 'tag'
     };
     const BLOCK_HEADER_STYLE = {
       css: 'com',
       block_id: -1,
       location: function(text) {
         let matches = /\d+/.exec(text);
@@ skipping 77 matching lines @@
         [/^B\d+$/, BLOCK_LINK_STYLE, -1],
         [/^B\d+/, BLOCK_LINK_STYLE],
         [/^, /, null]
       ],
       [
         [/^ -> /, ARROW_STYLE],
         [/^B\d+$/, BLOCK_LINK_STYLE, -1]
       ]
     ];
     this.setPatterns(patterns);
-    this.setNodePositionMap(nodePositionMap);
   }
 
   initializeContent(data, rememberedSelection) {
     super.initializeContent(data, rememberedSelection);
     var graph = this;
     var locations = [];
     for (var id of rememberedSelection) {
       locations.push({ node_id : id });
     }
-    this.selectLocations(locations, true, false);
+    this.selectLocations(locations, true, true);
   }
 
   detachSelection() {
     var selection = this.selection.detachSelection();
     var s = new Set();
     for (var i of selection) {
-      s.add(i.location.node_id);
+      if (i.location.node_id != undefined && i.location.node_id > 0) {
+        s.add(i.location.node_id);
+      }
     };
     return s;
   }
 }