fixup! [M44_2403] Chromium upversion to m44_2403 branch
authorArnaud Renevier <a.renevier@samsung.com>
Thu, 17 Sep 2015 01:04:42 +0000 (18:04 -0700)
committerYoungsoo Choi <kenshin.choi@samsung.com>
Tue, 10 Jul 2018 06:57:09 +0000 (06:57 +0000)
Use StopChildProcess instead of base::KillProcess to kill a renderer
process.

Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=14039

Reviewed by: a1.gomes, sns.park

Change-Id: Icfdb4a657cf63d82a90b2de1796aa8069d7d7e41
Signed-off-by: Arnaud Renevier <a.renevier@samsung.com>
tizen_src/chromium_impl/content/browser/web_contents/web_contents_impl_efl.cc

index 671614a..25291b4 100644 (file)
@@ -107,16 +107,12 @@ void WebContentsImplEfl::CreateNewWindow(
   // this WebContentsImpl instance. If any other process sends the request,
   // it is invalid and the process must be terminated.
   if (GetRenderProcessHost()->GetID() != render_process_id) {
-    base::ProcessHandle process_handle =
-        RenderProcessHost::FromID(render_process_id)->GetHandle();
+    RenderProcessHost* rph = RenderProcessHost::FromID(render_process_id);
+    base::ProcessHandle process_handle = rph->GetHandle();
     if (process_handle != base::kNullProcessHandle) {
       RecordAction(
           base::UserMetricsAction("Terminate_ProcessMismatch_CreateNewWindow"));
-#if !defined(EWK_BRINGUP)
-// [M44_2403] Temporary disabling the codes for switching to new chromium
-//            FIXME: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=14039
-      base::KillProcess(process_handle, RESULT_CODE_KILLED, false);
-#endif
+      rph->Shutdown(RESULT_CODE_KILLED, false);
     }
     return;
   }