Bug 1642989 - Use RemoveLastElements instead of RemoveElementsAt where possible. r=froydnj

Differential Revision: https://phabricator.services.mozilla.com/D78049
This commit is contained in:
Simon Giesecke
2020-06-08 08:50:15 +00:00
parent 2e6d1e8632
commit 057eeeade2
16 changed files with 38 additions and 41 deletions

View File

@@ -3536,7 +3536,8 @@ void HTMLMediaElement::UpdateOutputTrackSources() {
}
// Then work out the differences.
mediaTracksToAdd.RemoveElementsAt(
mediaTracksToAdd.RemoveLastElements(
mediaTracksToAdd.end() -
std::remove_if(mediaTracksToAdd.begin(), mediaTracksToAdd.end(),
[this, &trackSourcesToRemove](const auto& track) {
const bool remove =
@@ -3545,8 +3546,7 @@ void HTMLMediaElement::UpdateOutputTrackSources() {
trackSourcesToRemove.RemoveElement(track->GetId());
}
return remove;
}),
mediaTracksToAdd.end());
}));
// First remove stale track sources.
for (const auto& id : trackSourcesToRemove) {