Closed
Bug 5180
Opened 26 years ago
Closed 26 years ago
GTK nsWidget gets new look and feel objects in constructor
Categories
(Core :: XUL, defect, P3)
Tracking
()
VERIFIED
FIXED
People
(Reporter: bruce, Assigned: pavlov)
Details
Attachments
(2 files)
(deleted),
patch
|
Details | Diff | Splinter Review | |
(deleted),
patch
|
Details | Diff | Splinter Review |
the GTK nsWidget constructor gets a Look and feel object everytime, just to get
a single value from it and destroy it.
http://lxr.mozilla.org/seamonkey/source/widget/src/gtk/nsWidget.cpp#39
we could do this by creating a style each time, but in order to get the correct
colors, this is needed. (saving the overhead from L&F but still creating some
overhead). This is required in part because we are using GTK and therefore
should support its themes. If we don't do this, then the colors get fubar, and
broken. I will look into this in more detail.
Comment 2•26 years ago
|
||
Is there any way to be notified when GTK's style info changes? Can we just
cache the nsILookAndFeel * for now, and avoid the CreateInterface overhead every
time?
Updated•26 years ago
|
Status: NEW → RESOLVED
Closed: 26 years ago
Resolution: --- → FIXED
Comment 3•26 years ago
|
||
I've committed the attached patch.
Comment 4•26 years ago
|
||
Comment 5•26 years ago
|
||
Pavlov believes that it's safe for the LnF object to cache the GtkStyle * as
well, so I've committed again.
Comment 6•26 years ago
|
||
Reporter | ||
Updated•26 years ago
|
Status: RESOLVED → VERIFIED
Assignee | ||
Comment 7•26 years ago
|
||
sorry for bugzilla spam. trying to get these off my radar
You need to log in
before you can comment on or make changes to this bug.
Description
•