Eliminate unnecessary surface lookups in SurfaceCacheImpl::Insert

NEW
Assigned to

Status

()

3 years ago
2 years ago

People

(Reporter: seth, Assigned: seth)

Tracking

({feature})

Firefox Tracking Flags

(Not tracked)

Details

(Assignee)

Description

3 years ago
Bug 1176124 introduced some unnecessary surface lookups in SurfaceCacheImpl::Insert(), because Insert() calls Lookup() to check if there's an existing surface or placeholder, but if there *is* a placeholder, we remove it with Remove(), which needs to do a second lookup. That's because Insert() and Remove() are implementations of the public API.

We need a private API that can do the lookup and the removal of the placeholder, if needed, in a single pass.
(Assignee)

Comment 1

3 years ago
Ack, meant RemoveSurface() in comment 0, not Remove().
Keywords: feature
You need to log in before you can comment on or make changes to this bug.