Closed Bug 1489702 Opened 6 years ago Closed 6 years ago

Store mClipChainsToDestroy inside the clip chains themselves

Categories

(Core :: Web Painting, enhancement)

enhancement
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla64
Tracking Status
firefox64 --- fixed

People

(Reporter: dthayer, Assigned: dthayer)

References

(Blocks 1 open bug)

Details

(Whiteboard: [fxperf:p1])

Attachments

(1 file)

(Just add a pointer to the next item in the linked list to DisplayItemClipChain.)
Fairly straightforward - went with extra indirection in the
FreeClipChains method to avoid all the extra variables and
code paths.
Comment on attachment 9007398 [details]
Bug 1489702 - Move mClipChainsToDestroy inline r?mattwoodrow

Matt Woodrow (:mattwoodrow) has approved the revision.
Attachment #9007398 - Flags: review+
Keywords: checkin-needed
Woops - single patch, landing this one through Lando.
Keywords: checkin-needed
Pushed by dothayer@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/283bfe626f5b
Move mClipChainsToDestroy inline r=mattwoodrow
https://hg.mozilla.org/mozilla-central/rev/283bfe626f5b
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla64
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: