Closed Bug 1268050 Opened 8 years ago Closed 8 years ago

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

Categories

(Core :: XUL, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla49
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: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla49
Whiteboard: [checkin-needed-aurora]
You need to log in before you can comment on or make changes to this bug.