Closed Bug 1429616 Opened 6 years ago Closed 6 years ago

Use Variant::fmatch in RetainedDisplayListBuilder.cpp's ListLookup

Categories

(Core :: Web Painting, enhancement, P3)

Product:
Core
Component:
Web Painting
Type:
enhancement

Tracking

()

Status:
RESOLVED WONTFIX
Tracking Flags:
Tracking Status
firefox59 --- affected

People

(Reporter: mozbugz, Unassigned)

References

Details

Matt suggested making Variant::match take lambdas instead (I've opened bug 1429613 for that) and use that in ListLookup::GetSame and ListLookup::Remove for clearer code.
Priority: -- → P3
Since bug 1426044 (proposed RDL merging optimization that would have benefited from Variant::match(lambdas)) is not happening, we don't need this bug here.

But I think bug 1429613 (implementing Variant::match(lambdas)) would still be useful, so I'll let that one live.
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.