Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(444)

Unified Diff: ui/display/chromeos/x11/native_display_delegate_x11.cc

Issue 2348953002: Remove stl_util's STLDeleteContainerPairSecondPointers from ui/. (Closed)
Patch Set: fix Created 4 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: ui/display/chromeos/x11/native_display_delegate_x11.cc
diff --git a/ui/display/chromeos/x11/native_display_delegate_x11.cc b/ui/display/chromeos/x11/native_display_delegate_x11.cc
index 7e4db1524102fdafd3222526c626ce4fe0583eaf..47f141f63c8b13c7faa1607a2d0bd1e8a3220e71 100644
--- a/ui/display/chromeos/x11/native_display_delegate_x11.cc
+++ b/ui/display/chromeos/x11/native_display_delegate_x11.cc
@@ -14,7 +14,7 @@
#include "base/logging.h"
#include "base/macros.h"
-#include "base/stl_util.h"
+#include "base/memory/ptr_util.h"
#include "ui/display/chromeos/x11/display_mode_x11.h"
#include "ui/display/chromeos/x11/display_snapshot_x11.h"
#include "ui/display/chromeos/x11/display_util_x11.h"
@@ -114,8 +114,6 @@ NativeDisplayDelegateX11::~NativeDisplayDelegateX11() {
ui::PlatformEventSource::GetInstance()->RemovePlatformEventDispatcher(
platform_event_dispatcher_.get());
}
-
- base::STLDeleteContainerPairSecondPointers(modes_.begin(), modes_.end());
}
void NativeDisplayDelegateX11::Initialize() {
@@ -285,7 +283,6 @@ void NativeDisplayDelegateX11::RemoveObserver(NativeDisplayObserver* observer) {
void NativeDisplayDelegateX11::InitModes() {
CHECK(screen_) << "Server not grabbed";
- base::STLDeleteContainerPairSecondPointers(modes_.begin(), modes_.end());
modes_.clear();
for (int i = 0; i < screen_->nmode; ++i) {
@@ -297,12 +294,10 @@ void NativeDisplayDelegateX11::InitModes() {
(static_cast<float>(info.hTotal) * static_cast<float>(info.vTotal));
}
- modes_.insert(
- std::make_pair(info.id,
- new DisplayModeX11(gfx::Size(info.width, info.height),
- info.modeFlags & RR_Interlace,
- refresh_rate,
- info.id)));
+ modes_.insert(std::make_pair(
+ info.id, base::MakeUnique<DisplayModeX11>(
+ gfx::Size(info.width, info.height),
+ info.modeFlags & RR_Interlace, refresh_rate, info.id)));
}
}

Powered by Google App Engine
This is Rietveld 408576698