1024004 - Hitting MOZ_ASSERT(!pattern->IsAzure()); in gfxContext.cpp

Status: RESOLVED FIXED

(Core :: Graphics, defect)

Description

[Adam Roach [:abr]]

Attachment: Crash Stack

I'm seeing this reliably in a relatively unused profile (the one I use for testing WebRTC), although I've been unable to isolate it to any particular aspect of that profile. It happens with 100% reproducibility immediately upon start-up with this profile. 

Stack attached. If you need me to bring this up in a debugger so you can poke around at state, let me know.

Comment 2

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Attachment: gdb output

I just hit this as well, while loading three articles in tabs from http://www.nytimes.com/ (at least 2 of which were about today's World Cup matches).

Comment 3

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

I claim (by inspection) that bug 994081 seems like the most likely cause of this.

Comment 4

[Jonathan Watt [:jwatt]]

nsSVGMaskFrame is in the attached stack, so basically we want to fix bug 987194 to fix this.

Comment 5

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

It's in abr's stack and mcmanus's but not mine.

Comment 6

[Jonathan Watt [:jwatt]]

Can you attach yours?

Comment 7

[Jonathan Watt [:jwatt]]

Oh, you already did. Didn't bug 932198 fix that crash stack?

Comment 8

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

It certainly could have, given:

$ hg log -r'ancestor(a6a457fe2a2a, 617e3d552045)'
changeset:   188731:a6a457fe2a2a

(i.e., that my crash was on a revision without bug 932198).


Is bug 987194 going to happen soon?  It seems bad to leave trunk in a state where certain combinations of Web features yield fatal assertions (and presumably worse things in non-DEBUG builds?); would it be possible to handle the relevant case instead?

Comment 9

[Jonathan Watt [:jwatt]]

It depends how quickly Matt gets to the review (he's been getting through my review request pretty quickly for other Moz2D conversion stuff). We should then get approval for aurora too.

Comment 10

[Jonathan Watt [:jwatt]]

Fixed by the patch for bug 987194.