1931008 - Expose stacktrace on console.log and console.warn

Status: RESOLVED DUPLICATE of bug 1744705

(DevTools :: Console, enhancement)

Description

[Julian Descottes [:jdescottes]]

At the moment the stacktrace is only provide for some message types:
https://searchfox.org/mozilla-central/rev/016925857e2f81a9425de9e03021dcf4251cafcc/dom/console/Console.cpp#2363-2372

bool Console::ShouldIncludeStackTrace(MethodName aMethodName) {
  switch (aMethodName) {
    case MethodError:
    case MethodException:
    case MethodAssert:
    case MethodTrace:
      return true;
    default:
      return false;
  }

The original implementation goes back to Bug 920116, where performance concerns were mentioned.

For WebDriver BiDi, and for consistency with Chrome, we would like to expose stacktrace information for all console APIs, not just error/assert/trace.

This would allow users to filter out messages from unwanted sources.

Could we consider adding stacktrace information to other messages? Do we have an idea of the performance impact of such a change?

(Note: if this is not possible, I imagine we could still expose a minimal/truncated stacktrace based on the filename, line. column attached to the message)

Comment 1

[Henrik Skupin [:whimboo][⌚️UTC+2]]

See bug 1744705 as well which is about adding the stack trace to the platform code for both the warn and log messages.

Comment 2

[Julian Descottes [:jdescottes]]

Ah thanks let's close as duplicate then.