Created attachment 278484 [details] testcase ###!!! ASSERTION: Invalid offset: 'aOffset <= mSkipChars->mCharCount', file /Users/jruderman/trunk/mozilla/gfx/thebes/src/gfxSkipChars.cpp, line 92 ###!!! ASSERTION: aPos out of range: '0 <= aPos && aPos < mCharacterCount', file ../../../dist/include/thebes/gfxFont.h, line 563 Also, none of the letters gets the first-letter style.
doesn't seem like this should assert but i don't get any characters in firefox2 so i'm not sure if we should block on this. thoughts?
We probably should, but this should be in Layout anyway so it's not your problem
I'd nominate this for blocking just because the assertions sound scary. (Also, the "Invalid offset" assertion can be triggered by lots of bugs, and having this fixed will make it easier to continue looking for such bugs.)
Created attachment 278711 [details] [diff] [review] fix assertions Stupid bug here, trivial fix. This fixes the assertions, but the first-letter isn't displayed as magenta. That must be a separate bug.
I filed bug 393985 on this and other cases where the first-letter style isn't applied. (In reply to comment #1) > doesn't seem like this should assert but i don't get any characters in firefox2 > so i'm not sure if we should block on this. thoughts? That was bug 198928, FWIW
Comment on attachment 278711 [details] [diff] [review] fix assertions a1.9=dbaron
10 years ago
Crashtest checked in. Bug 393985 will eventually get a reftest.
Mass-assigning the new rtl keyword to RTL-related (see bug 349193).