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

Unified Diff: Source/core/inspector/InspectorConsoleAgent.cpp

Issue 185713007: DevTools: Add timestamp support in the console (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Up for review Created 6 years, 9 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: Source/core/inspector/InspectorConsoleAgent.cpp
diff --git a/Source/core/inspector/InspectorConsoleAgent.cpp b/Source/core/inspector/InspectorConsoleAgent.cpp
index f5316b043718644a4e3844f75dfe942af5b8252b..dc2546148cc22f7cd9bd8cee5a5576ffad3cd30b 100644
--- a/Source/core/inspector/InspectorConsoleAgent.cpp
+++ b/Source/core/inspector/InspectorConsoleAgent.cpp
@@ -302,27 +302,14 @@ void InspectorConsoleAgent::setMonitoringXHREnabled(ErrorString*, bool enabled)
m_state->setBoolean(ConsoleAgentState::monitoringXHR, enabled);
}
-static bool isGroupMessage(MessageType type)
-{
- return type == StartGroupMessageType
- || type == StartGroupCollapsedMessageType
- || type == EndGroupMessageType;
-}
-
void InspectorConsoleAgent::addConsoleMessage(PassOwnPtr<ConsoleMessage> consoleMessage)
{
ASSERT_ARG(consoleMessage, consoleMessage);
- if (m_previousMessage && !isGroupMessage(m_previousMessage->type()) && m_previousMessage->isEqual(consoleMessage.get())) {
- m_previousMessage->incrementCount();
- if (m_frontend && m_enabled)
- m_previousMessage->updateRepeatCountInConsole(m_frontend);
- } else {
- m_previousMessage = consoleMessage.get();
- m_consoleMessages.append(consoleMessage);
- if (m_frontend && m_enabled)
- m_previousMessage->addToFrontend(m_frontend, m_injectedScriptManager, true);
- }
+ m_previousMessage = consoleMessage.get();
apavlov 2014/03/21 09:30:46 Is m_previousMessage used anywhere at all?
+ m_consoleMessages.append(consoleMessage);
+ if (m_frontend && m_enabled)
+ m_previousMessage->addToFrontend(m_frontend, m_injectedScriptManager, true);
if (!m_frontend && m_consoleMessages.size() >= maximumConsoleMessages) {
m_expiredConsoleMessageCount += expireConsoleMessagesStep;

Powered by Google App Engine
This is Rietveld 408576698