Closed
Bug 412575
Opened 18 years ago
Closed 18 years ago
Add getBrowserForDocument() method to the tabbrowser widget
Categories
(SeaMonkey :: UI Design, enhancement)
SeaMonkey
UI Design
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: philip.chee, Assigned: philip.chee)
Details
Attachments
(1 file)
2.95 KB,
patch
|
neil
:
review+
jag+mozilla
:
superreview+
|
Details | Diff | Splinter Review |
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.
![]() |
Assignee | |
Updated•18 years ago
|
Attachment #297329 -
Flags: review?(neil)
Updated•18 years ago
|
Attachment #297329 -
Flags: review?(neil) → review+
![]() |
Assignee | |
Comment 1•18 years ago
|
||
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)
Updated•18 years ago
|
Attachment #297329 -
Flags: superreview?(jag) → superreview+
![]() |
Assignee | |
Updated•18 years ago
|
Keywords: checkin-needed
Updated•18 years ago
|
Assignee: nobody → philip.chee
Comment 2•18 years ago
|
||
Checking in suite/browser/tabbrowser.xml;
/cvsroot/mozilla/suite/browser/tabbrowser.xml,v <-- tabbrowser.xml
new revision: 1.192; previous revision: 1.191
done
You need to log in
before you can comment on or make changes to this bug.
Description
•