Closed Bug 1268050 Opened 9 years ago Closed 9 years ago

dom/xbl/crashtests/336744-1.html | assertion count 2 is more than expected 0 assertions

Categories

(Core :: XUL, defect)

Product:
Core
Component:
XUL
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla49
Tracking Flags:
Tracking Status
firefox48 --- fixed
firefox49 --- fixed

People

(Reporter: gbrown, Assigned: gbrown)

References

Details

(Keywords: intermittent-failure)

Android reftest assertion count failure detection has been faulty for several weeks - bug 1267020. In the meantime, this failure has been happening with high frequency. See https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1267020&startday=2016-04-18&endday=2016-04-24&tree=all for some failures that the sheriffs noticed. https://treeherder.mozilla.org/#/jobs?repo=try&revision=b556cb611378 shows what happens when Android assertion count mismatches are reported correctly - 336744-1.html fails about half the time.
Assertions can be found in the logcat. http://mozilla-releng-blobs.s3.amazonaws.com/blobs/try/sha512/01b4db29903afb40f39271ccb17912e6eaab636d39766912b21832367170beff133c42f70e022d234dd9eb94cd5a0318aaabd0ccac4ea130a4b0ad5ddcf919eb 04-27 06:19:26.979 1091 1109 I Gecko : {"action":"test_start","time":1461763166977,"thread":null,"pid":null,"source":"reftest","test":"http://10.0.2.2:8854/tests/dom/xbl/crashtests/336744-1.html"} 04-27 06:19:26.979 1091 1109 I Gecko : REFTEST TEST-LOAD | http://10.0.2.2:8854/tests/dom/xbl/crashtests/336744-1.html | 40 / 303 (13%) 04-27 06:19:26.999 1091 1109 I Gecko : 04-27 06:19:26.999 1091 1109 I Gecko : {"action":"log","time":1461763167001,"thread":null,"pid":null,"source":"reftest","level":"DEBUG","message":"START http://10.0.2.2:8854/tests/dom/xbl/crashtests/336744-1.html"} 04-27 06:19:27.388 1091 1109 I Gecko : [1091] WARNING: NS_ENSURE_TRUE(aURI) failed: file /builds/slave/try-and-api-15-d-0000000000000/build/src/netwerk/cache2/CacheStorage.cpp, line 63 04-27 06:19:27.458 1091 1109 I Gecko : ++DOMWINDOW == 23 (0x63177000) [pid = 1091] [serial = 91] [outer = 0x62fc0c00] 04-27 06:19:28.219 1091 1109 I Gecko : ++DOCSHELL 0x63b0f800 == 4 [pid = 1091] [id = 5] 04-27 06:19:28.229 1091 1109 I Gecko : ++DOMWINDOW == 24 (0x63f4f400) [pid = 1091] [serial = 92] [outer = 0x0] 04-27 06:19:28.239 1091 1109 E GeckoConsole: [JavaScript Error: "The character encoding of the HTML document was not declared. The document will render with garbled text in some browser configurations if the document contains characters from outside the US-ASCII range. The character encoding of the page must be declared in the document or in the transfer protocol." {file: "http://10.0.2.2:8854/tests/dom/xbl/crashtests/336744-1.html" line: 0}] 04-27 06:19:28.338 1091 1109 I Gecko : ++DOMWINDOW == 25 (0x63b11000) [pid = 1091] [serial = 93] [outer = 0x63f4f400] 04-27 06:19:29.728 1091 1109 I Gecko : ++DOCSHELL 0x63b0ec00 == 5 [pid = 1091] [id = 6] 04-27 06:19:29.748 1091 1109 I Gecko : ++DOMWINDOW == 26 (0x63b11c00) [pid = 1091] [serial = 94] [outer = 0x0] 04-27 06:19:30.468 1091 1109 I Gecko : ++DOMWINDOW == 27 (0x63ba7c00) [pid = 1091] [serial = 95] [outer = 0x63b11c00] 04-27 06:19:31.239 1091 1180 D GeckoSuggestedSites: Number of suggested sites: 8 04-27 06:19:31.889 1091 1109 I Gecko : 04-27 06:19:31.889 1091 1109 I Gecko : {"action":"log","time":1461763171891,"thread":null,"pid":null,"source":"reftest","level":"DEBUG","message":"[CONTENT] OnDocumentLoad triggering AfterOnLoadScripts"} 04-27 06:19:31.950 1091 1109 I Gecko : nsWindow[0x529d88a0]::Create 0x0 [20 58 0 0] 04-27 06:19:31.950 1091 1109 I Gecko : nsWindow[0x529d88a0]::SetZIndex 2147483647 ignored 04-27 06:19:32.839 1091 1109 I Gecko : nsWindow[0x529d88a0]::Resize [20.000000 58.000000 29.000000 19.000000] (repaint 1) 04-27 06:19:32.839 1091 1109 I Gecko : nsWindow: 0x529d88a0 OnSizeChanged [29 19] 04-27 06:19:32.839 1091 1109 I Gecko : [1091] ###!!! ASSERTION: Should only schedule view manager flush on root prescontexts: 'mPresContext->IsRoot()', file /builds/slave/try-and-api-15-d-0000000000000/build/src/layout/base/nsRefreshDriver.cpp, line 2155 04-27 06:19:32.839 1091 1109 I Gecko : nsWindow[0x529d88a0]::Show 1 04-27 06:19:32.859 1091 1109 I Gecko : [1091] ###!!! ASSERTION: Should only schedule view manager flush on root prescontexts: 'mPresContext->IsRoot()', file /builds/slave/try-and-api-15-d-0000000000000/build/src/layout/base/nsRefreshDriver.cpp, line 2155 04-27 06:19:33.129 1091 1109 I Gecko : --DOCSHELL 0x63b0ec00 == 4 [pid = 1091] [id = 6]
In searching for a regression range, I noticed that the failure frequency varied widely from revision to revision - frustrating. However, I finally found that prior to http://hg.mozilla.org/mozilla-central/rev/a57d3b17dcf8 (March 11), this test was skipped on Android! Ryan -- Should I perhaps just add back the skip-if(Android)? ...with a reference back to this bug?
Blocks: 617653
Flags: needinfo?(ryanvm)
Go for it. I'd rather annotate the expected assertion range instead, but if they're varying wildly from run to run, seems doubtful it'd do much good in practice.
Flags: needinfo?(ryanvm)
An assertion range might have worked, but I prefer the historical consistency of re-instating the skip-if.
Assignee: nobody → gbrown
https://hg.mozilla.org/mozilla-central/rev/2b3373745ef7
Status: NEW → RESOLVED
Closed: 9 years ago
status-firefox49: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla49
status-firefox48: --- → affected
Whiteboard: [checkin-needed-aurora]
You need to log in before you can comment on or make changes to this bug.