Open Bug 154101 Opened 23 years ago Updated 3 years ago

Investigate having ResolveStyleContextForNonElement() return smaller style contexts

Categories

(Core :: CSS Parsing and Computation, task)

task

Tracking

()

People

(Reporter: bzbarsky, Unassigned)

Details

It may make sense to create a tiny nsIStyleContext impl that basically just has a pointer to the parent and to make ResolveStyleContextForNonElement() return instances of this impl... This may or may not reduce footprint; in any case, it's worth trying.
Component: Layout → Style System
bryner, do you think this is worth it at this point? It'd require making nsStyleContext stuff almost pure virtual, creating two subclasses, etc -- sounds like a recreation of nsIStyleContext....
Priority: -- → P3
Target Milestone: --- → mozilla1.6alpha
QA Contact: chrispetersen → style-system
Keywords: helpwanted
Priority: P3 → P5
Target Milestone: mozilla1.6alpha → ---

Probably not worth it with stylo, I suspect.

Assignee: bzbarsky → nobody
Type: defect → task
Keywords: helpwanted
Priority: P5 → --

Probably not as-described at least.

We could do something like giving nsTextFrame a pointer to the parent style instead of computing a new style, and teach the frame to look at its inherited style only, or forward accessors and use the initial style for other stuff... That seems a bit error prone, but maybe worth it?

Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.