Kill the app when render process is killed

android_webview/browser/aw_browser_terminator.cc

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "android_webview/browser/aw_browser_terminator.h"
 
 #include <unistd.h>
 
 #include "android_webview/browser/aw_render_process_gone_delegate.h"
 #include "android_webview/common/aw_descriptors.h"
@@ skipping 49 matching lines @@
   for (auto delegate : delegates)
     delegate->OnRenderProcessGone(child_process_id);
 }
 
 void OnRenderProcessGoneDetail(int child_process_id, bool crashed) {
   DCHECK_CURRENTLY_ON(BrowserThread::UI);
   std::vector<AwRenderProcessGoneDelegate*> delegates;
   GetAwRenderProcessGoneDelegatesForRenderProcess(child_process_id, &delegates);
   for (auto delegate : delegates) {
     if (!delegate->OnRenderProcessGoneDetail(child_process_id, crashed)) {
-      // Keeps this log unchanged, CTS test uses it to detect crash.
-      LOG(FATAL) << "Render process's abnormal termination wasn't handled by"
-                 << " all associated webviews, triggering application crash";
+      if (crashed) {
+        // Keeps this log unchanged, CTS test uses it to detect crash.
+        LOG(FATAL) << "Render process's crash wasn't handled by all associated"
+                   << " webviews, triggering application crash";
+      } else {
+        // The render process was most likely killed for OOM or switching
+        // WebView provider, to make WebView backward compatible, kills the
+        // browser process instead of triggering crash.
+        LOG(ERROR) << "The event of render process has been killed wasn't"

[Torne, 2017/01/31 11:56:41]

not very clear wording; how about "Render process kill (OOM or update) wasn't
handed by all associated webviews, killing application." ?

[michaelbai, 2017/01/31 17:46:21]

Done.

+                   << " handled by all associated webviews, killing the"
+                   << " process.";
+        kill(getpid(), SIGKILL);
+      }
     }
   }
 }
 
 }  // namespace
 
 AwBrowserTerminator::AwBrowserTerminator() {}
 
 AwBrowserTerminator::~AwBrowserTerminator() {}
 
@@ skipping 57 matching lines @@
   OnRenderProcessGone(child_process_id);
   DCHECK(pipe->handle() != base::SyncSocket::kInvalidHandle);
   BrowserThread::PostTask(
       BrowserThread::FILE, FROM_HERE,
       base::Bind(&AwBrowserTerminator::ProcessTerminationStatus,
                  child_process_id,
                  base::Passed(std::move(pipe))));
 }
 
 }  // namespace breakpad