Closed
Bug 70995
Opened 24 years ago
Closed 24 years ago
static copy of a parsed selectors unused in StyleRule
Categories
(Core :: CSS Parsing and Computation, defect)
Core
CSS Parsing and Computation
Tracking
()
RESOLVED
FIXED
People
(Reporter: glazou, Assigned: glazou)
References
Details
Attachments
(1 file)
|
8.81 KB,
patch
|
Details | Diff | Splinter Review |
Implementing new selectors, I saw yesterday that we still (a) build the string
image of a parsed selector (b) still keep it in nsCSSStyleRule.mSourceString !!
This causes many useless AppendToString and consumes memory.
Patch for bug 52381 introduced dynamic building of nsCSSSelector::ToString...
Proposing tomorrow morning an optimization patch
|
Assignee | |
Comment 1•24 years ago
|
||
Status: NEW → ASSIGNED
|
|
Assignee | |
Comment 2•24 years ago
|
||
|
|
Assignee | |
Comment 3•24 years ago
|
||
Patch proposed above. Cc:ing David Baron.
need r= ?
|
||
Comment 4•24 years ago
|
||
r=hewitt
|
||
Comment 5•24 years ago
|
||
sr=attinasi - very nice savings, Daniel. Why didn't I see that???
|
|
Assignee | |
Comment 6•24 years ago
|
||
checked in :-)
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
This problem has also been discussed in bug 42953 and is not limited to selectors.
You need to log in
before you can comment on or make changes to this bug.
Description
•