Bug 376581 - ASSERT: null node when deleting last history item. patch from onemen.one@gmail.com, r=me.

2007-04-23 23:38:04 +00:00 · 2007-04-23 23:38:04 +00:00 · 50d6031753
--- a/browser/components/places/content/tree.xml
+++ b/browser/components/places/content/tree.xml
@ -947,8 +947,18 @@

      <method name="restoreSelection">
        <body><![CDATA[
+          var allowedMaxIndex = this.view.rowCount - 1;
+
          for (var i = 0; i < this._nextSelection.length; ++i) {
            var range = this._nextSelection[i];
+
+            // make sure the index is not over the end of the view
+            // this fixes bug 376581
+            if (range.max > allowedMaxIndex)
+              range.max = allowedMaxIndex;
+            if (range.min > allowedMaxIndex)
+              range.min = allowedMaxIndex;
+
            if (range.min > -1 && range.max > -1)
              this.view.selection.rangedSelect(range.min, range.max, true);
          }