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

Unified Diff: third_party/WebKit/Source/core/inspector/ConsoleMessage.cpp

Issue 2035653006: [DevTools] Move Console to v8 inspector. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: all tests pass Created 4 years, 6 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/inspector/ConsoleMessage.cpp
diff --git a/third_party/WebKit/Source/core/inspector/ConsoleMessage.cpp b/third_party/WebKit/Source/core/inspector/ConsoleMessage.cpp
index 4bb74a6c33a75eed685c52e6c35e5f6cbfefc3ea..bc0640dd7efe452ae001ae7116f297c8e2576acf 100644
--- a/third_party/WebKit/Source/core/inspector/ConsoleMessage.cpp
+++ b/third_party/WebKit/Source/core/inspector/ConsoleMessage.cpp
@@ -4,25 +4,12 @@
#include "core/inspector/ConsoleMessage.h"
-#include "bindings/core/v8/ScriptValue.h"
#include "bindings/core/v8/SourceLocation.h"
-#include "core/inspector/ScriptArguments.h"
#include "wtf/CurrentTime.h"
#include "wtf/PassOwnPtr.h"
namespace blink {
-unsigned nextMessageId()
-{
- struct MessageId {
- MessageId() : value(0) { }
- unsigned value;
- };
-
- DEFINE_THREAD_SAFE_STATIC_LOCAL(WTF::ThreadSpecific<MessageId>, messageId, new WTF::ThreadSpecific<MessageId>);
- return ++messageId->value;
-}
-
// static
ConsoleMessage* ConsoleMessage::createForRequest(MessageSource source, MessageLevel level, const String& message, const String& url, unsigned long requestIdentifier)
{
@@ -32,17 +19,9 @@ ConsoleMessage* ConsoleMessage::createForRequest(MessageSource source, MessageLe
}
// static
-ConsoleMessage* ConsoleMessage::createForConsoleAPI(MessageLevel level, MessageType type, const String& message, ScriptArguments* arguments)
-{
- ConsoleMessage* consoleMessage = ConsoleMessage::create(ConsoleAPIMessageSource, level, message, SourceLocation::capture(), arguments);
- consoleMessage->m_type = type;
- return consoleMessage;
-}
-
-// static
-ConsoleMessage* ConsoleMessage::create(MessageSource source, MessageLevel level, const String& message, PassOwnPtr<SourceLocation> location, ScriptArguments* arguments)
+ConsoleMessage* ConsoleMessage::create(MessageSource source, MessageLevel level, const String& message, PassOwnPtr<SourceLocation> location)
{
- return new ConsoleMessage(source, level, message, std::move(location), arguments);
+ return new ConsoleMessage(source, level, message, std::move(location));
}
// static
@@ -54,18 +33,13 @@ ConsoleMessage* ConsoleMessage::create(MessageSource source, MessageLevel level,
ConsoleMessage::ConsoleMessage(MessageSource source,
MessageLevel level,
const String& message,
- PassOwnPtr<SourceLocation> location,
- ScriptArguments* arguments)
+ PassOwnPtr<SourceLocation> location)
: m_source(source)
, m_level(level)
- , m_type(LogMessageType)
, m_message(message)
, m_location(std::move(location))
- , m_scriptArguments(arguments)
, m_requestIdentifier(0)
, m_timestamp(WTF::currentTime())
- , m_messageId(0)
- , m_relatedMessageId(0)
{
}
@@ -73,21 +47,11 @@ ConsoleMessage::~ConsoleMessage()
{
}
-MessageType ConsoleMessage::type() const
-{
- return m_type;
-}
-
SourceLocation* ConsoleMessage::location() const
{
return m_location.get();
}
-ScriptArguments* ConsoleMessage::scriptArguments() const
-{
- return m_scriptArguments;
-}
-
unsigned long ConsoleMessage::requestIdentifier() const
{
return m_requestIdentifier;
@@ -98,13 +62,6 @@ double ConsoleMessage::timestamp() const
return m_timestamp;
}
-unsigned ConsoleMessage::assignMessageId()
-{
- if (!m_messageId)
- m_messageId = nextMessageId();
- return m_messageId;
-}
-
MessageSource ConsoleMessage::source() const
{
return m_source;
@@ -120,27 +77,8 @@ const String& ConsoleMessage::message() const
return m_message;
}
-void ConsoleMessage::frameWindowDiscarded(LocalDOMWindow* window)
-{
- if (!m_scriptArguments)
- return;
- if (m_scriptArguments->getScriptState()->domWindow() != window)
- return;
- if (!m_message)
- m_message = "<message collected>";
- m_scriptArguments.clear();
-}
-
-unsigned ConsoleMessage::argumentCount()
-{
- if (m_scriptArguments)
- return m_scriptArguments->argumentCount();
- return 0;
-}
-
DEFINE_TRACE(ConsoleMessage)
{
- visitor->trace(m_scriptArguments);
}
} // namespace blink

Powered by Google App Engine
This is Rietveld 408576698