Summary: Paul → Paul's website hangs Nightly
Keywords: regression, regressionwindow-wanted
Hey Alice, any chance you could see if you can narrow this down please? Thanks a lot!
(In reply to :Ehsan Akhgari (needinfo? me!) from comment #2) > Hey Alice, any chance you could see if you can narrow this down please? > Thanks a lot! I cannot reproduce the problem on Windows7 and Ubuntu. And I cannot test this on Mac because I do not have Mac.
Thanks, Alice! Jeff, can you please see if you can make this bug actionable?
It looks like we are stuck in gfxFont::Draw() calling GlyphBufferAzure::Flush() for ever.
Component: Graphics → Layout: Text
Simplified testcase: http://people.mozilla.org/~jdaggett/tests/syntheticbold-rotated.html Two problems here: the matrix code in gfxContext::UserToDevice(const gfxSize& size) is wrong (yikes!) and the synthetic bolding code isn't handling the singular matrix case, so it ends up with stamps = huge number and this causes an infinite loop while drawing. Scary part is this bug has been there since Bas added it in early 2012.
Summary: Paul's website hangs Nightly → rotated text with synthetic bolding causes infinite loop in text drawing
Created attachment 795895 [details] [diff] [review] patch, fix matrix calc bug Fix the matrix math error and check the result of CalcXScale.
Attachment #795895 - Flags: review?(jmuizelaar)
Created attachment 795897 [details] [diff] [review] patch, add reftest for the rotated synthetic bold case Reftest based on simplified testcase. This will cause the infinite loop in current trunk code and render correctly with patch.
Attachment #795897 - Flags: review?(jmuizelaar)
Attachment #795895 - Flags: review?(jmuizelaar) → review+
Attachment #795897 - Flags: review?(jmuizelaar) → review+
Looks like this is the same issue Jesse filed in bug 895051. I'll dupe that report to this one, given that the fix is being done here.
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla26
Issue is resolved - clearing old keywords - qa-wanted clean-up
You need to log in before you can comment on or make changes to this bug.