Closed Bug 573384 Opened 14 years ago Closed 14 years ago

rights infobar not displayed any more

Categories

(SeaMonkey :: Startup & Profiles, defect)

defect
Not set
normal

Tracking

(blocking-seamonkey2.1 b1+)

VERIFIED FIXED
seamonkey2.1a2
Tracking Status
blocking-seamonkey2.1 --- b1+

People

(Reporter: kairo, Assigned: neil)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

As I noted in bug 508039 comment #38, bug 558664 removed the subject parameter on the observer notification for the sessionstore-windows-restored topic, and so the infobar for "Know Your Rights" isn't being displayed any more.

I found this because places bookmarks also want to display another notification on the same event if the DB locked.
Ouch, the bug that caused the problem is bug 558644, actually.

IMHO the solutions here is one of:
1) Making our way from bug 508039 work again AND port it to Firefox as well so we don't re-do the bug 558644 mistake, or
2) Getting the window the Firefox way and leave the observer notification as it is.
Depends on: 558644
No longer depends on: 558664
Actually it was bug 547406 but yes I completely overlooked it :-(
Depends on: 547406
No longer depends on: 558644
blocking-seamonkey2.1: --- → ?
Assignee: nobody → neil
Status: NEW → ASSIGNED
Attachment #452610 - Flags: review?(kairo)
blocking-seamonkey2.1: ? → b1+
Attachment #452610 - Flags: review?(kairo) → review+
Oh, in this case, we really should port this to Firefox as well, both to improve their code and to not port over the error again.

Are you willing to file a bug, possibly even a patch?
Pushed changeset de79d19f1f17 to comm-central.
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.1a2
Version: unspecified → Trunk
User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.3a6pre) Gecko/20100619 SeaMonkey/2.1a2pre
Build ID:   20100620161917

I see the infobar in a new profile. I don't expect it to work differently on W32, Mac, or L64 than it does on L32. => VERIFIED. (If someone does not see the infobar in a fresh profile in a SeaMonkey 2.1a2pre built later than comment #5, feel free to REOPEN with details of where and how.)

The fact that, in the default theme, it is the same color as the chrome makes it easy to miss, but if it is wanted to change that it would of course be a different bug.

(I wonder why the user-agent has stopped tracking the build date but that is yet another bug if desired. Maybe intentional against fingerprinting but might make bug-tracking and finding regression ranges harder.)
Status: RESOLVED → VERIFIED
Blocks: 574645
I filed bug 574645 on doing the same trick of passing the window in Firefox code.
See Also: → 1612725
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: