1427833 - Assertion failure: aStatus.IsEmpty() (Caller should pass a fresh reflow status!), at /builds/worker/workspace/build/src/layout/tables/nsTableRowFrame.cpp:1057

Status: NEW

(Core :: Layout: Tables, defect, P3)

Description

[Jason Kratzer [:jkratzer]]

Testcase found while fuzzing mozilla-central rev ac93fdadf102.

OS|Linux|0.0.0 Linux 4.4.0-104-generic #127-Ubuntu SMP Mon Dec 11 12:16:42 UTC 2017 x86_64
CPU|amd64|family 6 model 78 stepping 3|1
GPU|||
Crash|SIGSEGV|0x0|0
0|0|libxul.so|nsTableRowFrame::Reflow|hg:hg.mozilla.org/mozilla-central:layout/tables/nsTableRowFrame.cpp:ac93fdadf102|1057|0x18
0|1|libxul.so|nsContainerFrame::ReflowChild|hg:hg.mozilla.org/mozilla-central:layout/generic/nsContainerFrame.cpp:ac93fdadf102|934|0x1a
0|2|libxul.so|nsTableRowGroupFrame::ReflowChildren|hg:hg.mozilla.org/mozilla-central:layout/tables/nsTableRowGroupFrame.cpp:ac93fdadf102|426|0x5
0|3|libxul.so|nsTableRowGroupFrame::Reflow|hg:hg.mozilla.org/mozilla-central:layout/tables/nsTableRowGroupFrame.cpp:ac93fdadf102|1384|0x5
0|4|libxul.so|nsContainerFrame::ReflowChild|hg:hg.mozilla.org/mozilla-central:layout/generic/nsContainerFrame.cpp:ac93fdadf102|934|0x1a
0|5|libxul.so|nsTableFrame::ReflowChildren|hg:hg.mozilla.org/mozilla-central:layout/tables/nsTableFrame.cpp:ac93fdadf102|3392|0x43
0|6|libxul.so|nsTableFrame::ReflowTable|hg:hg.mozilla.org/mozilla-central:layout/tables/nsTableFrame.cpp:ac93fdadf102|2337|0x17
0|7|libxul.so|nsTableFrame::Reflow|hg:hg.mozilla.org/mozilla-central:layout/tables/nsTableFrame.cpp:ac93fdadf102|2137|0x1a
0|8|libxul.so|nsContainerFrame::ReflowChild|hg:hg.mozilla.org/mozilla-central:layout/generic/nsContainerFrame.cpp:ac93fdadf102|934|0x1a
0|9|libxul.so|nsTableWrapperFrame::OuterDoReflowChild|hg:hg.mozilla.org/mozilla-central:layout/tables/nsTableWrapperFrame.cpp:ac93fdadf102|841|0x1a
0|10|libxul.so|nsTableWrapperFrame::Reflow|hg:hg.mozilla.org/mozilla-central:layout/tables/nsTableWrapperFrame.cpp:ac93fdadf102|1003|0x20
0|11|libxul.so|nsBlockReflowContext::ReflowBlock|hg:hg.mozilla.org/mozilla-central:layout/generic/nsBlockReflowContext.cpp:ac93fdadf102|306|0x10
0|12|libxul.so|nsBlockFrame::ReflowBlockFrame|hg:hg.mozilla.org/mozilla-central:layout/generic/nsBlockFrame.cpp:ac93fdadf102|3476|0x1e
0|13|libxul.so|nsBlockFrame::ReflowLine|hg:hg.mozilla.org/mozilla-central:layout/generic/nsBlockFrame.cpp:ac93fdadf102|2824|0x13
0|14|libxul.so|nsBlockFrame::ReflowDirtyLines|hg:hg.mozilla.org/mozilla-central:layout/generic/nsBlockFrame.cpp:ac93fdadf102|2363|0x20
0|15|libxul.so|nsBlockFrame::Reflow|hg:hg.mozilla.org/mozilla-central:layout/generic/nsBlockFrame.cpp:ac93fdadf102|1236|0xf
0|16|libxul.so|nsContainerFrame::ReflowChild|hg:hg.mozilla.org/mozilla-central:layout/generic/nsContainerFrame.cpp:ac93fdadf102|934|0x1a
0|17|libxul.so|nsCanvasFrame::Reflow|hg:hg.mozilla.org/mozilla-central:layout/generic/nsCanvasFrame.cpp:ac93fdadf102|758|0x4d
0|18|libxul.so|nsContainerFrame::ReflowChild|hg:hg.mozilla.org/mozilla-central:layout/generic/nsContainerFrame.cpp:ac93fdadf102|934|0x1a
0|19|libxul.so|nsHTMLScrollFrame::ReflowScrolledFrame|hg:hg.mozilla.org/mozilla-central:layout/generic/nsGfxScrollFrame.cpp:ac93fdadf102|554|0x5
0|20|libxul.so|nsHTMLScrollFrame::ReflowContents|hg:hg.mozilla.org/mozilla-central:layout/generic/nsGfxScrollFrame.cpp:ac93fdadf102|676|0x14
0|21|libxul.so|nsHTMLScrollFrame::Reflow|hg:hg.mozilla.org/mozilla-central:layout/generic/nsGfxScrollFrame.cpp:ac93fdadf102|1052|0x5
0|22|libxul.so|nsContainerFrame::ReflowChild|hg:hg.mozilla.org/mozilla-central:layout/generic/nsContainerFrame.cpp:ac93fdadf102|978|0x19
0|23|libxul.so|mozilla::ViewportFrame::Reflow|hg:hg.mozilla.org/mozilla-central:layout/generic/ViewportFrame.cpp:ac93fdadf102|336|0x2b
0|24|libxul.so|mozilla::PresShell::DoReflow|hg:hg.mozilla.org/mozilla-central:layout/base/PresShell.cpp:ac93fdadf102|9002|0x25
0|25|libxul.so|mozilla::PresShell::ProcessReflowCommands|hg:hg.mozilla.org/mozilla-central:layout/base/PresShell.cpp:ac93fdadf102|9175|0xe
0|26|libxul.so|mozilla::PresShell::DoFlushPendingNotifications|hg:hg.mozilla.org/mozilla-central:layout/base/PresShell.cpp:ac93fdadf102|4273|0x15
0|27|libxul.so|mozilla::PresShell::DoFlushPendingNotifications|hg:hg.mozilla.org/mozilla-central:layout/base/PresShell.cpp:ac93fdadf102|4098|0x7
0|28|libxul.so|nsDocumentViewer::LoadComplete|hg:hg.mozilla.org/mozilla-central:layout/base/nsIPresShell.h:ac93fdadf102|575|0xf
0|29|libxul.so|nsDocShell::EndPageLoad|hg:hg.mozilla.org/mozilla-central:docshell/base/nsDocShell.cpp:ac93fdadf102|7902|0x18
0|30|libxul.so|nsDocShell::OnStateChange|hg:hg.mozilla.org/mozilla-central:docshell/base/nsDocShell.cpp:ac93fdadf102|7695|0x18
0|31|libxul.so|nsDocLoader::DoFireOnStateChange|hg:hg.mozilla.org/mozilla-central:uriloader/base/nsDocLoader.cpp:ac93fdadf102|1319|0x2b
0|32|libxul.so|nsDocLoader::doStopDocumentLoad|hg:hg.mozilla.org/mozilla-central:uriloader/base/nsDocLoader.cpp:ac93fdadf102|862|0x22
0|33|libxul.so|nsDocLoader::DocLoaderIsEmpty|hg:hg.mozilla.org/mozilla-central:uriloader/base/nsDocLoader.cpp:ac93fdadf102|751|0xf
0|34|libxul.so|nsDocLoader::OnStopRequest|hg:hg.mozilla.org/mozilla-central:uriloader/base/nsDocLoader.cpp:ac93fdadf102|633|0x16
0|35|libxul.so|mozilla::net::nsLoadGroup::RemoveRequest|hg:hg.mozilla.org/mozilla-central:netwerk/base/nsLoadGroup.cpp:ac93fdadf102|629|0x1f
0|36|libxul.so|nsDocument::DoUnblockOnload|hg:hg.mozilla.org/mozilla-central:dom/base/nsDocument.cpp:ac93fdadf102|9052|0x20
0|37|libxul.so|nsDocument::UnblockOnload|hg:hg.mozilla.org/mozilla-central:dom/base/nsDocument.cpp:ac93fdadf102|8974|0x5
0|38|libxul.so|nsDocument::DispatchContentLoadedEvents|hg:hg.mozilla.org/mozilla-central:dom/base/nsDocument.cpp:ac93fdadf102|5606|0x11
0|39|libxul.so|mozilla::detail::RunnableMethodImpl<nsDocument*, void (nsDocument::*)(), true, (mozilla::RunnableKind)0u>::Run|hg:hg.mozilla.org/mozilla-central:xpcom/threads/nsThreadUtils.h:ac93fdadf102|1142|0x13
0|40|libxul.so|mozilla::SchedulerGroup::Runnable::Run|hg:hg.mozilla.org/mozilla-central:xpcom/threads/SchedulerGroup.cpp:ac93fdadf102|395|0x1c
0|41|libxul.so|nsThread::ProcessNextEvent|hg:hg.mozilla.org/mozilla-central:xpcom/threads/nsThread.cpp:ac93fdadf102|1039|0x15
0|42|libxul.so|NS_ProcessNextEvent|hg:hg.mozilla.org/mozilla-central:xpcom/threads/nsThreadUtils.cpp:ac93fdadf102|510|0x11
0|43|libxul.so|mozilla::ipc::MessagePump::Run|hg:hg.mozilla.org/mozilla-central:ipc/glue/MessagePump.cpp:ac93fdadf102|97|0xa
0|44|libxul.so|MessageLoop::RunInternal|hg:hg.mozilla.org/mozilla-central:ipc/chromium/src/base/message_loop.cc:ac93fdadf102|326|0x17
0|45|libxul.so|MessageLoop::Run|hg:hg.mozilla.org/mozilla-central:ipc/chromium/src/base/message_loop.cc:ac93fdadf102|319|0x8
0|46|libxul.so|nsBaseAppShell::Run|hg:hg.mozilla.org/mozilla-central:widget/nsBaseAppShell.cpp:ac93fdadf102|157|0xd
0|47|libxul.so|XRE_RunAppShell|hg:hg.mozilla.org/mozilla-central:toolkit/xre/nsEmbedFunctions.cpp:ac93fdadf102|875|0x11
0|48|libxul.so|mozilla::ipc::MessagePumpForChildProcess::Run|hg:hg.mozilla.org/mozilla-central:ipc/glue/MessagePump.cpp:ac93fdadf102|269|0x5
0|49|libxul.so|MessageLoop::RunInternal|hg:hg.mozilla.org/mozilla-central:ipc/chromium/src/base/message_loop.cc:ac93fdadf102|326|0x17
0|50|libxul.so|MessageLoop::Run|hg:hg.mozilla.org/mozilla-central:ipc/chromium/src/base/message_loop.cc:ac93fdadf102|319|0x8
0|51|libxul.so|XRE_InitChildProcess|hg:hg.mozilla.org/mozilla-central:toolkit/xre/nsEmbedFunctions.cpp:ac93fdadf102|701|0x8
0|52|firefox|content_process_main|hg:hg.mozilla.org/mozilla-central:ipc/contentproc/plugin-container.cpp:ac93fdadf102|63|0x14
0|53|firefox|main|hg:hg.mozilla.org/mozilla-central:browser/app/nsBrowserApp.cpp:ac93fdadf102|280|0x11
0|54|libc-2.23.so||||0x20830
0|55|firefox|MOZ_ReportAssertionFailure|hg:hg.mozilla.org/mozilla-central:mfbt/Assertions.h:ac93fdadf102|165|0x5

Comment 1

[Jason Kratzer [:jkratzer]]

Attachment: trigger.html

Comment 2

[Jet Villegas (inactive)]

[ Triage 2017/02/20: P3 ]

Comment 3

[Bob Clary [:bc] (inactive)]

Attachment: testcase-astatus-isempty.html

Assertion failure: aStatus.IsEmpty() (Caller should pass a fresh reflow status!), at /mozilla/builds/nightly/mozilla/layout/tables/nsTableRowFrame.cpp:1057

Comment 4

[Bob Clary [:bc] (inactive)]

Attachment: testcase-astatus-isempty.log

Comment 5

[Bob Clary [:bc] (inactive)]

Attachment: testcase-astatus-isincomplete.html

Assertion failure: !aStatus.IsIncomplete() || isPaginated (Table contents should only fragment in paginated contexts), at /mozilla/builds/nightly/mozilla/layout/tables/nsTableFrame.cpp:3521

with only a slight change in indentation.

Comment 6

[Bob Clary [:bc] (inactive)]

Attachment: testcase-astatus-isincomplete.log

Comment 7

[Tyson Smith [:tsmith]]

A Pernosco session is available here: https://pernos.co/debug/H1O6KbsPPJECC2mfXg-zqw/index.html

Comment 8

[Daniel Holbert [:dholbert]]

Note: when we get to fixing this bug, we should also check the testcase in bug 1758205 ( https://bugzilla.mozilla.org/attachment.cgi?id=9266620 ) which trips the same fatal-assertion in debug builds.

(I've landed a version of that testcase as a crashtest layout/printing/crashtests/1758205-1.html which gets skipped in debug builds due to this issue for now. Hopefully when this bug is fixed we can remove the skip annotation there.)