Use base::EraseIf() instead of erase(std::remove_if()) in content/
This patch simplifies conditions around the code in content/.
Bug: 875665
Change-Id: I52e639adce40a86e20304ee64c52d3b399c39841
Reviewed-on: https://chromium-review.googlesource.com/1215147
Reviewed-by: Avi Drissman <[email protected]>
Commit-Queue: Jinho Bang <[email protected]>
Cr-Commit-Position: refs/heads/master@{#596261}diff --git a/content/browser/media/media_internals.cc b/content/browser/media/media_internals.cc
index d82abfc..81d0ffa 100644
--- a/content/browser/media/media_internals.cc
+++ b/content/browser/media/media_internals.cc
@@ -12,6 +12,7 @@
#include "base/macros.h"
#include "base/metrics/histogram_functions.h"
#include "base/metrics/histogram_macros.h"
+#include "base/stl_util.h"
#include "base/strings/string16.h"
#include "base/strings/string_number_conversions.h"
#include "base/strings/stringprintf.h"
@@ -873,11 +874,9 @@
// Remove all events for a given player as soon as we have to remove a
// single event for that player to avoid showing incomplete players.
const int id_to_remove = saved_events.front().id;
- saved_events.erase(std::remove_if(saved_events.begin(), saved_events.end(),
- [&](const media::MediaLogEvent& event) {
- return event.id == id_to_remove;
- }),
- saved_events.end());
+ base::EraseIf(saved_events, [&](const media::MediaLogEvent& event) {
+ return event.id == id_to_remove;
+ });
}
}
