Closed Bug 551153 Opened 10 years ago Closed 10 years ago
###!!! ASSERTION: Not a UTF-8 string when running Test
UTF and Test Atoms
Following bug 534136, debug builds are showing the following assertion when running TestUTF: ###!!! ASSERTION: Not a UTF-8 string. This code should only be used for converting from known UTF-8 strings.: 'Error', file ../../dist/include/nsUTF8Utils.h, line 100 nsCRT::moz_strdup(unsigned short const*)+0x000003EC [/Users/moztest/comm/main/tb/mozilla/dist/bin/./libxpcom_core.dylib +0x00024A68] nsCRT::HashCodeAsUTF16(char const*, unsigned int, int*)+0x00000043 [/Users/moztest/comm/main/tb/mozilla/dist/bin/./libxpcom_core.dylib +0x000243CF] From what I can tell, this is a test to check that a non-UTF8 string is handled correctly, and therefore it shouldn't fail the test. If I fix that, then TestAtoms also gets a similar assertion. Therefore I think its right that we just disable the part of the tests that is giving the assertion for debug builds (obviously release builds will still run the test).
Attachment #431345 - Flags: review?(jonas)
Am I right that there are a few tests that SeaMonkey runs but Firefox doesn't due to being libxul? Is there a bug filed on that? It doesn't make too much sense if we have core tests that aren't run by Firefox...
Comment on attachment 431345 [details] [diff] [review] The fix I'm actually fixing most of this in bug 551129 already. Though had missed that the hashas16 asserts. However we should only disable the part of the test that uses the Malformed8Strings array. Mind disabling just that in this patch?
Attachment #431345 - Flags: review?(jonas) → review-
Per comment 2. NB: I assume Mark won't mind if I take over.
Attachment #431654 - Flags: review?(jonas) → review+
Comment on attachment 431654 [details] [diff] [review] (Bv1) Make it fully explicit that Malformed8Strings is not to be used in debug builds [Checkin: Comment 4] http://hg.mozilla.org/mozilla-central/rev/1912459c016b
Attachment #431654 - Attachment description: (Bv1) Make it fully explicit that Malformed8Strings is not to be used in debug builds → (Bv1) Make it fully explicit that Malformed8Strings is not to be used in debug builds [Checkin: Comment 4]
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.