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

Unified Diff: third_party/WebKit/Source/platform/v8_inspector/V8Console.cpp

Issue 2037593004: [DevTools] Remove unnecessary knowledge about console from core. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: one more test 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
« no previous file with comments | « third_party/WebKit/Source/core/inspector/ScriptArguments.cpp ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/WebKit/Source/platform/v8_inspector/V8Console.cpp
diff --git a/third_party/WebKit/Source/platform/v8_inspector/V8Console.cpp b/third_party/WebKit/Source/platform/v8_inspector/V8Console.cpp
index b38063734e78832b95906168dc621f247e912d35..10ca0f6487db1efc963df8a9741c6495cf7712d1 100644
--- a/third_party/WebKit/Source/platform/v8_inspector/V8Console.cpp
+++ b/third_party/WebKit/Source/platform/v8_inspector/V8Console.cpp
@@ -76,12 +76,12 @@ public:
return m_debuggerClient;
}
- void addMessage(MessageType type, MessageLevel level, bool allowEmptyArguments, int skipArgumentCount)
+ void addMessage(MessageType type, MessageLevel level, String16 emptyText, int skipArgumentCount)
{
- if (!allowEmptyArguments && !m_info.Length())
+ if (emptyText.isEmpty() && !m_info.Length())
return;
if (V8DebuggerClient* debuggerClient = ensureDebuggerClient())
- debuggerClient->reportMessageToConsole(m_context, type, level, String16(), &m_info, skipArgumentCount);
+ debuggerClient->reportMessageToConsole(m_context, type, level, m_info.Length() <= skipArgumentCount ? emptyText : String16(), &m_info, skipArgumentCount);
}
void addMessage(MessageType type, MessageLevel level, const String16& message)
@@ -266,67 +266,67 @@ void createBoundFunctionProperty(v8::Local<v8::Context> context, v8::Local<v8::O
void V8Console::debugCallback(const v8::FunctionCallbackInfo<v8::Value>& info)
{
- ConsoleHelper(info).addMessage(LogMessageType, DebugMessageLevel, false, 0);
+ ConsoleHelper(info).addMessage(LogMessageType, DebugMessageLevel, String16(), 0);
}
void V8Console::errorCallback(const v8::FunctionCallbackInfo<v8::Value>& info)
{
- ConsoleHelper(info).addMessage(LogMessageType, ErrorMessageLevel, false, 0);
+ ConsoleHelper(info).addMessage(LogMessageType, ErrorMessageLevel, String16(), 0);
}
void V8Console::infoCallback(const v8::FunctionCallbackInfo<v8::Value>& info)
{
- ConsoleHelper(info).addMessage(LogMessageType, InfoMessageLevel, false, 0);
+ ConsoleHelper(info).addMessage(LogMessageType, InfoMessageLevel, String16(), 0);
}
void V8Console::logCallback(const v8::FunctionCallbackInfo<v8::Value>& info)
{
- ConsoleHelper(info).addMessage(LogMessageType, LogMessageLevel, false, 0);
+ ConsoleHelper(info).addMessage(LogMessageType, LogMessageLevel, String16(), 0);
}
void V8Console::warnCallback(const v8::FunctionCallbackInfo<v8::Value>& info)
{
- ConsoleHelper(info).addMessage(LogMessageType, WarningMessageLevel, false, 0);
+ ConsoleHelper(info).addMessage(LogMessageType, WarningMessageLevel, String16(), 0);
}
void V8Console::dirCallback(const v8::FunctionCallbackInfo<v8::Value>& info)
{
- ConsoleHelper(info).addMessage(DirMessageType, LogMessageLevel, false, 0);
+ ConsoleHelper(info).addMessage(DirMessageType, LogMessageLevel, String16(), 0);
}
void V8Console::dirxmlCallback(const v8::FunctionCallbackInfo<v8::Value>& info)
{
- ConsoleHelper(info).addMessage(DirXMLMessageType, LogMessageLevel, false, 0);
+ ConsoleHelper(info).addMessage(DirXMLMessageType, LogMessageLevel, String16(), 0);
}
void V8Console::tableCallback(const v8::FunctionCallbackInfo<v8::Value>& info)
{
- ConsoleHelper(info).addMessage(TableMessageType, LogMessageLevel, false, 0);
+ ConsoleHelper(info).addMessage(TableMessageType, LogMessageLevel, String16(), 0);
}
void V8Console::traceCallback(const v8::FunctionCallbackInfo<v8::Value>& info)
{
- ConsoleHelper(info).addMessage(TraceMessageType, LogMessageLevel, true, 0);
+ ConsoleHelper(info).addMessage(TraceMessageType, LogMessageLevel, String16("console.trace"), 0);
}
void V8Console::groupCallback(const v8::FunctionCallbackInfo<v8::Value>& info)
{
- ConsoleHelper(info).addMessage(StartGroupMessageType, LogMessageLevel, true, 0);
+ ConsoleHelper(info).addMessage(StartGroupMessageType, LogMessageLevel, String16("console.group"), 0);
}
void V8Console::groupCollapsedCallback(const v8::FunctionCallbackInfo<v8::Value>& info)
{
- ConsoleHelper(info).addMessage(StartGroupCollapsedMessageType, LogMessageLevel, true, 0);
+ ConsoleHelper(info).addMessage(StartGroupCollapsedMessageType, LogMessageLevel, String16("console.groupCollapsed"), 0);
}
void V8Console::groupEndCallback(const v8::FunctionCallbackInfo<v8::Value>& info)
{
- ConsoleHelper(info).addMessage(EndGroupMessageType, LogMessageLevel, true, 0);
+ ConsoleHelper(info).addMessage(EndGroupMessageType, LogMessageLevel, String16("console.groupEnd"), 0);
}
void V8Console::clearCallback(const v8::FunctionCallbackInfo<v8::Value>& info)
{
- ConsoleHelper(info).addMessage(ClearMessageType, LogMessageLevel, true, 0);
+ ConsoleHelper(info).addMessage(ClearMessageType, LogMessageLevel, String16("console.clear"), 0);
}
void V8Console::countCallback(const v8::FunctionCallbackInfo<v8::Value>& info)
@@ -356,7 +356,7 @@ void V8Console::assertCallback(const v8::FunctionCallbackInfo<v8::Value>& info)
ConsoleHelper helper(info);
if (helper.firstArgToBoolean(false))
return;
- helper.addMessage(AssertMessageType, ErrorMessageLevel, true, 1);
+ helper.addMessage(AssertMessageType, ErrorMessageLevel, String16("console.assert"), 1);
if (V8DebuggerAgentImpl* debuggerAgent = helper.debuggerAgent())
debuggerAgent->breakProgramOnException(protocol::Debugger::Paused::ReasonEnum::Assert, nullptr);
}
« no previous file with comments | « third_party/WebKit/Source/core/inspector/ScriptArguments.cpp ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698