Closed Bug 404315 Opened 13 years ago Closed 13 years ago
[FIX]Improve signature of ns
As the summary says. Also drops more stuff when done parsing, and even more if we recycle the parser, so we don't keep things alive for a long time.
Comment on attachment 289293 [details] [diff] [review] Like so r+sr=dbaron. Sorry for the delay here. I wonder if we should move even further to a model where the sheet is input to nsICSSParser::Parse and some of the other methods (instead of a pile of URIs and a principal), and the parser only holds onto it for the duration of those methods.
That might not be a bad idea, esp. once we have the mmgc stuff and don't need to refcount all those things as we get them off the sheet.
Comment on attachment 289293 [details] [diff] [review] Like so This is a simple change to make it harder for users of this API to shoot themselves in the foot.
Attachment #289293 - Flags: approval1.9?
Comment on attachment 289293 [details] [diff] [review] Like so a=beltzner for 1.9
Attachment #289293 - Flags: approval1.9? → approval1.9+
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.