Closed Bug 1176981 Opened 4 years ago Closed 4 years ago

kill pref'd off, never finished prototype tracing debugger

Categories

(DevTools :: Debugger, defect)

defect
Not set

Tracking

(firefox42 fixed)

RESOLVED FIXED
Firefox 42
Tracking Status
firefox42 --- fixed

People

(Reporter: fitzgen, Assigned: fitzgen)

Details

Attachments

(3 files)

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.
+1, or as dcamp would say 'with fire'.
Assignee: nobody → nfitzgerald
Attachment #8630085 - Flags: review?(past)
Attachment #8630086 - Flags: review?(past)
Attachment #8630087 - Flags: review?(past)
Panos, review ping.
I can steal this from panos if he's too busy.
(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 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.
Attachment #8630085 - Flags: review?(past) → review+
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
Attachment #8630086 - Flags: review?(past) → review+
Attachment #8630087 - Flags: review?(past) → review+
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.
Status: NEW → RESOLVED
Closed: 4 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → Firefox 42
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.