Closed Bug 1616379 Opened 5 months ago Closed 2 months ago

MOZ_ASSERT_UNREACHABLE: invalid hyphenation resource? in nsHyphenator.cpp when loading addons.mozilla.org

Categories

(Core :: Layout: Text and Fonts, defect, P3)

defect

Tracking

()

RESOLVED FIXED
mozilla78
Tracking Status
firefox78 --- fixed

People

(Reporter: mattwoodrow, Assigned: jfkthame)

References

Details

Attachments

(1 file)

Huh, I wonder what hyphenation it's even trying to load. Will look into it...

Priority: -- → P3

is there any updates on this?

It makes the use of debug build close to impossible as we always hit this assertion sooner or later.

At least disable the crash assertion and make that a NS_ASSERTION instead.

Flags: needinfo?(jfkthame)

I've tried to reproduce this (loading URL from comment 0 with a debug build), but have not seen the problem. Do you have any other STR?

Flags: needinfo?(jfkthame)

Anyhow, I guess we can safely change this to NS_ASSERTION, as although this is unexpected -- it seems to suggest either something is broken/mispackaged in the build, or maybe we're out of memory? -- it should still be safe.

Assignee: nobody → jfkthame
Status: NEW → ASSIGNED
Pushed by jkew@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/740634c137bb
Weaken the MOZ_ASSERT on failing to load a hyphenation resource to an NS_ASSERTION. r=jya
Status: ASSIGNED → RESOLVED
Closed: 2 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla78

Matt, Jean-Yves: do you recall, were you both using Windows when you ran into this, by any chance?

Flags: needinfo?(matt.woodrow)
Flags: needinfo?(jyavenard)

Yeah, I was using Windows at the time.

Flags: needinfo?(matt.woodrow)

(In reply to Jonathan Kew (:jfkthame) from comment #8)

Matt, Jean-Yves: do you recall, were you both using Windows when you ran into this, by any chance?

I was indeed.

Flags: needinfo?(jyavenard)
See Also: → 1650013

OK, thanks -- that was the important detail I didn't realize. Filed bug 1650013 to actually fix this.

You need to log in before you can comment on or make changes to this bug.