Closed Bug 1668200 Opened 4 years ago Closed 2 years ago

Use StableSort member function in AnimationEventDispatcher

Categories

(Core :: DOM: Animation, task)

Product:
Core
Component:
DOM: Animation
Type:
task

Tracking

()

Status:
RESOLVED FIXED
Milestone:
118 Branch
Tracking Flags:
Tracking Status
firefox118 --- fixed

People

(Reporter: sg, Assigned: Logan)

References

Details

(Keywords: good-first-bug)

Attachments

(1 file)

Bug 1668200 - use StableSort member function in AnimationEventDispatcher r?#firefox-animation-reviewers
48 bytes, text/x-phabricator-request
Details | Review

I stumpled upon the comment in AnimationEventDispatcher::SortEvents that mentions the StableSort member function should be used, which is now available (it uses std::stable_sort but is more efficient since it avoids redundant bounds checks).

Keywords: good-first-bug
Severity: -- → S3
Assignee: nobody → loganrosen
Status: NEW → ASSIGNED
Pushed by hikezoe.birchill@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/087ea3149d46 use StableSort member function in AnimationEventDispatcher r=birtles

https://hg.mozilla.org/mozilla-central/rev/087ea3149d46

Status: ASSIGNED → RESOLVED
Closed: 2 years ago
status-firefox118: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 118 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: