DevTools: fix keyboard navigation in "GoTo File" dialog

third_party/WebKit/Source/devtools/front_end/ui/ViewportControl.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 567 matching lines @@
     _onScroll: function(event)
     {
         this.refresh();
     },
 
     /**
      * @return {number}
      */
     firstVisibleIndex: function()
     {
-        var firstVisibleIndex;
-        if (this._stickToBottom)
-            firstVisibleIndex = Math.max(this._itemCount - Math.ceil(this._visibleHeight() / this._provider.minimumRowHeight()), 0);
-        else
-            firstVisibleIndex = Math.max(Array.prototype.lowerBound.call(this._cumulativeHeights, this.element.scrollTop + 1), 0);
+        var firstVisibleIndex = Math.max(Array.prototype.lowerBound.call(this._cumulativeHeights, this.element.scrollTop + 1), 0);
         return Math.max(firstVisibleIndex, this._firstActiveIndex);
     },
 
     /**
      * @return {number}
      */
     lastVisibleIndex: function()
     {
         var lastVisibleIndex;
         if (this._stickToBottom)
@@ skipping 14 matching lines @@
             return null;
         return this._renderedItems[index - this._firstActiveIndex].element();
     },
 
     /**
      * @param {number} index
      * @param {boolean=} makeLast
      */
     scrollItemIntoView: function(index, makeLast)
     {
-        if (index > this._firstActiveIndex && index < this._lastActiveIndex)
+        var firstVisibleIndex = this.firstVisibleIndex();
+        var lastVisibleIndex = this.lastVisibleIndex();
+        if (index > firstVisibleIndex && index < lastVisibleIndex)
             return;
         if (makeLast)
             this.forceScrollItemToBeLast(index);
-        else if (index <= this._firstActiveIndex)
+        else if (index <= firstVisibleIndex)
             this.forceScrollItemToBeFirst(index);
-        else if (index >= this._lastActiveIndex)
+        else if (index >= lastVisibleIndex)
             this.forceScrollItemToBeLast(index);
     },
 
     /**
      * @param {number} index
      */
     forceScrollItemToBeFirst: function(index)
     {
         this.setStickToBottom(false);
         this._rebuildCumulativeHeightsIfNeeded();
@@ skipping 18 matching lines @@
 
     /**
      * @return {number}
      */
     _visibleHeight: function()
     {
         // Use offsetHeight instead of clientHeight to avoid being affected by horizontal scroll.
         return this.element.offsetHeight;
     }
 }