Closed Bug 982917 Opened 6 years ago Closed 6 years ago

TEST-UNEXPECTED-PASS | http://10.12.136.22:30855/tests/layout/forms/crashtests/378413-1.xhtml | assertion count 4 is less than expected 5 to 10 assertions on Android 4.0 Debug

Categories

(Firefox for Android :: General, defect)

x86
Android
defect
Not set

Tracking

()

RESOLVED FIXED
Firefox 33

People

(Reporter: gbrown, Assigned: gbrown)

References

Details

Attachments

(1 file)

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, crashtests consistently fail with various assertion count mismatches. This is one of them.

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

08:50:59     INFO -  REFTEST TEST-START | http://10.12.136.22:30855/tests/layout/forms/crashtests/378413-1.xhtml
08:50:59     INFO -  REFTEST TEST-LOAD | http://10.12.136.22:30855/tests/layout/forms/crashtests/378413-1.xhtml | 1221 / 2632 (46%)
08:50:59     INFO -  REFTEST INFO | START http://10.12.136.22:30855/tests/layout/forms/crashtests/378413-1.xhtml
08:50:59     INFO -  REFTEST INFO | [CONTENT] AfterPaintListener in data:text/html;charset=UTF-8,%3C%21%2D%2DCLEAR%2D%2D%3E
08:50:59     INFO -  REFTEST INFO | [CONTENT] AfterPaintListener in data:text/html;charset=UTF-8,%3C%21%2D%2DCLEAR%2D%2D%3E
08:50:59     INFO -  REFTEST INFO | [CONTENT] OnDocumentLoad triggering AfterOnLoadScripts
08:50:59     INFO -  REFTEST INFO | [CONTENT] AfterPaintListener in http://10.12.136.22:30855/tests/layout/forms/crashtests/378413-1.xhtml
08:50:59     INFO -  REFTEST INFO | [CONTENT] AfterPaintListener in http://10.12.136.22:30855/tests/layout/forms/crashtests/378413-1.xhtml
08:50:59     INFO -  REFTEST INFO | Initializing canvas snapshot
08:50:59     INFO -  REFTEST INFO | [CONTENT] AfterOnLoadScripts belatedly entering WaitForTestEnd
08:50:59     INFO -  REFTEST INFO | [CONTENT] WaitForTestEnd: Adding listeners
08:50:59     INFO -  REFTEST INFO | Initializing canvas snapshot
08:50:59     INFO -  REFTEST INFO | [CONTENT] MakeProgress: STATE_WAITING_TO_FIRE_INVALIDATE_EVENT
08:50:59     INFO -  REFTEST INFO | [CONTENT] MakeProgress: dispatching MozReftestInvalidate
08:50:59     INFO -  REFTEST INFO | [CONTENT] MakeProgress: STATE_WAITING_FOR_REFTEST_WAIT_REMOVAL
08:50:59     INFO -  REFTEST INFO | [CONTENT] MakeProgress: STATE_WAITING_FOR_SPELL_CHECKS
08:50:59     INFO -  REFTEST INFO | [CONTENT] MakeProgress: STATE_WAITING_TO_FINISH
08:50:59     INFO -  REFTEST INFO | [CONTENT] MakeProgress: Completed
08:50:59     INFO -  REFTEST INFO | [CONTENT] RecordResult fired
08:50:59     INFO -  REFTEST INFO | RecordResult fired
08:50:59     INFO -  REFTEST TEST-PASS | http://10.12.136.22:30855/tests/layout/forms/crashtests/378413-1.xhtml | (LOAD ONLY)
08:50:59     INFO -  REFTEST INFO | Loading a blank page
08:50:59     INFO -  REFTEST TEST-UNEXPECTED-PASS | http://10.12.136.22:30855/tests/layout/forms/crashtests/378413-1.xhtml | assertion count 4 is less than expected 5 to 10 assertions
08:50:59     INFO -  REFTEST TEST-END | http://10.12.136.22:30855/tests/layout/forms/crashtests/378413-1.xhtml

Full logcat at http://mozilla-releng-blobs.s3.amazonaws.com/blobs/cedar/sha512/faa649ea08d1fa6e7aae90cfff9917cfb422bfbfcf5d39183ec3f954aeb5a704003a43008bfb31faabdd96f579169828d268c05432d9f3320059d78486d1092c


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

https://tbpl.mozilla.org/php/getParsedLog.php?id=41379199&tree=Mozilla-Central&full=1

13:12:25     INFO -  REFTEST TEST-START | file:///builds/slave/test/build/tests/reftest/tests/layout/forms/crashtests/378413-1.xhtml
13:12:25     INFO -  REFTEST TEST-LOAD | file:///builds/slave/test/build/tests/reftest/tests/layout/forms/crashtests/378413-1.xhtml | 1231 / 2659 (46%)
13:12:25     INFO -  ++DOMWINDOW == 99 (0xa3d93f90) [pid = 1718] [serial = 2952] [outer = 0xa3d914f0]
13:12:25     INFO -  [1718] ###!!! ASSERTION: Bogus availSize.width; should be bigger: '!inner || nsLayoutUtils::IntrinsicForContainer(aReflowState.rendContext, inner, nsLayoutUtils::MIN_WIDTH) <= availSize.width', file /builds/slave/m-cen-lx-d-0000000000000000000/build/layout/forms/nsFieldSetFrame.cpp, line 377
13:12:25     INFO -  nsFieldSetFrame::Reflow(nsPresContext*, nsHTMLReflowMetrics&, nsHTMLReflowState const&, unsigned int&) [layout/forms/nsFieldSetFrame.cpp:372]
13:12:25     INFO -  nsFrame::BoxReflow(nsBoxLayoutState&, nsPresContext*, nsHTMLReflowMetrics&, nsRenderingContext*, int, int, int, int, bool) [layout/generic/nsFrame.cpp:8350]
13:12:25     INFO -  nsFrame::DoLayout(nsBoxLayoutState&) [layout/generic/nsFrame.cpp:8093]
13:12:25     INFO -  nsIFrame::Layout(nsBoxLayoutState&) [layout/xul/nsBox.cpp:512]
13:12:25     INFO -  nsStackLayout::Layout(nsIFrame*, nsBoxLayoutState&) [layout/generic/nsIFrame.h:630]
13:12:25     INFO -  nsBoxFrame::DoLayout(nsBoxLayoutState&) [layout/xul/nsBoxFrame.cpp:901]
13:12:25     INFO -  nsDeckFrame::DoLayout(nsBoxLayoutState&) [layout/xul/nsDeckFrame.cpp:185]
13:12:25     INFO -  nsIFrame::Layout(nsBoxLayoutState&) [layout/xul/nsBox.cpp:512]
13:12:25     INFO -  nsBoxFrame::Reflow(nsPresContext*, nsHTMLReflowMetrics&, nsHTMLReflowState const&, unsigned int&) [layout/xul/nsBoxFrame.cpp:706]
13:12:25     INFO -  nsLineLayout::ReflowFrame(nsIFrame*, unsigned int&, nsHTMLReflowMetrics*, bool&) [layout/generic/nsLineLayout.cpp:845]
13:12:25     INFO -  nsBlockFrame::ReflowInlineFrame(nsBlockReflowState&, nsLineLayout&, nsLineList_iterator, nsIFrame*, LineReflowStatus*) [layout/generic/nsBlockFrame.cpp:3757]
13:12:25     INFO -  nsBlockFrame::DoReflowInlineFrames(nsBlockReflowState&, nsLineLayout&, nsLineList_iterator, nsFlowAreaRect&, int&, nsFloatManager::SavedState*, bool*, LineReflowStatus*, bool) [layout/generic/nsBlockFrame.cpp:3556]
13:12:25     INFO -  nsBlockFrame::ReflowInlineFrames(nsBlockReflowState&, nsLineList_iterator, bool*) [layout/generic/nsBlockFrame.cpp:3429]
13:12:25     INFO -  nsBlockFrame::ReflowLine(nsBlockReflowState&, nsLineList_iterator, bool*) [layout/generic/nsBlockFrame.cpp:2585]
13:12:25     INFO -  nsBlockFrame::ReflowDirtyLines(nsBlockReflowState&) [layout/generic/nsBlockFrame.cpp:7212]
13:12:25     INFO -  nsBlockFrame::Reflow(nsPresContext*, nsHTMLReflowMetrics&, nsHTMLReflowState const&, unsigned int&) [layout/generic/nsBlockFrame.cpp:1135]
13:12:25     INFO -  nsBlockReflowContext::ReflowBlock(nsRect const&, bool, nsCollapsingMargin&, int, bool, nsLineBox*, nsHTMLReflowState&, unsigned int&, nsBlockReflowState&) [layout/generic/nsBlockReflowContext.cpp:261]
13:12:25     INFO -  nsBlockFrame::ReflowBlockFrame(nsBlockReflowState&, nsLineList_iterator, bool*) [layout/generic/nsBlockFrame.cpp:3156]
13:12:25     INFO -  nsBlockFrame::ReflowLine(nsBlockReflowState&, nsLineList_iterator, bool*) [layout/generic/nsBlockFrame.cpp:2582]
13:12:25     INFO -  nsBlockFrame::ReflowDirtyLines(nsBlockReflowState&) [layout/generic/nsBlockFrame.cpp:7212]
13:12:25     INFO -  nsBlockFrame::Reflow(nsPresContext*, nsHTMLReflowMetrics&, nsHTMLReflowState const&, unsigned int&) [layout/generic/nsBlockFrame.cpp:1135]
13:12:25     INFO -  nsContainerFrame::ReflowChild(nsIFrame*, nsPresContext*, nsHTMLReflowMetrics&, nsHTMLReflowState const&, int, int, unsigned int, unsigned int&, nsOverflowContinuationTracker*) [layout/generic/nsContainerFrame.cpp:935]
13:12:25     INFO -  nsCanvasFrame::Reflow(nsPresContext*, nsHTMLReflowMetrics&, nsHTMLReflowState const&, unsigned int&) [layout/generic/nsCanvasFrame.cpp:587]
13:12:25     INFO -  nsContainerFrame::ReflowChild(nsIFrame*, nsPresContext*, nsHTMLReflowMetrics&, nsHTMLReflowState const&, int, int, unsigned int, unsigned int&, nsOverflowContinuationTracker*) [layout/generic/nsContainerFrame.cpp:935]
13:12:25     INFO -  nsHTMLScrollFrame::ReflowScrolledFrame(ScrollReflowState*, bool, bool, nsHTMLReflowMetrics*, bool) [layout/generic/nsGfxScrollFrame.cpp:454]
13:12:25     INFO -  nsHTMLScrollFrame::ReflowContents(ScrollReflowState*, nsHTMLReflowMetrics const&) [layout/generic/nsGfxScrollFrame.cpp:581]
13:12:25     INFO -  nsHTMLScrollFrame::Reflow(nsPresContext*, nsHTMLReflowMetrics&, nsHTMLReflowState const&, unsigned int&) [layout/generic/nsGfxScrollFrame.cpp:798]
13:12:25     INFO -  nsContainerFrame::ReflowChild(nsIFrame*, nsPresContext*, nsHTMLReflowMetrics&, nsHTMLReflowState const&, int, int, unsigned int, unsigned int&, nsOverflowContinuationTracker*) [layout/generic/nsContainerFrame.cpp:935]
13:12:25     INFO -  ViewportFrame::Reflow(nsPresContext*, nsHTMLReflowMetrics&, nsHTMLReflowState const&, unsigned int&) [layout/generic/nsViewportFrame.cpp:219]
13:12:25     INFO -  PresShell::DoReflow(nsIFrame*, bool) [obj-firefox/dist/include/mozilla/gfx/BaseRect.h:54]
13:12:25     INFO -  PresShell::ProcessReflowCommands(bool) [layout/base/nsPresShell.cpp:8905]
13:12:25     INFO -  PresShell::FlushPendingNotifications(mozilla::ChangesToFlush) [layout/base/nsPresShell.cpp:4233]
13:12:25     INFO -  nsRefreshDriver::Tick(long long, mozilla::TimeStamp) [layout/base/nsRefreshDriver.cpp:1223]
13:12:25     INFO -  mozilla::RefreshDriverTimer::Tick() [layout/base/nsRefreshDriver.cpp:162]
13:12:25     INFO -  nsTimerImpl::Fire() [xpcom/threads/nsTimerImpl.cpp:609]
13:12:25     INFO -  nsTimerEvent::Run() [xpcom/threads/nsTimerImpl.cpp:704]
13:12:25     INFO -  nsThread::ProcessNextEvent(bool, bool*) [xpcom/threads/nsThread.cpp:766]
13:12:25     INFO -  NS_ProcessNextEvent(nsIThread*, bool) [xpcom/glue/nsThreadUtils.cpp:263]
13:12:25     INFO -  mozilla::ipc::MessagePump::Run(base::MessagePump::Delegate*) [ipc/glue/MessagePump.cpp:96]
13:12:25     INFO -  MessageLoop::RunInternal() [ipc/chromium/src/base/message_loop.cc:229]
13:12:25     INFO -  MessageLoop::Run() [ipc/chromium/src/base/message_loop.cc:504]
13:12:25     INFO -  nsBaseAppShell::Run() [widget/xpwidgets/nsBaseAppShell.cpp:166]
13:12:25     INFO -  nsAppStartup::Run() [toolkit/components/startup/nsAppStartup.cpp:279]
13:12:25     INFO -  XREMain::XRE_mainRun() [toolkit/xre/nsAppRunner.cpp:4013]
13:12:25     INFO -  XREMain::XRE_main(int, char**, nsXREAppData const*) [toolkit/xre/nsAppRunner.cpp:4083]
13:12:25     INFO -  XRE_main [toolkit/xre/nsAppRunner.cpp:4298]
13:12:25     INFO -  do_main [browser/app/nsBrowserApp.cpp:282]
13:12:25     INFO -  main [browser/app/nsBrowserApp.cpp:645]
13:12:25     INFO -  [1718] ###!!! ASSERTION: Bogus availSize.width; should be bigger: '!legend || nsLayoutUtils::IntrinsicForContainer(aReflowState.rendContext, legend, nsLayoutUtils::MIN_WIDTH) <= availSize.width', file /builds/slave/m-cen-lx-d-0000000000000000000/build/layout/forms/nsFieldSetFrame.cpp, line 383
13:12:25     INFO -  nsFieldSetFrame::Reflow(nsPresContext*, nsHTMLReflowMetrics&, nsHTMLReflowState const&, unsigned int&) [layout/forms/nsFieldSetFrame.cpp:378]
13:12:25     INFO -  nsFrame::BoxReflow(nsBoxLayoutState&, nsPresContext*, nsHTMLReflowMetrics&, nsRenderingContext*, int, int, int, int, bool) [layout/generic/nsFrame.cpp:8350]
13:12:25     INFO -  nsFrame::DoLayout(nsBoxLayoutState&) [layout/generic/nsFrame.cpp:8093]
13:12:25     INFO -  nsIFrame::Layout(nsBoxLayoutState&) [layout/xul/nsBox.cpp:512]
13:12:25     INFO -  nsStackLayout::Layout(nsIFrame*, nsBoxLayoutState&) [layout/generic/nsIFrame.h:630]
13:12:25     INFO -  nsBoxFrame::DoLayout(nsBoxLayoutState&) [layout/xul/nsBoxFrame.cpp:901]
13:12:25     INFO -  nsDeckFrame::DoLayout(nsBoxLayoutState&) [layout/xul/nsDeckFrame.cpp:185]
13:12:25     INFO -  nsIFrame::Layout(nsBoxLayoutState&) [layout/xul/nsBox.cpp:512]
13:12:25     INFO -  nsBoxFrame::Reflow(nsPresContext*, nsHTMLReflowMetrics&, nsHTMLReflowState const&, unsigned int&) [layout/xul/nsBoxFrame.cpp:706]
13:12:25     INFO -  nsLineLayout::ReflowFrame(nsIFrame*, unsigned int&, nsHTMLReflowMetrics*, bool&) [layout/generic/nsLineLayout.cpp:845]
13:12:25     INFO -  nsBlockFrame::ReflowInlineFrame(nsBlockReflowState&, nsLineLayout&, nsLineList_iterator, nsIFrame*, LineReflowStatus*) [layout/generic/nsBlockFrame.cpp:3757]
13:12:25     INFO -  nsBlockFrame::DoReflowInlineFrames(nsBlockReflowState&, nsLineLayout&, nsLineList_iterator, nsFlowAreaRect&, int&, nsFloatManager::SavedState*, bool*, LineReflowStatus*, bool) [layout/generic/nsBlockFrame.cpp:3556]
13:12:25     INFO -  nsBlockFrame::ReflowInlineFrames(nsBlockReflowState&, nsLineList_iterator, bool*) [layout/generic/nsBlockFrame.cpp:3429]
13:12:25     INFO -  nsBlockFrame::ReflowLine(nsBlockReflowState&, nsLineList_iterator, bool*) [layout/generic/nsBlockFrame.cpp:2585]
13:12:25     INFO -  nsBlockFrame::ReflowDirtyLines(nsBlockReflowState&) [layout/generic/nsBlockFrame.cpp:7212]
13:12:25     INFO -  nsBlockFrame::Reflow(nsPresContext*, nsHTMLReflowMetrics&, nsHTMLReflowState const&, unsigned int&) [layout/generic/nsBlockFrame.cpp:1135]
13:12:25     INFO -  nsBlockReflowContext::ReflowBlock(nsRect const&, bool, nsCollapsingMargin&, int, bool, nsLineBox*, nsHTMLReflowState&, unsigned int&, nsBlockReflowState&) [layout/generic/nsBlockReflowContext.cpp:261]
13:12:25     INFO -  nsBlockFrame::ReflowBlockFrame(nsBlockReflowState&, nsLineList_iterator, bool*) [layout/generic/nsBlockFrame.cpp:3156]
13:12:25     INFO -  nsBlockFrame::ReflowLine(nsBlockReflowState&, nsLineList_iterator, bool*) [layout/generic/nsBlockFrame.cpp:2582]
13:12:25     INFO -  nsBlockFrame::ReflowDirtyLines(nsBlockReflowState&) [layout/generic/nsBlockFrame.cpp:7212]
13:12:25     INFO -  nsBlockFrame::Reflow(nsPresContext*, nsHTMLReflowMetrics&, nsHTMLReflowState const&, unsigned int&) [layout/generic/nsBlockFrame.cpp:1135]
13:12:25     INFO -  nsContainerFrame::ReflowChild(nsIFrame*, nsPresContext*, nsHTMLReflowMetrics&, nsHTMLReflowState const&, int, int, unsigned int, unsigned int&, nsOverflowContinuationTracker*) [layout/generic/nsContainerFrame.cpp:935]
13:12:25     INFO -  nsCanvasFrame::Reflow(nsPresContext*, nsHTMLReflowMetrics&, nsHTMLReflowState const&, unsigned int&) [layout/generic/nsCanvasFrame.cpp:587]
13:12:25     INFO -  nsContainerFrame::ReflowChild(nsIFrame*, nsPresContext*, nsHTMLReflowMetrics&, nsHTMLReflowState const&, int, int, unsigned int, unsigned int&, nsOverflowContinuationTracker*) [layout/generic/nsContainerFrame.cpp:935]
13:12:25     INFO -  nsHTMLScrollFrame::ReflowScrolledFrame(ScrollReflowState*, bool, bool, nsHTMLReflowMetrics*, bool) [layout/generic/nsGfxScrollFrame.cpp:454]
13:12:25     INFO -  nsHTMLScrollFrame::ReflowContents(ScrollReflowState*, nsHTMLReflowMetrics const&) [layout/generic/nsGfxScrollFrame.cpp:581]
13:12:25     INFO -  nsHTMLScrollFrame::Reflow(nsPresContext*, nsHTMLReflowMetrics&, nsHTMLReflowState const&, unsigned int&) [layout/generic/nsGfxScrollFrame.cpp:798]
13:12:25     INFO -  nsContainerFrame::ReflowChild(nsIFrame*, nsPresContext*, nsHTMLReflowMetrics&, nsHTMLReflowState const&, int, int, unsigned int, unsigned int&, nsOverflowContinuationTracker*) [layout/generic/nsContainerFrame.cpp:935]
13:12:25     INFO -  ViewportFrame::Reflow(nsPresContext*, nsHTMLReflowMetrics&, nsHTMLReflowState const&, unsigned int&) [layout/generic/nsViewportFrame.cpp:219]
13:12:25     INFO -  PresShell::DoReflow(nsIFrame*, bool) [obj-firefox/dist/include/mozilla/gfx/BaseRect.h:54]
13:12:25     INFO -  PresShell::ProcessReflowCommands(bool) [layout/base/nsPresShell.cpp:8905]
13:12:25     INFO -  PresShell::FlushPendingNotifications(mozilla::ChangesToFlush) [layout/base/nsPresShell.cpp:4233]
13:12:25     INFO -  nsRefreshDriver::Tick(long long, mozilla::TimeStamp) [layout/base/nsRefreshDriver.cpp:1223]
13:12:25     INFO -  mozilla::RefreshDriverTimer::Tick() [layout/base/nsRefreshDriver.cpp:162]
13:12:25     INFO -  nsTimerImpl::Fire() [xpcom/threads/nsTimerImpl.cpp:609]
13:12:25     INFO -  nsTimerEvent::Run() [xpcom/threads/nsTimerImpl.cpp:704]
13:12:25     INFO -  nsThread::ProcessNextEvent(bool, bool*) [xpcom/threads/nsThread.cpp:766]
13:12:25     INFO -  NS_ProcessNextEvent(nsIThread*, bool) [xpcom/glue/nsThreadUtils.cpp:263]
13:12:25     INFO -  mozilla::ipc::MessagePump::Run(base::MessagePump::Delegate*) [ipc/glue/MessagePump.cpp:96]
13:12:25     INFO -  MessageLoop::RunInternal() [ipc/chromium/src/base/message_loop.cc:229]
13:12:25     INFO -  MessageLoop::Run() [ipc/chromium/src/base/message_loop.cc:504]
13:12:25     INFO -  nsBaseAppShell::Run() [widget/xpwidgets/nsBaseAppShell.cpp:166]
13:12:25     INFO -  nsAppStartup::Run() [toolkit/components/startup/nsAppStartup.cpp:279]
13:12:25     INFO -  XREMain::XRE_mainRun() [toolkit/xre/nsAppRunner.cpp:4013]
13:12:25     INFO -  XREMain::XRE_main(int, char**, nsXREAppData const*) [toolkit/xre/nsAppRunner.cpp:4083]
13:12:25     INFO -  XRE_main [toolkit/xre/nsAppRunner.cpp:4298]
13:12:25     INFO -  do_main [browser/app/nsBrowserApp.cpp:282]
13:12:25     INFO -  main [browser/app/nsBrowserApp.cpp:645]
...
13:12:26     INFO -  REFTEST TEST-KNOWN-FAIL | file:///builds/slave/test/build/tests/reftest/tests/layout/forms/crashtests/378413-1.xhtml | assertion count 10 matches expected 5 to 10 assertions
13:12:26     INFO -  REFTEST TEST-END | file:///builds/slave/test/build/tests/reftest/tests/layout/forms/crashtests/378413-1.xhtml
Assignee: nobody → gbrown
This seems like the same scenario as bug 1021998: The test experiences repeated assertions on reflow, but on Android, there are fewer assertions than expected. I am changing the asserts() range to allow for fewer assertions.
Attachment #8437318 - Flags: review?(dholbert)
Comment on attachment 8437318 [details] [diff] [review]
allow fewer assertions for this test

># HG changeset patch
># Parent 2200339083791ff8374354df35636c0341868beb
># User Geoff Brown <gbrown@mozilla.com>
>Bug 982917 - Increase expected assertion range for layout/forms/crashtests/378413-1.xhtml; r=dholbert

Nit: The use of "increase" here sounds a bit like you're saying "allow more assertions", when really you're allowing for us to have *fewer* assertions.

"Broaden" might be a better term, maybe?

r=me regardless.
Attachment #8437318 - Flags: review?(dholbert) → review+
https://hg.mozilla.org/mozilla-central/rev/1c8320f66c5c
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 33
You need to log in before you can comment on or make changes to this bug.