Closed Bug 586363 Opened 9 years ago Closed 9 years ago

Bookmark Manager doesn't open if all windows are closed

Categories

(SeaMonkey :: Bookmarks & History, defect)

x86
macOS
defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED
seamonkey2.1b1

People

(Reporter: stefanh, Assigned: kairo)

References

Details

Attachments

(1 file, 1 obsolete file)

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
--------------------------------------
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.
Oops, I'm sorry. Somehow I have a tendency to forget that hidden window. :(
Attached patch v1: I hope that does it (obsolete) — Splinter Review
Stefan, please test this, I hope it works.
Assignee: nobody → kairo
Status: NEW → ASSIGNED
Attachment #468178 - Flags: review?
Attachment #468178 - Flags: review? → review?(stefanh)
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-
OK, I guess we need to add the places overlay as well.
Attachment #468178 - Attachment is obsolete: true
Attachment #469646 - Flags: review?(stefanh)
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
------------------------------------------------------------------
Not sure that's related, though...
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
this happens btw while selecting a bookmark in the bm.
(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.
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+
(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
(In reply to comment #12)
> Filed bug 586363.

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