Closed Bug 412575 Opened 18 years ago Closed 18 years ago

Add getBrowserForDocument() method to the tabbrowser widget

Categories

(SeaMonkey :: UI Design, enhancement)

enhancement
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: philip.chee, Assigned: philip.chee)

Details

Attachments

(1 file)

Bug 407082 removed getBrowserForDocument() from navigator.js. Several Firefox extensions I ported to SeaMonkey rely on gBrowser.getBrowserForDocument() which SeaMonkey doesn't have. I was able to workaround that by using the implementation in navigator.js but that's gone away. So to make it easier for Firefox extension authors to adapt their tabbrowser extensions to the new SeaMonkey suite I propose to add this convenience method to our tabbrowser implementation. Also a some Firefox extensions expect that the menupopup to have an anonid="tabContextMenu" so that they can hook their own menu items to this popup.
Attachment #297329 - Flags: review?(neil)
Attachment #297329 - Flags: review?(neil) → review+
Comment on attachment 297329 [details] [diff] [review] Patch 1.0 add getBrowserForDocument method to tabbrowser. I hope I've got the right super-reviewer.
Attachment #297329 - Flags: superreview?(jag)
Attachment #297329 - Flags: superreview?(jag) → superreview+
Keywords: checkin-needed
Assignee: nobody → philip.chee
Checking in suite/browser/tabbrowser.xml; /cvsroot/mozilla/suite/browser/tabbrowser.xml,v <-- tabbrowser.xml new revision: 1.192; previous revision: 1.191 done
Status: NEW → RESOLVED
Closed: 18 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Component: XP Apps: GUI Features → UI Design
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: