Index: content/public/test/mock_render_process_host.cc |
diff --git a/content/public/test/mock_render_process_host.cc b/content/public/test/mock_render_process_host.cc |
index 7ea1a745f1be43dc456098ffd11795e67f7c962f..a5eb87852aae85609e0598b81b4d5268410dd174 100644 |
--- a/content/public/test/mock_render_process_host.cc |
+++ b/content/public/test/mock_render_process_host.cc |
@@ -10,6 +10,7 @@ |
#include "base/lazy_instance.h" |
#include "base/location.h" |
+#include "base/memory/ptr_util.h" |
#include "base/process/process_handle.h" |
#include "base/single_thread_task_runner.h" |
#include "base/threading/thread_task_runner_handle.h" |
@@ -377,28 +378,24 @@ MockRenderProcessHostFactory::MockRenderProcessHostFactory() {} |
MockRenderProcessHostFactory::~MockRenderProcessHostFactory() { |
// Detach this object from MockRenderProcesses to prevent STLDeleteElements() |
// from calling MockRenderProcessHostFactory::Remove(). |
- for (ScopedVector<MockRenderProcessHost>::iterator it = processes_.begin(); |
- it != processes_.end(); ++it) { |
- (*it)->SetFactory(NULL); |
- } |
+ for (const auto& process : processes_) |
+ process->SetFactory(nullptr); |
} |
RenderProcessHost* MockRenderProcessHostFactory::CreateRenderProcessHost( |
BrowserContext* browser_context, |
SiteInstance* site_instance) const { |
- MockRenderProcessHost* host = new MockRenderProcessHost(browser_context); |
- if (host) { |
- processes_.push_back(host); |
- host->SetFactory(this); |
- } |
- return host; |
+ processes_.push_back( |
+ base::MakeUnique<MockRenderProcessHost>(browser_context)); |
+ processes_.back()->SetFactory(this); |
+ return processes_.back().get(); |
} |
void MockRenderProcessHostFactory::Remove(MockRenderProcessHost* host) const { |
- for (ScopedVector<MockRenderProcessHost>::iterator it = processes_.begin(); |
- it != processes_.end(); ++it) { |
- if (*it == host) { |
- processes_.weak_erase(it); |
+ for (auto it = processes_.begin(); it != processes_.end(); ++it) { |
+ if (it->get() == host) { |
+ it->release(); |
+ processes_.erase(it); |
break; |
} |
} |