Error: aOldNode is undefined

RESOLVED WORKSFORME

Status

()

Firefox
Bookmarks & History
RESOLVED WORKSFORME
7 years ago
6 years ago

People

(Reporter: ithinc, Unassigned)

Tracking

Trunk
x86
Windows XP
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

7 years ago
User-Agent:       Mozilla/5.0 (Windows NT 5.1; rv:2.0b8pre) Gecko/20101106 Firefox/4.0b8pre
Build Identifier: Mozilla/5.0 (Windows NT 5.1; rv:2.0b8pre) Gecko/20101106 Firefox/4.0b8pre

Error: aOldNode is undefined
Source File: chrome://browser/content/places/treeView.js
Line: 424

Reproducible: Always

Steps to Reproduce:
1. Drag the current tab to bookmarks toolbar
2. Repeat step 1 to have two identical bookmarks in Recently Bookmarked folder
3. Open bookmarks sidebar
4. Open Recently Bookmarked folder
5. Left-click the last bookmarked bookmark
Actual Results:  
The above error is reported in Error Console.
(Reporter)

Updated

7 years ago
Version: unspecified → Trunk
(Reporter)

Updated

7 years ago
Blocks: 560198

Updated

7 years ago
Status: UNCONFIRMED → NEW
Ever confirmed: true
I could think the problem is a missing check at line 964
>+if (nodesToReselect.length > 0) {
>  this._restoreSelection(nodesToReselect, aContainer);
>+}

but I cannot reproduce with the steps in comment 0, do you have better steps working in current nightly?
even if actually that check would be useless since _restoreSelection early returns, so looks like instead it's a undefined node in _getSelectedNodesInRange node: this._rows[i], so possibly this should use _getNodeForRow(i)...
But I need better steps to reproduce.
(Reporter)

Comment 3

7 years ago
I have no other reproducing steps. As you said, I cannot reproduce it any more. Maybe it is a RESOLVED WORKSFORME?
If you still want to investigate it, I suggest using 20101106 nightly.
(Reporter)

Comment 4

6 years ago
Unable to reproduce it in Firefox 4.0 final.
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.