service worker: Add crash keys for OnMessageReceived crash

content/browser/service_worker/service_worker_dispatcher_host.cc

 // Copyright 2013 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 "content/browser/service_worker/service_worker_dispatcher_host.h"
 
 #include <utility>
 
+#include "base/debug/crash_logging.h"
 #include "base/logging.h"
 #include "base/macros.h"
 #include "base/memory/ptr_util.h"
 #include "base/profiler/scoped_tracker.h"
 #include "base/strings/utf_string_conversions.h"
 #include "base/threading/thread_task_runner_handle.h"
 #include "base/trace_event/trace_event.h"
 #include "content/browser/bad_message.h"
 #include "content/browser/message_port_message_filter.h"
 #include "content/browser/message_port_service.h"
@@ skipping 59 matching lines @@
 
 bool CanGetRegistration(const GURL& document_url,
                         const GURL& given_document_url) {
   DCHECK(document_url.is_valid());
   DCHECK(given_document_url.is_valid());
   return document_url.GetOrigin() == given_document_url.GetOrigin() &&
          OriginCanAccessServiceWorkers(document_url) &&
          OriginCanAccessServiceWorkers(given_document_url);
 }
 
+std::string GetIPCMessageTypeAsString(uint32_t type) {
+  switch (type) {
+    case ServiceWorkerHostMsg_InstallEventFinished::ID:
+      return "ServiceWorkerHostMsg_InstallEventFinished";
+    case ServiceWorkerHostMsg_ActivateEventFinished::ID:
+      return "ServiceWorkerHostMsg_ActivateEventFinished";
+    case ServiceWorkerHostMsg_ExtendableMessageEventFinished::ID:
+      return "ServiceWorkerHostMsg_ExtendableMessageEventFinished";
+    case ServiceWorkerHostMsg_FetchEventResponse::ID:
+      return "ServiceWorkerHostMsg_FetchEventResponse";
+    case ServiceWorkerHostMsg_FetchEventFinished::ID:
+      return "ServiceWorkerHostMsg_FetchEventFinished";
+    case ServiceWorkerHostMsg_NotificationClickEventFinished::ID:
+      return "ServiceWorkerHostMsg_NotificationClickEventFinished";
+    case ServiceWorkerHostMsg_NotificationCloseEventFinished::ID:
+      return "ServiceWorkerHostMsg_NotificationCloseEventFinished";
+    case ServiceWorkerHostMsg_PushEventFinished::ID:
+      return "ServiceWorkerHostMsg_PushEventFinished";
+    case ServiceWorkerHostMsg_Pong::ID:
+      return "ServiceWorkerHostMsg_Pong";
+    case ServiceWorkerHostMsg_GetClient::ID:
+      return "ServiceWorkerHostMsg_GetClient";
+    case ServiceWorkerHostMsg_GetClients::ID:
+      return "ServiceWorkerHostMsg_GetClients";
+    case ServiceWorkerHostMsg_PostMessageToClient::ID:
+      return "ServiceWorkerHostMsg_PostMessageToClient";
+    case ServiceWorkerHostMsg_SetCachedMetadata::ID:
+      return "ServiceWorkerHostMsg_SetCachedMetadata";
+    case ServiceWorkerHostMsg_ClearCachedMetadata::ID:
+      return "ServiceWorkerHostMsg_ClearCachedMetadata";
+    case ServiceWorkerHostMsg_OpenWindow::ID:
+      return "ServiceWorkerHostMsg_OpenWindow";
+    case ServiceWorkerHostMsg_FocusClient::ID:
+      return "ServiceWorkerHostMsg_FocusClient";
+    case ServiceWorkerHostMsg_NavigateClient::ID:
+      return "ServiceWorkerHostMsg_NavigateClient";
+    case ServiceWorkerHostMsg_SkipWaiting::ID:
+      return "ServiceWorkerHostMsg_SkipWaiting";
+    case ServiceWorkerHostMsg_ClaimClients::ID:
+      return "ServiceWorkerHostMsg_ClaimClients";
+    case ServiceWorkerHostMsg_RegisterForeignFetchScopes::ID:
+      return "ServiceWorkerHostMsg_RegisterForeignFetchScopes";
+  }
+  return "Unknown";
+}
+
 }  // namespace
 
 ServiceWorkerDispatcherHost::ServiceWorkerDispatcherHost(
     int render_process_id,
     MessagePortMessageFilter* message_port_message_filter,
     ResourceContext* resource_context)
     : BrowserMessageFilter(kFilteredMessageClasses,
                            arraysize(kFilteredMessageClasses)),
       render_process_id_(render_process_id),
       message_port_message_filter_(message_port_message_filter),
@@ skipping 102 matching lines @@
                         OnIncrementRegistrationRefCount)
     IPC_MESSAGE_HANDLER(ServiceWorkerHostMsg_DecrementRegistrationRefCount,
                         OnDecrementRegistrationRefCount)
     IPC_MESSAGE_HANDLER(ServiceWorkerHostMsg_TerminateWorker, OnTerminateWorker)
     IPC_MESSAGE_UNHANDLED(handled = false)
   IPC_END_MESSAGE_MAP()
 
   if (!handled && GetContext()) {
     handled = GetContext()->embedded_worker_registry()->OnMessageReceived(
         message, render_process_id_);
-    if (!handled)
+    if (!handled) {
+      // Temporary debugging for https://crbug.com/625040
+      EmbeddedWorkerInstance* worker =
+          GetContext()->embedded_worker_registry()->GetWorkerForMessage(
+              render_process_id_, message.routing_id());
+      base::debug::ScopedCrashKey("swdh_not_handled_message_type",
+                                  GetIPCMessageTypeAsString(message.type()));
+      base::debug::ScopedCrashKey(
+          "swdh_not_handled_worker_status",
+          EmbeddedWorkerInstance::StatusToString(worker->status()));
       bad_message::ReceivedBadMessage(this, bad_message::SWDH_NOT_HANDLED);
+    }
   }
 
   return handled;
 }
 
 bool ServiceWorkerDispatcherHost::Send(IPC::Message* message) {
   if (channel_ready_) {
     BrowserMessageFilter::Send(message);
     // Don't bother passing through Send()'s result: it's not reliable.
     return true;
@@ skipping 1179 matching lines @@
   if (!handle) {
     bad_message::ReceivedBadMessage(this,
                                     bad_message::SWDH_TERMINATE_BAD_HANDLE);
     return;
   }
   handle->version()->StopWorker(
       base::Bind(&ServiceWorkerUtils::NoOpStatusCallback));
 }
 
 }  // namespace content