Closed
Bug 283419
Opened 21 years ago
Closed 21 years ago
Clean up interaction between BrowserWrapper and BrowserWindowController
Categories
(Camino Graveyard :: General, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: sfraser_bugs, Assigned: sfraser_bugs)
References
Details
Attachments
(1 file)
|
33.04 KB,
patch
|
Details | Diff | Splinter Review |
The BrowserWrapper currently has a lot of code that tests for 'mIsPrimary' to
determine whether to update the UI in various ways.
This would be much better if the BrowserWrapper had a delegate, implemented by
the BWC, and the delegate is only non-null for the frontmost window.
|
Assignee | |
Comment 1•21 years ago
|
||
I goofed and checked in my changes by accident. But, hey, they were ready to go
in anyway :)
Here's the patch for good measure. I've been running this for a while, and
haven't seen any regressions.
|
|
Assignee | |
Comment 2•21 years ago
|
||
Fixed. Please be on the lookout for regressions.
Status: NEW → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
|
|
Assignee | |
Comment 3•21 years ago
|
||
*** Bug 280619 has been marked as a duplicate of this bug. ***
You need to log in
before you can comment on or make changes to this bug.
Description
•