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

Unified Diff: content/plugin/plugin_thread.cc

Issue 999033003: Remove uses of KillProcess() (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 9 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: content/plugin/plugin_thread.cc
diff --git a/content/plugin/plugin_thread.cc b/content/plugin/plugin_thread.cc
index a688902c27d0d7538ab6fc1d68ae40bb02a7916b..7fd9bddeba4df5b39750a7743f0dc498ebf864b4 100644
--- a/content/plugin/plugin_thread.cc
+++ b/content/plugin/plugin_thread.cc
@@ -16,8 +16,7 @@
#include "base/bind.h"
#include "base/command_line.h"
#include "base/lazy_instance.h"
-#include "base/process/kill.h"
-#include "base/process/process_handle.h"
+#include "base/process/process.h"
#include "base/threading/thread_local.h"
#include "content/child/blink_platform_impl.h"
#include "content/child/child_process.h"
@@ -57,7 +56,7 @@ class EnsureTerminateMessageFilter : public IPC::MessageFilter {
private:
void Terminate() {
- base::KillProcess(base::GetCurrentProcessHandle(), 0, false);
+ base::Process::Current().Terminate(0, false);
}
};
@@ -116,7 +115,7 @@ void PluginThread::Shutdown() {
PluginLib::UnloadAllPlugins();
if (forcefully_terminate_plugin_process_)
- base::KillProcess(base::GetCurrentProcessHandle(), 0, /* wait= */ false);
+ base::Process::Current().Terminate(0, /* wait= */ false);
cpu_(ooo_6.6-7.5) 2015/03/17 20:36:31 this code review is a good example of why bool par
lazy_tls.Pointer()->Set(NULL);
}

Powered by Google App Engine
This is Rietveld 408576698