Closed Bug 283419 Opened 20 years ago Closed 20 years ago

Clean up interaction between BrowserWrapper and BrowserWindowController

Categories

(Camino Graveyard :: General, defect)

Product:
Camino Graveyard
Component:
General
Platform:
PowerPC
macOS
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: sfraser_bugs, Assigned: sfraser_bugs)

References

Details

Attachments

(1 file)

Patch that got checked in
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.
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.
Fixed. Please be on the lookout for regressions.
Status: NEW → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
*** 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.

Attachment

General

Creator:
Created:
Updated:
Size: