Don't silently eat Inspector.TargetCrashed

headless/lib/browser/headless_devtools_client_impl.cc

Index: headless/lib/browser/headless_devtools_client_impl.cc
diff --git a/headless/lib/browser/headless_devtools_client_impl.cc b/headless/lib/browser/headless_devtools_client_impl.cc
index 010d3d113da09e661e20e5a558a5b0d0e8064494..318ab8543cea28eb4f2a0c85010532305e53963e 100644
--- a/headless/lib/browser/headless_devtools_client_impl.cc
+++ b/headless/lib/browser/headless_devtools_client_impl.cc
@@ -133,15 +133,10 @@ bool HeadlessDevToolsClientImpl::DispatchEvent(
   std::string method;
   if (!message_dict.GetString("method", &method))
     return false;
+  if (method == "Inspector.targetCrashed")
+    renderer_crashed_ = true;
   EventHandlerMap::const_iterator it = event_handlers_.find(method);
   if (it == event_handlers_.end()) {
-    // Silently swallow errors related to the target crashing. This can be
-    // observed via HeadlessWebContents::Observer::RenderProcessExited.
-    if (method == "Inspector.targetCrashed") {
-      renderer_crashed_ = true;
-      return true;
-    }
-
     NOTREACHED() << "Unknown event: " << method;
     return false;
   }