Remove MOZ_DEV_EDITION checks in HttpChannelChild.cpp

RESOLVED FIXED in Firefox 62

Status

()

enhancement
P3
normal
RESOLVED FIXED
Last year
Last year

People

(Reporter: dholbert, Assigned: dholbert)

Tracking

(Blocks 1 bug)

Trunk
mozilla62
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox62 fixed)

Details

(Whiteboard: [necko-triaged])

Attachments

(1 attachment)

HttpChannelChild.cpp has some compile-time MOZ_DEV_EDITION checks that would be nice to remove, in service of bug 1459240.

It looks like these checks were just part of adding a diagnostic assert to investigate a crash, in bug 1322610.

Based on bug 1322610 comment 7, it sounded like maybe the diagnostic might still have some value down the line (when we get to bug 1306801 at least).

valentin, do you have any updates on the usefulness of the diagnostic?  If it's no longer going to be useful, can/should we back it out?  Or if not, would it be OK to make it Nightly-only, so that we've got one fewer compile-time  MOZ_DEV_EDITION check?
Assignee

Updated

Last year
Flags: needinfo?(valentin.gosu)
It's no longer useful, and we can remove it.
Flags: needinfo?(valentin.gosu)
Assignee

Comment 2

Last year
Great, thanks!

(Sorry to have bothered you on your PTO - this could've waited until you returned. :)  Thank you for the quick reply!)
Comment hidden (mozreview-request)

Comment 4

Last year
mozreview-review
Comment on attachment 8982315 [details]
Bug 1465884: Back out no-longer-needed diagnostic for networking crash.

https://reviewboard.mozilla.org/r/248260/#review254786
Attachment #8982315 - Flags: review?(mcmanus) → review+
Assignee

Updated

Last year
Assignee: nobody → dholbert

Comment 5

Last year
Pushed by dholbert@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/9fd9a942834c
Back out no-longer-needed diagnostic for networking crash. r=mcmanus
Priority: -- → P3
Whiteboard: [necko-triaged]

Comment 6

Last year
bugherder
https://hg.mozilla.org/mozilla-central/rev/9fd9a942834c
Status: NEW → RESOLVED
Closed: Last year
Resolution: --- → FIXED
Target Milestone: --- → mozilla62
You need to log in before you can comment on or make changes to this bug.