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

Unified Diff: content/child/child_thread_impl.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/child/child_thread_impl.cc
diff --git a/content/child/child_thread_impl.cc b/content/child/child_thread_impl.cc
index cccefe597917512c7a51c35b821ba4add9ed0911..4aa4aa6b5e4701c5cf82a85e26580189fd4327bb 100644
--- a/content/child/child_thread_impl.cc
+++ b/content/child/child_thread_impl.cc
@@ -18,7 +18,7 @@
#include "base/message_loop/message_loop.h"
#include "base/message_loop/timer_slack.h"
#include "base/metrics/field_trial.h"
-#include "base/process/kill.h"
+#include "base/process/process.h"
#include "base/process/process_handle.h"
#include "base/strings/string_number_conversions.h"
#include "base/strings/string_util.h"
@@ -713,7 +713,7 @@ void ChildThreadImpl::OnProcessFinalRelease() {
void ChildThreadImpl::EnsureConnected() {
VLOG(0) << "ChildThreadImpl::EnsureConnected()";
- base::KillProcess(base::GetCurrentProcessHandle(), 0, false);
+ base::Process::Current().Terminate(0, false);
}
cpu_(ooo_6.6-7.5) 2015/03/17 20:36:31 ensure connected --> kills ensure dead --> connect
rvargas (doing something else) 2015/03/17 22:40:53 That sounds reasonable!. (I don't really know wha
void ChildThreadImpl::OnProcessBackgrounded(bool background) {

Powered by Google App Engine
This is Rietveld 408576698