Web Inspector: console.trace() should include passed argument values

[Nikita Vasilyev]

See: http://nv.github.io/captureCalls.js/


Expected:

console.trace()
  ► lookupKey(key="Tab", map=Object, handle=function) – codemirror.js
  ...

Same, but expanded:

console.trace()
  ▼ lookupKey(key="Tab", map=Object, handle=function) – codemirror.js
   · "Tab"
   · ► {Up: function, Down: function, Right: function, Esc: function, Enter: function, …}
   · function (b) { return doHandleBinding(cm, b); }


Actual:

console.trace()
  lookupKey – codemirror.js
  ...


Notes:

Currently, we use WebInspector.StackTraceView to display console.trace, and caught and uncaught errors. An example of an caught error:

    try {
      i.dont.exist++;
    } catch (e) {
      console.log(e);
    }

For this case, we can't display passed arguments values because "e" doesn't have them.

I think, we should be able to get passed arguments values for uncaught errors, e.g:

    i.dont.exist++;

and console.trace().

[Radar WebKit Bug Importer]

<rdar://problem/22451487>

[Timothy Hatcher]

<rdar://problem/8082776>

[Joseph Pecoraro]

Clearing assignee, no action in 30 days. I think and open source contributor is interested in this.

[Nikita Vasilyev]

Good to hear. I can provide guidance since I implemented the last version of the stack trace view.

[Nikita Vasilyev]

Any ideas where/how can we get argument values? WebInspector.ConsoleMessage doesn't have this data.

[João Oliveira]

Created attachment 264812 [details]
patch