[DevTools] Changed checks in InspectorConsoleWindow::autogenerateMetadata

Source/core/inspector/InspectorConsoleMessage.cpp

Index: Source/core/inspector/InspectorConsoleMessage.cpp
diff --git a/Source/core/inspector/InspectorConsoleMessage.cpp b/Source/core/inspector/InspectorConsoleMessage.cpp
index fc496de080d664fe8fc15c49bdb514dcebfa83a7..409e5fa10a97f0d9eead5dbb0f8a81d707a828d8 100644
--- a/Source/core/inspector/InspectorConsoleMessage.cpp
+++ b/Source/core/inspector/InspectorConsoleMessage.cpp
@@ -45,7 +45,7 @@
 
 namespace blink {
 
-InspectorConsoleMessage::InspectorConsoleMessage(bool canGenerateCallStack, MessageSource source, MessageType type, MessageLevel level, const String& message)
+InspectorConsoleMessage::InspectorConsoleMessage(MessageSource source, MessageType type, MessageLevel level, const String& message)
     : m_source(source)
     , m_type(type)
     , m_level(level)
@@ -57,10 +57,10 @@ InspectorConsoleMessage::InspectorConsoleMessage(bool canGenerateCallStack, Mess
     , m_requestId(IdentifiersFactory::requestId(0))
     , m_timestamp(WTF::currentTime())
 {
-    autogenerateMetadata(canGenerateCallStack);
+    autogenerateMetadata();
 }
 
-InspectorConsoleMessage::InspectorConsoleMessage(bool canGenerateCallStack, MessageSource source, MessageType type, MessageLevel level, const String& message, const String& url, unsigned line, unsigned column, ScriptState* scriptState, unsigned long requestIdentifier)
+InspectorConsoleMessage::InspectorConsoleMessage(bool shouldGenerateCallStack, MessageSource source, MessageType type, MessageLevel level, const String& message, const String& url, unsigned line, unsigned column, ScriptState* scriptState, unsigned long requestIdentifier)
     : m_source(source)
     , m_type(type)
     , m_level(level)
@@ -72,7 +72,7 @@ InspectorConsoleMessage::InspectorConsoleMessage(bool canGenerateCallStack, Mess
     , m_requestId(IdentifiersFactory::requestId(requestIdentifier))
     , m_timestamp(WTF::currentTime())
 {
-    autogenerateMetadata(canGenerateCallStack, scriptState);
+    autogenerateMetadata(shouldGenerateCallStack);
 }
 
 InspectorConsoleMessage::InspectorConsoleMessage(MessageSource source, MessageType type, MessageLevel level, const String& message, PassRefPtrWillBeRawPtr<ScriptCallStack> callStack, unsigned long requestIdentifier)
@@ -82,21 +82,16 @@ InspectorConsoleMessage::InspectorConsoleMessage(MessageSource source, MessageTy
     , m_message(message)
     , m_scriptState(0)
     , m_arguments(nullptr)
+    , m_callStack(callStack)
     , m_line(0)
     , m_column(0)
     , m_requestId(IdentifiersFactory::requestId(requestIdentifier))
     , m_timestamp(WTF::currentTime())
 {
-    if (callStack && callStack->size()) {
-        const ScriptCallFrame& frame = callStack->at(0);
-        m_url = frame.sourceURL();
-        m_line = frame.lineNumber();
-        m_column = frame.columnNumber();
-    }
-    m_callStack = callStack;
+    autogenerateMetadata(false);
 }
 
-InspectorConsoleMessage::InspectorConsoleMessage(bool canGenerateCallStack, MessageSource source, MessageType type, MessageLevel level, const String& message, PassRefPtrWillBeRawPtr<ScriptArguments> arguments, ScriptState* scriptState, unsigned long requestIdentifier)
+InspectorConsoleMessage::InspectorConsoleMessage(MessageSource source, MessageType type, MessageLevel level, const String& message, PassRefPtrWillBeRawPtr<ScriptArguments> arguments, ScriptState* scriptState, unsigned long requestIdentifier)
     : m_source(source)
     , m_type(type)
     , m_level(level)
@@ -109,24 +104,20 @@ InspectorConsoleMessage::InspectorConsoleMessage(bool canGenerateCallStack, Mess
     , m_requestId(IdentifiersFactory::requestId(requestIdentifier))
     , m_timestamp(WTF::currentTime())
 {
-    autogenerateMetadata(canGenerateCallStack, scriptState);
+    autogenerateMetadata();
 }
 
 InspectorConsoleMessage::~InspectorConsoleMessage()
 {
 }
 
-void InspectorConsoleMessage::autogenerateMetadata(bool canGenerateCallStack, ScriptState* scriptState)
+void InspectorConsoleMessage::autogenerateMetadata(bool shouldGenerateCallStack)
 {
     if (m_type == EndGroupMessageType)
         return;
 
-    if (scriptState)
-        m_callStack = createScriptCallStackForConsole();
-    else if (canGenerateCallStack)
-        m_callStack = createScriptCallStack(ScriptCallStack::maxCallStackSizeToCapture, true);
-    else
-        return;
+    if (shouldGenerateCallStack)
+        m_callStack = createScriptCallStackForConsole(ScriptCallStack::maxCallStackSizeToCapture, true);
 
     if (m_callStack && m_callStack->size()) {
         const ScriptCallFrame& frame = m_callStack->at(0);