Changeset 82943 in webkit


Ignore:
Timestamp:
Apr 5, 2011 8:55:10 AM (13 years ago)
Author:
podivilov@chromium.org
Message:

2011-04-05 Pavel Podivilov <podivilov@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: go to line dialog highlights wrong line.
https://bugs.webkit.org/show_bug.cgi?id=57845

  • inspector/front-end/GoToLineDialog.js: (WebInspector.GoToLineDialog.prototype._highlightSelectedLine):
Location:
trunk/Source/WebCore
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • trunk/Source/WebCore/ChangeLog

    r82942 r82943  
     12011-04-05  Pavel Podivilov  <podivilov@chromium.org>
     2
     3        Reviewed by Pavel Feldman.
     4
     5        Web Inspector: go to line dialog highlights wrong line.
     6        https://bugs.webkit.org/show_bug.cgi?id=57845
     7
     8        * inspector/front-end/GoToLineDialog.js:
     9        (WebInspector.GoToLineDialog.prototype._highlightSelectedLine):
     10
    1112011-04-05  Luke Macpherson   <macpherson@chromium.org>
    212
  • trunk/Source/WebCore/inspector/front-end/GoToLineDialog.js

    r81023 r82943  
    119119    {
    120120        var value = this._input.value;
    121         var lineNumber = parseInt(value, 10);
    122         if (!isNaN(lineNumber) && lineNumber > 0) {
    123             lineNumber = Math.min(lineNumber, this._view.textModel.linesCount);
     121        var lineNumber = parseInt(value, 10) - 1;
     122        if (!isNaN(lineNumber) && lineNumber >= 0) {
     123            lineNumber = Math.min(lineNumber, this._view.textModel.linesCount - 1);
    124124            this._view.highlightLine(lineNumber);
    125125        }
Note: See TracChangeset for help on using the changeset viewer.