[turbolizer] Improve code comments in disassembly

tools/turbolizer/text-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 TextView extends View {
   constructor(id, broker, patterns, allowSpanSelection) {
     super(id, broker);
     let view = this;
@@ skipping 10 matching lines @@
         broker.clear(selectionHandler);
       },
       select: function(items, selected) {
         for (let i of items) {
           if (selected) {
             i.classList.add("selected");
           } else {
             i.classList.remove("selected");
           }
         }
+        broker.clear(selectionHandler);
         broker.select(selectionHandler, view.getRanges(items), selected);
       },
       selectionDifference: function(span1, inclusive1, span2, inclusive2) {
         return null;
       },
       brokeredSelect: function(ranges, selected) {
         let locations = view.rangesToLocations(ranges);
         view.selectLocations(locations, selected, true);
       },
       brokeredClear: function() {
@@ skipping 20 matching lines @@
     return range;
   }
 
   rangesToLocations(ranges) {
     let view = this;
     let nodes = new Set();
     let result = [];
     for (let range of ranges) {
       let start = range[0];
       let end = range[1];
-      let location = { pos_start: start, pos_end: end };
+      let block_id = range[3];
+      let location = { pos_start: start, pos_end: end, block_id: block_id };
       if (range[2] !== null && range[2] != -1) {
         location.node_id = range[2];
         if (range[0] == -1 && range[1] == -1) {
           location.pos_start = view.nodePositionMap[location.node_id];
           location.pos_end = location.pos_start + 1;
         }
       } else {
         if (range[0] != undefined) {
           location.pos_start = range[0];
           location.pos_end = range[1];
@@ skipping 96 matching lines @@
 
   getRanges(items) {
     let result = [];
     let lastObject = null;
     for (let i of items) {
       if (i.location) {
         let location = i.location;
         let start = -1;
         let end = -1;
         let node_id = -1;
+        let block_id = -1;
         if (location.node_id !== undefined) {
           node_id = location.node_id;
         }
+        if (location.block_id !== undefined) {
+          block_id = location.block_id;
+        }
         if (location.pos_start !== undefined) {
           start = location.pos_start;
           end = location.pos_end;
         } else {
           if (this.nodePositionMap && this.nodePositionMap[node_id]) {
             start = this.nodePositionMap[node_id];
             end = start + 1;
           }
         }
         if (lastObject == null ||
             (lastObject[2] != node_id ||
              lastObject[0] != start ||
-             lastObject[1] != end)) {
-          lastObject = [start, end, node_id];
+             lastObject[1] != end ||
+             lastObject[3] != block_id)) {
+          lastObject = [start, end, node_id, block_id];
           result.push(lastObject);
         }
       }
     }
     return result;
   }
 
   createFragment(text, style) {
     let view = this;
     let span = document.createElement("SPAN");
@@ skipping 95 matching lines @@
       view.selection.select(s, selected);
     }
   }
 
   mouseDownLine(li, e) {
     let view = this;
     e.stopPropagation();
     if (!e.shiftKey) {
       view.selection.clear();
     }
+    console.log(li.location);

[danno, 2016/07/08 11:28:07]

Was adding this line intentional?

     if (li.location != undefined) {
       view.selectLocations([li.location], true, false);
     }
   }
 
   mouseDownSpan(span, e) {
     let view = this;
     if (view.allowSpanSelection) {
       e.stopPropagation();
       if (!e.shiftKey) {
@@ skipping 55 matching lines @@
   lineLocation(li) {
     let view = this;
     for (let i = 0; i < li.children.length; ++i) {
       let fragment = li.children[i];
       if (fragment.location != undefined && !view.allowSpanSelection) {
         return fragment.location;
       }
     }
   }
 }