Stefan [:stefanh] Reporter
	Description • 14 years ago

STR:

1) Get a mac, launch SeaMonkey
2) Close any open windows
3) From the main menu bar, select "Manage Bookmarks..."
4) In the BM window select a bookmark

--> an error in the js console appears, indicating code that relies on an open browser window:

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

	Stefan [:stefanh] Reporter
	Comment 1 • 14 years ago

Oh, right - forgot to say that this assumes that bug 586363 is resolved, since atm the BM won't open at all if all windows are closed.

	Robert Kaiser
	Comment 2 • 14 years ago

Hmm, the microsummary service is a toolkit component. Does Firefox still have the same hiddenWindow stuff? Is this happening there as well?

	Stefan [:stefanh] Reporter
	Comment 3 • 14 years ago

Firefox has a slightly different implementation of the hiddenwindow stuff, so I'm not sure we can rely on toolkit doing the right thing here.

	Ian Neal
	Comment 4 • 14 years ago

I suppose the question is whether it does happen in Firefox, if it does then this bug or a clone should be put against firefox/toolkit. Either way I don't think we can block on this but if we get a fix we should try and take it.

	Jens Hatlak (:InvisibleSmiley)
	Comment 5 • 14 years ago

Bug 524091 removed microsummaries support, including the cited file. I guess we need to open a bug about removing all traces from our code and then investigate closing this one. Before I file such a bug, I think the Council should decide whether that's OK or the feature should be kept (in which case we would have to maintain it; just asking for "better safe than sorry").

	Philip Chee
	Comment 6 • 14 years ago

The backend was removed including the toolkit components. So the front end UI code in /suite/ doesn't work anyway.

	Jens Hatlak (:InvisibleSmiley)
	Comment 7 • 14 years ago

(In reply to comment #6)
> The backend was removed including the toolkit components. So the front end
> UI code in /suite/ doesn't work anyway.

Yes, as I said. My questions was whether anyone intends to bring it back (to suite land) or that removal bug should be filed.

	Robert Kaiser
	Comment 8 • 14 years ago

Microsummaries are/were never really used in any way broadly and removing simplifies code, I'd suggest following that and also removing all traces of it in SeaMonkey.

	Philip Chee
	Comment 9 • 14 years ago

When I say backend I include the removal of code from the CPP widget code. It's hard to see how that can be brought into the Suite.

> Microsummaries are/were never really used in any way broadly and removing 
> simplifies code, I'd suggest following that and also removing all traces of it 
> in SeaMonkey.
Agreed.

	Jens Hatlak (:InvisibleSmiley)
	Comment 10 • 14 years ago

Filed bug 659277 for the removal. Once that is fixed, this one can be closed.

	Jens Hatlak (:InvisibleSmiley)
	Comment 11 • 14 years ago

Stefan, please check again with the next trunk nightly, then resolve if the error is gone.

	Jens Hatlak (:InvisibleSmiley)
	Comment 12 • 14 years ago

Bah, make that "the next trunk nightly that contains the full fix for bug 659277"...

	Stefan [:stefanh] Reporter
	Comment 13 • 14 years ago

(In reply to comment #11)
> Stefan, please check again with the next trunk nightly, then resolve if the
> error is gone.

Yeah, it's gone. Resolving as fixed, since bug 659277 obviously fixed it ;-)