Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(108)

Unified Diff: third_party/WebKit/Source/core/frame/FrameConsole.cpp

Issue 2003433004: Remove dependency from ConsoleMessage to workers. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@1999463002
Patch Set: messageN Created 4 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/core/frame/FrameConsole.cpp
diff --git a/third_party/WebKit/Source/core/frame/FrameConsole.cpp b/third_party/WebKit/Source/core/frame/FrameConsole.cpp
index 28180db3dc5186a84ae3ca56a384d1ebf9e58a4a..0e7c974e97a3b81a06acfe2f8ade08153f9cef66 100644
--- a/third_party/WebKit/Source/core/frame/FrameConsole.cpp
+++ b/third_party/WebKit/Source/core/frame/FrameConsole.cpp
@@ -69,19 +69,24 @@ FrameConsole::FrameConsole(LocalFrame& frame)
void FrameConsole::addMessage(ConsoleMessage* consoleMessage)
{
- 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;
+ if (addMessageToStorage(consoleMessage))
+ reportMessageToClient(consoleMessage);
+}
+bool FrameConsole::addMessageToStorage(ConsoleMessage* consoleMessage)
+{
+ if (muteCount && consoleMessage->source() != ConsoleAPIMessageSource)
+ return false;
+ if (!m_frame->document() || !messageStorage())
+ return false;
messageStorage()->reportMessage(m_frame->document(), consoleMessage);
+ return true;
+}
+void FrameConsole::reportMessageToClient(ConsoleMessage* consoleMessage)
+{
+ if (muteCount && consoleMessage->source() != ConsoleAPIMessageSource)
+ return;
if (consoleMessage->source() == NetworkMessageSource)
return;
@@ -101,6 +106,16 @@ void FrameConsole::addMessage(ConsoleMessage* consoleMessage)
frame().chromeClient().addMessageToConsole(m_frame, consoleMessage->source(), consoleMessage->level(), consoleMessage->message(), consoleMessage->lineNumber(), consoleMessage->url(), stackTrace);
}
+void FrameConsole::reportWorkerMessage(ConsoleMessage* consoleMessage)
+{
+ reportMessageToClient(consoleMessage);
+}
+
+void FrameConsole::adoptWorkerMessage(ConsoleMessage* consoleMessage)
+{
+ addMessageToStorage(consoleMessage);
+}
+
void FrameConsole::reportResourceResponseReceived(DocumentLoader* loader, unsigned long requestIdentifier, const ResourceResponse& response)
{
if (!loader)
@@ -140,13 +155,6 @@ void FrameConsole::clearMessages()
storage->clear(m_frame->document());
}
-void FrameConsole::adoptWorkerMessagesAfterTermination(WorkerInspectorProxy* proxy)
-{
- ConsoleMessageStorage* storage = messageStorage();
- if (storage)
- storage->adoptWorkerMessagesAfterTermination(proxy);
-}
-
void FrameConsole::didFailLoading(unsigned long requestIdentifier, const ResourceError& error)
{
if (error.isCancellation()) // Report failures only.
« no previous file with comments | « third_party/WebKit/Source/core/frame/FrameConsole.h ('k') | third_party/WebKit/Source/core/inspector/ConsoleMessage.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698