223611 - Bookmarks toolbar is blank when opening new window

Status: VERIFIED FIXED

(Firefox :: Bookmarks & History, defect)

Description

[Ali Ebrahim]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.6a) Gecko/20031024 Firebird/0.7+ (aebrahim)
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.6a) Gecko/20031024 Firebird/0.7+ (aebrahim)

Bookmarks toolbar is empty when opening a new window. In the initial open window
it displays correctly, but in all new windows, it is empty.

Reproducible: Always

Steps to Reproduce:
1. Press Ctrl-N to open a new window.
2.
3.

Actual Results:  
The Bookmarks toolbar is empty.

Expected Results:  
The Bookmarks toolbar should have all the bookmarks in the BTF.

Comment 1

[Ali Ebrahim]

Moving to Bookmarks as per mconnor's request.

Comment 2

[Jesse Ruderman]

I also see this bug.
Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.6a) Gecko/20031024
Firebird/0.7+

Comment 3

[Mike Connor [:mconnor]]

Confirmed on Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.6a)
Gecko/20031024 Firebird/0.7+

it breaks in bookmarks.js, in if (window.opener) stuff (which is why it's fine
on the first window, but breaks after than point).

On open:

Error: openSidebarBox is not defined
Source File: chrome://browser/content/browser.js
Line: 209

On close you'll also see:

Error: uncaught exception: [Exception... "Component returned failure code:
0x80004005 (NS_ERROR_FAILURE) [nsIObserverService.removeObserver]"  nsresult:
"0x80004005 (NS_ERROR_FAILURE)"  location: "JS frame ::
chrome://browser/content/browser.js :: Shutdown :: line 446"  data: no]

this is stemming from delayedStartup not firing and setting the observer its
supposed to set.

Comment 4

[Pierre Chanial]

that's due to a typo from ben's last checkin (openSidebarBox instead of
openerSidebarBox)

Comment 5

[Oleg Sidletskiy]

*** Bug 223712 has been marked as a duplicate of this bug. ***

Comment 6

[Simon Paquet [:sipaq]]

verified.

Comment 7

[Bob Firth]

*** Bug 226139 has been marked as a duplicate of this bug. ***

Comment 8

[Mike Connor [:mconnor]]

sorry for bugspam, long-overdue mass reassign of ancient QA contact bugs, filter on "beltznerLovesGoats" to get rid of this mass change