Paint Profiler: fix right boundary of selection window

Source/devtools/front_end/timeline/PaintProfilerView.js

 /*
  * Copyright (C) 2013 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 160 matching lines @@
     },
 
     _onWindowChanged: function()
     {
         this.dispatchEventToListeners(WebInspector.PaintProfilerView.Events.WindowChanged);
 
         // Update pie chart
         var window = this.windowBoundaries();
         var totalTime = 0;
         var timeByCategory = {};
-        for (var i = window.left; i <= window.right; ++i) {
+        for (var i = window.left; i < window.right; ++i) {
             var logEntry = this._log[i];
             var category = WebInspector.PaintProfilerView._categoryForLogItem(logEntry);
             timeByCategory[category.color] = timeByCategory[category.color] || 0;
             for (var j = 0; j < this._profiles.length; ++j) {
                 var time = this._profiles[j][logEntry.commandIndex];
                 totalTime += time;
                 timeByCategory[category.color] += time;
             }
         }
         this._pieChart.setTotal(totalTime / this._profiles.length);
@@ skipping 14 matching lines @@
         return Number.millisToString(value * 1000, true);
     },
 
     /**
      * @return {{left: number, right: number}}
      */
     windowBoundaries: function()
     {
         var screenLeft = this._selectionWindow.windowLeft * this._canvas.width;
         var screenRight = this._selectionWindow.windowRight * this._canvas.width;
-        var barLeft = Math.floor((screenLeft - this._barPaddingWidth) / this._outerBarWidth);
-        var barRight = Math.floor((screenRight - this._barPaddingWidth + this._innerBarWidth)/ this._outerBarWidth);
+        var barLeft = Math.floor(screenLeft / this._outerBarWidth);
+        var barRight = Math.floor((screenRight + this._innerBarWidth - this._barPaddingWidth / 2) / this._outerBarWidth);
         var stepLeft = Number.constrain(barLeft * this._samplesPerBar, 0, this._log.length - 1);
-        var stepRight = Number.constrain(barRight * this._samplesPerBar, 0, this._log.length - 1);
+        var stepRight = Number.constrain(barRight * this._samplesPerBar, 0, this._log.length);
 
         return { left: stepLeft, right: stepRight };
     },
 
     _updateImage: function()
     {
         delete this._updateImageTimer;
         if (!this._profiles || !this._profiles.length)
             return;
 
         var window = this.windowBoundaries();
-        this._snapshot.requestImage(this._log[window.left].commandIndex, this._log[window.right].commandIndex, 1, this._showImageCallback);
+        this._snapshot.requestImage(this._log[window.left].commandIndex, this._log[window.right - 1].commandIndex, 1, this._showImageCallback);
     },
 
     _reset: function()
     {
         this._snapshot = null;
         this._profiles = null;
         this._selectionWindow.reset();
     },
 
     __proto__: WebInspector.HBox.prototype
@@ skipping 42 matching lines @@
     /**
      * @param {number=} stepLeft
      * @param {number=} stepRight
      */
     updateWindow: function(stepLeft, stepRight)
     {
         this.sidebarTree.removeChildren();
         if (!this._log)
             return;
         stepLeft = stepLeft || 0;
-        stepRight = stepRight || this._log.length - 1;
-        for (var i = stepLeft; i <= stepRight; ++i)
+        stepRight = stepRight || this._log.length;
+        for (var i = stepLeft; i < stepRight; ++i)
             this._appendLogItem(this.sidebarTree, this._log[i]);
     },
 
     _reset: function()
     {
         this._log = [];
     },
 
     /**
      * @param {?Event} event
@@ skipping 275 matching lines @@
     var method = logItem.method.toTitleCase();
 
     var logItemCategories = WebInspector.PaintProfilerView._initLogItemCategories();
     var result = logItemCategories[method];
     if (!result) {
         result = WebInspector.PaintProfilerView.categories()["misc"];
         logItemCategories[method] = result;
     }
     return result;
 }