Closed Bug 492531 Opened 11 years ago Closed 11 years ago

Optimize some CSS and Reflow classes for memory usage

Categories

(Core :: CSS Parsing and Computation, defect)

x86_64
Linux
defect
Not set

Tracking

()

RESOLVED FIXED

People

(Reporter: Swatinem, Assigned: Swatinem)

References

(Depends on 1 open bug)

Details

Attachments

(1 file)

These are the wins:

RuleHash: 8bytes
nsStyleContent: 8bytes
nsStyleSVGReset: 8bytes
nsStyleSVGPaint: 8bytes
nsHTMLReflowState: 8bytes
nsCSSToken: 8bytes
nsCSSScanner: 0bytes :(
CascadeEnumData: 0bytes :(
nsCSSBorderRenderer: 8bytes
nsCSSSelector: 8bytes
nsFrameConstructorState: 8bytes
nsAttrSelector: 8bytes
InlineBackgroundData: 8bytes
nsCSSStyleSheetInner: 8bytes
Attachment #376905 - Flags: superreview?(dbaron)
Attachment #376905 - Flags: review?(dbaron)
I'm worried some of these might be making things a good bit less readable by separating related things (particularly RuleHash, maybe also nsCSSScanner, nsCSSSelector, nsAttrSelector).
Comment on attachment 376905 [details] [diff] [review]
patch [pushed: comment 3]

Please undo the changes to nsCSSScanner and RuleHash.  I think they make the code significantly less readable.

r+sr=dbaron on the rest

If you want to propose changing RuleHash or nsCSSScanner in a different way, please do that in a separate patch.  (The nsCSSToken changes are fine, though; they're in nsCSSScanner.h as well.)

r+sr=dbaron on the rest, although I'm not sure I'm qualified to say that this doesn't make InlineBackgroundData or nsCSSBorderRenderer less readable
Attachment #376905 - Flags: superreview?(dbaron)
Attachment #376905 - Flags: superreview+
Attachment #376905 - Flags: review?(dbaron)
Attachment #376905 - Flags: review+
Comment on attachment 376905 [details] [diff] [review]
patch [pushed: comment 3]

http://hg.mozilla.org/mozilla-central/rev/a1fc27041e53
Attachment #376905 - Attachment description: patch → patch [pushed: comment 3]
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.