1418320 - JSON Viewer: Split HeadersToolbar, JsonToolbar and TextToolbar out to their own files

Status: RESOLVED FIXED

(DevTools :: JSON Viewer, defect, P2)

Description

[Mike Ratcliffe [:miker] [:mratcliffe] [:mikeratcliffe]]

This allows us to use createFactory calls at the top of the files and generally tidies things up.

Comment 1

[Mike Ratcliffe [:miker] [:mratcliffe] [:mikeratcliffe]]

Attachment: Bug 1418320 - JSON Viewer: Split HeadersToolbar, JsonToolbar and TextToolbar out to their own files

Review commit: https://reviewboard.mozilla.org/r/200758/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/200758/

Comment 2

[Jan Honza Odvarko [:Honza] (always need-info? me)]

Comment on attachment 8929437 [details]
Bug 1418320 - JSON Viewer: Split HeadersToolbar, JsonToolbar and TextToolbar out to their own files

https://reviewboard.mozilla.org/r/200758/#review205984

Thanks Mike!

Honza

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/df95c863d1f8
JSON Viewer: Split HeadersToolbar, JsonToolbar and TextToolbar out to their own files r=Honza

Comment 4

[Noemi Erli[:noemi_erli]]

https://hg.mozilla.org/mozilla-central/rev/df95c863d1f8