1176981 - kill pref'd off, never finished prototype tracing debugger

Status: RESOLVED FIXED

(DevTools :: Debugger, defect)

Description

[Nick Fitzgerald [:fitzgen] [⏰PST; UTC-8]]

Using the tracelogger would mean re-writing the actor and server side.

Having the UI be performant would mean re-writing it.

We have vcs if we ever want to look again.

Let's remove this stuff from the tree.

Comment 1

[Jeff Griffiths (:canuckistani) (:⚡︎)]

+1, or as dcamp would say 'with fire'.

Comment 2

[Nick Fitzgerald [:fitzgen] [⏰PST; UTC-8]]

Attachment: Part 0: Remove the tracing debugger UI

Comment 3

[Nick Fitzgerald [:fitzgen] [⏰PST; UTC-8]]

Attachment: Part 1: Remove the tracing debugger actor from the server

Comment 4

[Nick Fitzgerald [:fitzgen] [⏰PST; UTC-8]]

Attachment: Part 2: Remove the tracing debugger pref

Comment 5

[Nick Fitzgerald [:fitzgen] [⏰PST; UTC-8]]

Panos, review ping.

Comment 6

[Victor Porof [:vporof][:vp]]

I can steal this from panos if he's too busy.

Comment 7

[Nick Fitzgerald [:fitzgen] [⏰PST; UTC-8]]

(In reply to Victor Porof [:vporof][:vp] from comment #6)
> I can steal this from panos if he's too busy.

Be my guest :)

Comment 8

[Victor Porof [:vporof][:vp]]

Comment on attachment 8630085 [details] [diff] [review]
Part 0: Remove the tracing debugger UI

Review of attachment 8630085 [details] [diff] [review]:
-----------------------------------------------------------------

Straightforward removal. r+ assuming all tests still pass.

Comment 9

[Victor Porof [:vporof][:vp]]

Comment on attachment 8630086 [details] [diff] [review]
Part 1: Remove the tracing debugger actor from the server

Review of attachment 8630086 [details] [diff] [review]:
-----------------------------------------------------------------

Ditto

Comment 10

[Pulsebot]

https://hg.mozilla.org/integration/fx-team/rev/1e2cbe60bbbd
https://hg.mozilla.org/integration/fx-team/rev/365f3d257b80
https://hg.mozilla.org/integration/fx-team/rev/0023b4a6e33c

Comment 11

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/1e2cbe60bbbd
https://hg.mozilla.org/mozilla-central/rev/365f3d257b80
https://hg.mozilla.org/mozilla-central/rev/0023b4a6e33c

Rev 0023b4a6e33c was an empty commit, FWIW.

Comment 12

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/d62b1d5ba4b6

Comment 13

[Ryan VanderMeulen [:RyanVM]]

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