diff options
| author | jaekyun <jaekyun@chromium.org> | 2014-12-04 15:33:35 -0800 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2014-12-04 23:34:12 +0000 |
| commit | 37e572a3b60109e457a09df566d15c6f96b79210 (patch) | |
| tree | 55d1f092f2ee9e5578f17e05213ec30554f16578 /content/browser/child_process_security_policy_browsertest.cc | |
| parent | 59246ff8886fe98b28405a8e358982a6dc0a585e (diff) | |
| download | chromium_src-37e572a3b60109e457a09df566d15c6f96b79210.zip chromium_src-37e572a3b60109e457a09df566d15c6f96b79210.tar.gz chromium_src-37e572a3b60109e457a09df566d15c6f96b79210.tar.bz2 | |
Use StopChildProcess instead of base::KillProcess to kill a renderer process
base::KillProcess doesn't work on android because a renderer process is running as an isolated process.
So we need to use StopChildProcess to kill it.
BUG=433068
Review URL: https://codereview.chromium.org/740813004
Cr-Commit-Position: refs/heads/master@{#306931}
Diffstat (limited to 'content/browser/child_process_security_policy_browsertest.cc')
| -rw-r--r-- | content/browser/child_process_security_policy_browsertest.cc | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/content/browser/child_process_security_policy_browsertest.cc b/content/browser/child_process_security_policy_browsertest.cc index b509a49..c1e3c39 100644 --- a/content/browser/child_process_security_policy_browsertest.cc +++ b/content/browser/child_process_security_policy_browsertest.cc @@ -48,8 +48,7 @@ IN_PROC_BROWSER_TEST_F(ChildProcessSecurityPolicyInProcessBrowserTest, NoLeak) { 1U); WebContents* web_contents = shell()->web_contents(); - base::KillProcess(web_contents->GetRenderProcessHost()->GetHandle(), - RESULT_CODE_KILLED, true); + web_contents->GetRenderProcessHost()->Shutdown(RESULT_CODE_KILLED, true); web_contents->GetController().Reload(true); EXPECT_EQ( |