Closed Bug 1548409 Opened 2 years ago Closed 2 years ago

Change MOZ_RELEASE_ASSERT into MOZ_DIAGNOSTIC_ASSERT

Categories

(Core :: MFBT, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla68
Tracking Status
firefox-esr60 --- unaffected
firefox67 --- unaffected
firefox68 --- fixed

People

(Reporter: jya, Assigned: jya)

References

Details

(Keywords: regression)

Attachments

(1 file)

The amount of weird content that can cause overflow when calculating time stamps is greater than expected.

None of those are a problem in themselves, and yet will cause firefox to crash.

In the review of bug 1533777, it was stated that MOZ_DIAGNOSTIC_ASSERT should be used.

I suggest we do that until we're actually ready and have covered all the required changes.

Pushed by jyavenard@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/d3215b8359d3
Use MOZ_DIAGNOSTIC_ASSERT. r=froydnj
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla68
Assignee: nobody → jyavenard
You need to log in before you can comment on or make changes to this bug.