Bookmark Manager doesn't open if all windows are closed

RESOLVED FIXED in seamonkey2.1b1

Status

SeaMonkey
Bookmarks & History
RESOLVED FIXED
7 years ago
7 years ago

People

(Reporter: stefanh, Assigned: Robert Kaiser)

Tracking

Trunk
seamonkey2.1b1
x86
Mac OS X
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Reporter)

Description

7 years ago
STR:

1) Use a Mac, launch SeaMonkey
2) Close all windows
3) From the Bookmarks menu, choose "Manage Bookmarks"

Expected results: Bookmarks Manager window opens.
Actual results: No window appear and the following error messages can be seen in the error console:

--------------------------------------
Error: PlacesMenu is not defined
Source File: chrome://navigator/content/hiddenWindow.xul
Line: 1
--------------------------------------
Error: PlacesCommandHook is not defined
Source File: chrome://navigator/content/hiddenWindow.xul
Line: 1
--------------------------------------
(Reporter)

Comment 1

7 years ago
It appears that hiddenWindow.xul still references the old js files, so no menu commands works. I also think that we need to reference the jsm modules in hiddenwindow.
(Assignee)

Comment 2

7 years ago
Oops, I'm sorry. Somehow I have a tendency to forget that hidden window. :(
(Assignee)

Comment 3

7 years ago
Created attachment 468178 [details] [diff] [review]
v1: I hope that does it

Stefan, please test this, I hope it works.
Assignee: nobody → kairo
Status: NEW → ASSIGNED
Attachment #468178 - Flags: review?
(Assignee)

Updated

7 years ago
Attachment #468178 - Flags: review? → review?(stefanh)
(Reporter)

Comment 4

7 years ago
Comment on attachment 468178 [details] [diff] [review]
v1: I hope that does it

OK, so the BM opens now, but there are a few issues:

1) The first one is a js error while opening the window:
Error: PlacesUtils is not defined
Source File: chrome://communicator/content/places/browserPlacesViews.js
Line: 76

2) The second one is that the bookmarks menu doesn't show any bookmarks while opening it from the menubar (I only see the default menuitems and the PT menu without any bookmarks in and I got a lot of BM)

I suspect that 2) is related to the jsm modules I mention in comment #1, but I'm not sure.
Attachment #468178 - Flags: review?(stefanh) → review-
(Assignee)

Comment 5

7 years ago
Created attachment 469646 [details] [diff] [review]
v1.1: add the overlay

OK, I guess we need to add the places overlay as well.
Attachment #468178 - Attachment is obsolete: true
Attachment #469646 - Flags: review?(stefanh)
(Reporter)

Comment 6

7 years ago
It works now, but I get an error in the console saying:
------------------------------------------------------------------
Error: An error occurred updating the cmd_selectAll command: [Exception... "'[JavaScript Error: "this._view.result is null" {file: "chrome://communicator/content/places/controller.js" line: 181}]' when calling method: [nsIController::isCommandEnabled]"  nsresult: "0x80570021 (NS_ERROR_XPC_JAVASCRIPT_ERROR_WITH_DETAILS)"  location: "JS frame :: chrome://global/content/globalOverlay.js :: goUpdateCommand :: line 55"  data: yes]
Source File: chrome://global/content/globalOverlay.js
Line: 61
------------------------------------------------------------------
(Reporter)

Comment 7

7 years ago
Not sure that's related, though...
(Reporter)

Comment 8

7 years ago
Actually, the error in comment #6 happens on non-mac as well. I have however found another problem - it seems that the microsummaryservice seems to relies on that you have a browser window open

Error: event is not defined
Source File: file:///Users/Stefan/objdir-sm-trunk/mozilla/dist/SeaMonkeyDebug.app/Contents/MacOS/components/nsMicrosummaryService.js
Line: 1982
(Reporter)

Comment 9

7 years ago
this happens btw while selecting a bookmark in the bm.
(Assignee)

Comment 10

7 years ago
(In reply to comment #9)
> this happens btw while selecting a bookmark in the bm.

Please file a new bug for that, either bad port or failure from addressing reviews.

Let's keep this bug to making hiddenWindow work - and I'm happy if we can succeed there.
(Reporter)

Comment 11

7 years ago
Comment on attachment 469646 [details] [diff] [review]
v1.1: add the overlay

> Please file a new bug for that, either bad port or failure from addressing
> reviews.
Sure, we can do that in another bug.
Attachment #469646 - Flags: review?(stefanh) → review+
(Reporter)

Comment 12

7 years ago
(In reply to comment #11)
> Comment on attachment 469646 [details] [diff] [review]
> v1.1: add the overlay
> 
> > Please file a new bug for that, either bad port or failure from addressing
> > reviews.
> Sure, we can do that in another bug.
Filed bug 586363.
Blocks: 591807
(Reporter)

Comment 13

7 years ago
(In reply to comment #12)
> Filed bug 586363.

Uh, I mean bug 591807.
(Assignee)

Comment 14

7 years ago
Pushed as http://hg.mozilla.org/comm-central/rev/e2155ffb330a - thanks for bringing it up!
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.1b1
You need to log in before you can comment on or make changes to this bug.