962658 - test_badMimeType.html | Assertion count 6 is greater than expected range 0-0 assertions. on Android 4.0 Debug

Status: RESOLVED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

[Geoff Brown [:gbrown]]

Android 4.0 *Debug* tests currently only run on the Cedar tree. We are trying to get these tests to pass so that they can be run on trunk trees.

On Cedar, on Android 4.0 Debug, mochitest-8 tests consistently fail with:

08:46:18     INFO -  145834 INFO TEST-START | /tests/uriloader/exthandler/tests/mochitest/test_badMimeType.html
08:46:18     INFO -  145835 INFO TEST-PASS | /tests/uriloader/exthandler/tests/mochitest/test_badMimeType.html | badMimeType has changed, making this test invalid
08:46:18     INFO -  145836 INFO TEST-PASS | /tests/uriloader/exthandler/tests/mochitest/test_badMimeType.html | The test shouldn't trigger a crash
08:46:18     INFO -  145837 INFO TEST-INFO | MEMORY STAT vsize after test: 844853248
08:46:18     INFO -  145838 INFO TEST-INFO | MEMORY STAT residentFast after test: 247242752
08:46:18     INFO -  145839 INFO TEST-INFO | MEMORY STAT heapAllocated after test: 59699156
08:46:18     INFO -  145840 INFO TEST-END | /tests/uriloader/exthandler/tests/mochitest/test_badMimeType.html | finished in 347ms
08:46:18     INFO -  145841 ERROR TEST-UNEXPECTED-FAIL | /tests/uriloader/exthandler/tests/mochitest/test_badMimeType.html | Assertion count 6 is greater than expected range 0-0 assertions.

Unfortunately, not enough logcat is reported to see the failed assertions.

One approach to resolving this would be to reproduce the failure on a local device with a debug build while observing logcat.

There are on-going efforts to report complete logcats on tbpl, but I am not sure how long it will be before a working solution is available.

Comment 1

[Geoff Brown [:gbrown]]

Example: https://tbpl.mozilla.org/php/getParsedLog.php?id=33336843&tree=Cedar&full=1

See https://tbpl.mozilla.org/?tree=Cedar&showall=1&jobname=android.*debug for more.

Comment 2

[Steven Michaud [:smichaud] (Retired)]

Why am I CCed on this bug? :-)

Comment 3

[Geoff Brown [:gbrown]]

(In reply to Steven Michaud from comment #2)
> Why am I CCed on this bug? :-)

In case you have insight into the failure (you are the author of the test, right?). Feel free to drop off if not interested.

Comment 4

[Steven Michaud [:smichaud] (Retired)]

Sorry, I'd completely forgotten about it :-(

I'd guess that the "badMimeType has changed, making this test invalid" error happens because something is eating the 0xFF character at the end of "text/plain".  I have no idea what, though.

Frankly I think you should just disable this test on Android :-)

Comment 5

[Geoff Brown [:gbrown]]

http://mozilla-releng-blobs.s3.amazonaws.com/blobs/cedar/sha512/c51530b7b6c73c5f9b1c01cfe26e88a974d4f4e877d1ba04af32aafd433fe32804cc452b2f9545708d6109037872799c540704fca0610b45da2d16dec7bd1dd8

02-12 08:46:22.781 I/GeckoDump( 2229): 3653 INFO TEST-START | /tests/uriloader/exthandler/tests/mochitest/test_badMimeType.html
02-12 08:46:22.843 I/Gecko   ( 2229): ++DOMWINDOW == 68 (0x68fb3b90) [pid = 2229] [serial = 1575] [outer = 0x66420020]
02-12 08:46:22.945 I/Gecko   ( 2229): [2229] ###!!! 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 421
02-12 08:46:22.953 I/Gecko   ( 2229): [2229] ###!!! 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 68
02-12 08:46:22.953 I/Gecko   ( 2229): [2229] ###!!! ASSERTION: Input wasn't UTF8 or incorrect length was calculated: 'Error', file /builds/slave/ced-and-d-00000000000000000000/build/xpcom/string/src/nsReadableUtils.cpp, line 222
02-12 08:46:22.953 I/Gecko   ( 2229): AndroidBridge::GetHandlersForMimeType
02-12 08:46:22.960 I/Gecko   ( 2229): AndroidBridge::GetExtensionFromMimeType
02-12 08:46:22.960 I/Gecko   ( 2229): [2229] ###!!! 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 421
02-12 08:46:22.960 I/Gecko   ( 2229): [2229] ###!!! 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 68
02-12 08:46:22.960 I/Gecko   ( 2229): [2229] ###!!! ASSERTION: Input wasn't UTF8 or incorrect length was calculated: 'Error', file /builds/slave/ced-and-d-00000000000000000000/build/xpcom/string/src/nsReadableUtils.cpp, line 222
02-12 08:46:22.960 I/Gecko   ( 2229): AndroidBridge::GetMimeTypeFromExtensions
02-12 08:46:23.015 I/GeckoDump( 2229): 3654 INFO TEST-INFO | MEMORY STAT vsize after test: 844910592
02-12 08:46:23.289 I/GeckoDump( 2229): 3655 INFO TEST-INFO | MEMORY STAT residentFast after test: 254291968
02-12 08:46:23.304 I/GeckoDump( 2229): 3656 INFO TEST-INFO | MEMORY STAT heapAllocated after test: 58353236
02-12 08:46:23.320 I/Gecko   ( 2229): nsWindow::SetFocus: can't set focus without raising, ignoring aRaise = false!
02-12 08:46:24.273 I/GeckoDump( 2229): 3657 INFO TEST-END | /tests/uriloader/exthandler/tests/mochitest/test_badMimeType.html | finished in 1486ms

Comment 6

[Geoff Brown [:gbrown]]

https://tbpl.mozilla.org/php/getParsedLog.php?id=35889593&tree=Cedar&full=1

Comment 7

[Geoff Brown [:gbrown]]

https://tbpl.mozilla.org/php/getParsedLog.php?id=37432190&tree=Cedar&full=1

03:50:40     INFO -  1701 INFO TEST-START | /tests/uriloader/exthandler/tests/mochitest/test_badMimeType.html
03:50:40     INFO -  1702 INFO TEST-INFO | MEMORY STAT vsize after test: 894562304
03:50:40     INFO -  1703 INFO TEST-INFO | MEMORY STAT residentFast after test: 301326336
03:50:40     INFO -  1704 INFO TEST-INFO | MEMORY STAT heapAllocated after test: 57963932
03:50:40     INFO -  1705 INFO TEST-END | /tests/uriloader/exthandler/tests/mochitest/test_badMimeType.html | finished in 440ms
03:50:40     INFO -  1706 ERROR TEST-UNEXPECTED-FAIL | /tests/uriloader/exthandler/tests/mochitest/test_badMimeType.html | Assertion count 4 is greater than expected range 0-0 assertions.
03:50:40     INFO -  1707 INFO TEST-START | /tests/uriloader/exthandler/tests/mochitest/test_unsafeBidiChars.xhtml

Comment 8

[Geoff Brown [:gbrown]]

:blassey -- There is a danger that chunking changes (enabling/disabling tests) would move this failure into M7, which runs on m-c, causing great confusion and distress. We must get this resolved soon, or turn off all Android 4.0 Debug mochitests.

Comment 9

[Geoff Brown [:gbrown]]

Latest logcat (Comment 7):

04-08 03:50:05.804 I/GeckoDump( 2235): 1701 INFO TEST-START | /tests/uriloader/exthandler/tests/mochitest/test_badMimeType.html^M
04-08 03:50:05.843 I/Gecko   ( 2235): ++DOMWINDOW == 64 (0x6ae1e5c0) [pid = 2235] [serial = 1519] [outer = 0x69a578e0]^M
04-08 03:50:05.968 I/Gecko   ( 2235): [2235] ###!!! 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 421^M
04-08 03:50:05.968 I/Gecko   ( 2235): [2235] ###!!! ASSERTION: Input wasn't UTF8 or incorrect length was calculated: 'Error', file /builds/slave/ced-and-d-00000000000000000000/build/xpcom/string/src/nsReadableUtils.cpp, line 222^M
04-08 03:50:05.968 I/Gecko   ( 2235): AndroidBridge::GetHandlersForMimeType^M
04-08 03:50:05.976 I/Gecko   ( 2235): AndroidBridge::GetExtensionFromMimeType^M
04-08 03:50:05.976 I/Gecko   ( 2235): [2235] ###!!! 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 421^M
04-08 03:50:05.976 I/Gecko   ( 2235): [2235] ###!!! ASSERTION: Input wasn't UTF8 or incorrect length was calculated: 'Error', file /builds/slave/ced-and-d-00000000000000000000/build/xpcom/string/src/nsReadableUtils.cpp, line 222^M
04-08 03:50:05.976 I/Gecko   ( 2235): AndroidBridge::GetMimeTypeFromExtensions^M
04-08 03:50:05.984 E/GeckoConsole( 2235): [JavaScript Warning: "The character encoding of a framed document was not declared. The document may appear different if viewed without the document framing it." {file: "http://mochi.test:8888/tests/uriloader/exthandler/tests/mochitest/test_badMimeType.html" line: 0}]^M
04-08 03:50:06.007 D/dalvikvm( 1896): GC_CONCURRENT freed 384K, 15% free 6695K/7815K, paused 0ms+5ms^M
04-08 03:50:06.007 I/GeckoDump( 2235): 1702 INFO TEST-INFO | MEMORY STAT vsize after test: 894562304^M
04-08 03:50:06.015 I/GeckoDump( 2235): 1703 INFO TEST-INFO | MEMORY STAT residentFast after test: 301326336^M
04-08 03:50:06.015 I/GeckoDump( 2235): 1704 INFO TEST-INFO | MEMORY STAT heapAllocated after test: 57963932^M
04-08 03:50:06.023 I/Gecko   ( 2235): nsWindow::SetFocus: can't set focus without raising, ignoring aRaise = false!^M
04-08 03:50:06.250 I/GeckoDump( 2235): 1705 INFO TEST-END | /tests/uriloader/exthandler/tests/mochitest/test_badMimeType.html | finished in 440ms^M

Comment 10

[Brad Lassey [:blassey] (use needinfo?)]

Attachment: check_UTF8.patch

Comment 11

[Geoff Brown [:gbrown]]

Comment on attachment 8405164 [details] [diff] [review]
check_UTF8.patch

Review of attachment 8405164 [details] [diff] [review]:
-----------------------------------------------------------------

Thanks! With this patch, we may be able to start running all mochitests on Debug.

Comment 12

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/9ecb37a99e2c