Closed Bug 1311506 Opened 5 years ago Closed 5 years ago
The old debugger has a (disabled by default) promise debugger feature. (It is sufficiently disabled that even after using old debugger and enabling "devtools.debugger.promise", it was still hidden AFAICT.) Since this would need to be rewritten for the new debugger anyway, we may want to just remove it at the stage. On IRC, :gl suggested removing may be reasonable.
:gl, :jlongster, thoughts?
Let's go ahead with this since there aren't any current resources allocated to working on the promise debugger. We can easily re-add the boilerplate code, but I suspect we will also do a bit of a rewrite to utilize react/redux.
Sounds fine to me. I'm not really familiar with it or how much code it is, but if would help to remove it let's do it.
5 years ago
Assignee: nobody → jryans
Status: NEW → ASSIGNED
Priority: -- → P1
Comment on attachment 8804896 [details] Bug 1311506 - Remove promise debugger. https://reviewboard.mozilla.org/r/88722/#review88018
Attachment #8804896 - Flags: review?(gl) → review+
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/autoland/rev/3d6e90d77c8d Remove promise debugger. r=gl
You need to log in before you can comment on or make changes to this bug.