Closed
Bug 676057
Opened 15 years ago
Closed 15 years ago
Nuke AttributeSelectorEntry
Categories
(Core :: CSS Parsing and Computation, defect, P2)
Core
CSS Parsing and Computation
Tracking
()
RESOLVED
FIXED
mozilla8
People
(Reporter: khuey, Assigned: bzbarsky)
References
Details
Attachments
(1 file)
|
6.86 KB,
patch
|
dbaron
:
review+
|
Details | Diff | Splinter Review |
It can be completely replaced with AtomSelectorEntry, which is basically identical.
| Assignee | ||
Comment 1•15 years ago
|
||
Attachment #550185 -
Flags: review?(dbaron)
| Assignee | ||
Updated•15 years ago
|
Priority: -- → P2
Whiteboard: [need review]
Comment on attachment 550185 [details] [diff] [review]
Get rid of the malloc-happy AttributeSelectorEntry class.
r=dbaron
though I'd note that AtomSelector_CSOps doesn't need the fancy RuleHash_CSMatchEntry (or, as a result, to need AtomSelector_GetKey or any of the RuleHashTableOps fanciness).
Attachment #550185 -
Flags: review?(dbaron) → review+
| Assignee | ||
Comment 3•15 years ago
|
||
> though I'd note that AtomSelector_CSOps doesn't need the fancy
> RuleHash_CSMatchEntry
Filed bug 676174.
Blocks: 676174
Whiteboard: [need review] → [need landing]
| Assignee | ||
Comment 4•15 years ago
|
||
Flags: in-testsuite-
Whiteboard: [need landing]
Target Milestone: --- → mozilla8
Comment 5•15 years ago
|
||
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•