Fixing crash with sig: blink::FrameConsole::addMessage

Source/core/frame/FrameConsole.cpp

 /*
  * Copyright (C) 2013 Apple Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  *
  * 1.  Redistributions of source code must retain the above copyright
  *     notice, this list of conditions and the following disclaimer.
  * 2.  Redistributions in binary form must reproduce the above copyright
@@ skipping 67 matching lines @@
 {
     RefPtrWillBeRawPtr<ConsoleMessage> consoleMessage = prpConsoleMessage;
     if (muteCount && consoleMessage->source() != ConsoleAPIMessageSource)
         return;
 
     // FIXME: This should not need to reach for the main-frame.
     // Inspector code should just take the current frame and know how to walk itself.
     ExecutionContext* context = frame().document();
     if (!context)
         return;
+    if (!messageStorage())
+        return;
 
     String messageURL;
     unsigned lineNumber = 0;
     if (consoleMessage->callStack() && consoleMessage->callStack()->size()) {
         lineNumber = consoleMessage->callStack()->at(0).lineNumber();
         messageURL = consoleMessage->callStack()->at(0).sourceURL();
     } else {
         lineNumber = consoleMessage->lineNumber();
         messageURL = consoleMessage->url();
     }
@@ skipping 60 matching lines @@
 }
 
 void FrameConsole::unmute()
 {
     ASSERT(muteCount > 0);
     muteCount--;
 }
 
 ConsoleMessageStorage* FrameConsole::messageStorage()
 {
-    ASSERT(m_frame->page());
-    return &m_frame->page()->frameHost().consoleMessageStorage();
+    if (!m_frame->host())
+        return nullptr;
+    return &m_frame->host()->consoleMessageStorage();
 }
 
 void FrameConsole::clearMessages()
 {
-    messageStorage()->clear();
+    ConsoleMessageStorage* storage = messageStorage();
+    if (storage)
+        storage->clear();
 }
 
 void FrameConsole::adoptWorkerMessagesAfterTermination(WorkerGlobalScopeProxy* proxy)
 {
-    messageStorage()->adoptWorkerMessagesAfterTermination(proxy);
+    ConsoleMessageStorage* storage = messageStorage();
+    if (storage)
+        storage->adoptWorkerMessagesAfterTermination(proxy);
 }
 
 void FrameConsole::didFailLoading(unsigned long requestIdentifier, const ResourceError& error)
 {
     if (error.isCancellation()) // Report failures only.
         return;
+    ConsoleMessageStorage* storage = messageStorage();
+    if (!storage)
+        return;
     StringBuilder message;
     message.appendLiteral("Failed to load resource");
     if (!error.localizedDescription().isEmpty()) {
         message.appendLiteral(": ");
         message.append(error.localizedDescription());
     }
     RefPtrWillBeRawPtr<ConsoleMessage> consoleMessage = ConsoleMessage::create(NetworkMessageSource, ErrorMessageLevel, message.toString(), error.failingURL());
     consoleMessage->setRequestIdentifier(requestIdentifier);
-    messageStorage()->reportMessage(consoleMessage.release());
+    storage->reportMessage(consoleMessage.release());
 }
 
 void FrameConsole::trace(Visitor* visitor)
 {
     visitor->trace(m_frame);
-    visitor->trace(m_consoleMessageStorage);
 }
 
 } // namespace blink