Closed
Bug 1168156
Opened 9 years ago
Closed 9 years ago
Remove unused nsCSSValueTokenStream::mSheet
Categories
(Core :: CSS Parsing and Computation, defect)
Core
CSS Parsing and Computation
Tracking
()
RESOLVED
FIXED
mozilla41
Tracking | Status | |
---|---|---|
firefox41 | --- | fixed |
People
(Reporter: nika, Assigned: nika)
References
Details
Attachments
(1 file, 1 obsolete file)
3.10 KB,
patch
|
Details | Diff | Splinter Review |
Assignee | ||
Comment 1•9 years ago
|
||
Assignee | ||
Updated•9 years ago
|
Attachment #8610182 -
Flags: review?(cam)
Comment 2•9 years ago
|
||
Comment on attachment 8610182 [details] [diff] [review] Remove unused nsCSSValueTokenStream::mSheet Review of attachment 8610182 [details] [diff] [review]: ----------------------------------------------------------------- Bug 952338 is open on restoring the mSheet argument to ParsePropertyWithVariableReferences in nsRuleNode::ResolveVariableReferences. I guess it's OK to remove in the meantime, though. Although generally commenting out code isn't great, I think it'd be fine here (particularly because in the case mentioned in bug 952338 the pointer can actually be bad). Can you comment out those three lines and add a "// XXX Should store sheet here (see bug 952338)." comment on them?
Attachment #8610182 -
Flags: review?(cam) → review+
Assignee | ||
Comment 3•9 years ago
|
||
Add comments mentioning Bug 952338.
Attachment #8610182 -
Attachment is obsolete: true
Assignee | ||
Updated•9 years ago
|
Assignee: nobody → michael
Keywords: checkin-needed
Comment 5•9 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/d6e5da6c4a35
Status: NEW → RESOLVED
Closed: 9 years ago
status-firefox41:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla41
You need to log in
before you can comment on or make changes to this bug.
Description
•