Nicolas Chevobbe [:nchevobbe] Assignee
	Description • 6 years ago

In Bug 1390768, we are probably going to end up creating a new component that will wrap debugger's Frames component.
This component will be used by the console and other panels (netmonitor, …), but it would be also good to allow Reps that display a trace (i.e. Errors) to have access to this component.

The idea would be to pass a `renderStacktrace` function that would render the stacktrace with the new component.

This way, all the stacktraces displayed in devtools would be the same (and handle sourcemap).

	Nicolas Chevobbe [:nchevobbe] Assignee
	Comment 1 • 6 years ago

This also depends on https://github.com/devtools-html/debugger.html/issues/7130

	Nicolas Chevobbe [:nchevobbe] Assignee
	Comment 2 • 6 years ago

Removing dependency on Bug 1390768 since we can already use the Stacktrace component in renderStacktrace.
Still need the next version of reps though.

	Nicolas Chevobbe [:nchevobbe] Assignee
	Comment 3 • 6 years ago

This patch makes use of the new renderStacktrace
prop in Reps to render Error objects stacktraces
using the shared Stacktrace component.
This way we are showing stacktraces in a consistent
way, and as a bonus, we support sourcemap in Error
object.

Depends on D9922

	Pulsebot
	Comment 4 • 6 years ago

Pushed by nchevobbe@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/006509a8370f
Use the Stacktrace component in Error reps; r=bgrins.

	Arthur Iakab [arthur_iakab]
	Comment 5 • 6 years ago
bugherder

https://hg.mozilla.org/mozilla-central/rev/006509a8370f