Closed Bug 1506216 Opened 10 months ago Closed 9 months ago

Assertion failure: anonKids.IsEmpty() (We support only column-span on block and inline frame. They should not create any native anonymous children.), at src/layout/generic/ColumnSetWrapperFrame.cpp:154

Categories

(Core :: Layout: Columns, defect, P2)

defect

Tracking

()

RESOLVED FIXED
mozilla65
Tracking Status
firefox-esr60 --- unaffected
firefox63 --- unaffected
firefox64 --- unaffected
firefox65 --- fixed

People

(Reporter: tsmith, Assigned: TYLin)

References

(Blocks 3 open bugs)

Details

(Keywords: assertion, testcase)

Attachments

(3 files, 1 obsolete file)

Attached file testcase.html
Reduced with m-c:
BuildID=20181109174520
SourceStamp=2f1158e5e0ce2523f93482abf6158db058a782a1

Assertion failure: anonKids.IsEmpty() (We support only column-span on block and inline frame. They should not create any native anonymous children.), at src/layout/generic/ColumnSetWrapperFrame.cpp:154

#0 mozilla::ColumnSetWrapperFrame::AssertColumnSpanWrapperSubtreeIsSane(nsIFrame const*) src/layout/generic/ColumnSetWrapperFrame.cpp:145:3
#1 mozilla::ColumnSetWrapperFrame::AppendFrames(mozilla::layout::FrameChildListID, nsFrameList&) src/layout/generic/ColumnSetWrapperFrame.cpp:103:7
#2 nsCSSFrameConstructor::FinishBuildingColumns(nsFrameConstructorState&, nsContainerFrame*, nsContainerFrame*, nsFrameList&) src/layout/base/nsCSSFrameConstructor.cpp:11372:22
#3 nsCSSFrameConstructor::ConstructBlock(nsFrameConstructorState&, nsIContent*, nsContainerFrame*, nsContainerFrame*, mozilla::ComputedStyle*, nsContainerFrame**, nsFrameItems&, nsIFrame*, PendingBinding*) src/layout/base/nsCSSFrameConstructor.cpp:11223:5
#4 nsCSSFrameConstructor::ConstructDocElementFrame(mozilla::dom::Element*, nsILayoutHistoryState*) src/layout/base/nsCSSFrameConstructor.cpp:2640:5
#5 nsCSSFrameConstructor::ContentRangeInserted(nsIContent*, nsIContent*, nsILayoutHistoryState*, nsCSSFrameConstructor::InsertionKind) src/layout/base/nsCSSFrameConstructor.cpp:7367:9
#6 nsCSSFrameConstructor::RecreateFramesForContent(nsIContent*, nsCSSFrameConstructor::InsertionKind) src/layout/base/nsCSSFrameConstructor.cpp:9107:7
#7 mozilla::RestyleManager::ProcessRestyledFrames(nsStyleChangeList&) src/layout/base/RestyleManager.cpp:1551:25
#8 mozilla::RestyleManager::DoProcessPendingRestyles(mozilla::ServoTraversalFlags) src/layout/base/RestyleManager.cpp:3065:9
#9 mozilla::PresShell::DoFlushPendingNotifications(mozilla::ChangesToFlush) src/layout/base/PresShell.cpp:4359:39
#10 nsRefreshDriver::Tick(mozilla::TimeStamp) src/layout/base/nsRefreshDriver.cpp:1907:18
#11 mozilla::RefreshDriverTimer::TickRefreshDrivers(mozilla::TimeStamp, nsTArray<RefPtr<nsRefreshDriver> >&) src/layout/base/nsRefreshDriver.cpp:301:7
#12 mozilla::RefreshDriverTimer::Tick(mozilla::TimeStamp) src/layout/base/nsRefreshDriver.cpp:318:5
#13 mozilla::VsyncRefreshDriverTimer::RefreshDriverVsyncObserver::TickRefreshDriver(mozilla::TimeStamp) src/layout/base/nsRefreshDriver.cpp:676:16
#14 mozilla::VsyncRefreshDriverTimer::RefreshDriverVsyncObserver::ParentProcessVsyncNotifier::Run() src/layout/base/nsRefreshDriver.cpp:514:20
#15 nsThread::ProcessNextEvent(bool, bool*) src/xpcom/threads/nsThread.cpp:1246:14
#16 NS_ProcessNextEvent(nsIThread*, bool) src/xpcom/threads/nsThreadUtils.cpp:530:10
#17 mozilla::ipc::MessagePump::Run(base::MessagePump::Delegate*) src/ipc/glue/MessagePump.cpp:97:21
#18 MessageLoop::RunInternal() src/ipc/chromium/src/base/message_loop.cc:325:10
#19 MessageLoop::Run() src/ipc/chromium/src/base/message_loop.cc:298:3
#20 nsBaseAppShell::Run() src/widget/nsBaseAppShell.cpp:158:27
#21 nsAppStartup::Run() src/toolkit/components/startup/nsAppStartup.cpp:290:30
#22 XREMain::XRE_mainRun() src/toolkit/xre/nsAppRunner.cpp:4789:22
#23 XREMain::XRE_main(int, char**, mozilla::BootstrapConfig const&) src/toolkit/xre/nsAppRunner.cpp:4934:8
#24 XRE_main(int, char**, mozilla::BootstrapConfig const&) src/toolkit/xre/nsAppRunner.cpp:5026:21
#25 do_main(int, char**, char**) src/browser/app/nsBrowserApp.cpp:233:22
#26 main src/browser/app/nsBrowserApp.cpp:315:16
#27 __libc_start_main /build/glibc-Cl5G7W/glibc-2.23/csu/../csu/libc-start.c:291
#28 _start (firefox+0x329f4)
Flags: in-testsuite?
Attached file prefs.js
Flags: needinfo?(aethanyc)
OS: Unspecified → All
Priority: -- → P2
Hardware: Unspecified → All
The test uses multi-columns on html element, and html element (root element) creates native anonymous children like scroll bars in https://searchfox.org/mozilla-central/rev/647b9eb1099e373e079e16f147f74f3d1d65eed3/dom/base/nsContentUtils.cpp#10530. Hence the assert.
Assignee: nobody → aethanyc
Status: NEW → ASSIGNED
Flags: needinfo?(aethanyc)
nsContentUtils::AppendNativeAnonymousChildren() calls
AppendDocumentLevelNativeAnonymousContentTo() to create native anonymous
children for root element.
Attachment #9026240 - Attachment description: Bug 1506216 - Relax the assertion in AssertColumnSpanWrapperSubtreeIsSane() for root element. → Bug 1506216 - Remove bogus assert in AssertColumnSpanWrapperSubtreeIsSane().
Attachment #9026561 - Attachment is obsolete: true
Pushed by aethanyc@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/e70144f3fc89
Remove bogus assert in AssertColumnSpanWrapperSubtreeIsSane(). r=bzbarsky
https://hg.mozilla.org/mozilla-central/rev/e70144f3fc89
Status: ASSIGNED → RESOLVED
Closed: 9 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla65
You need to log in before you can comment on or make changes to this bug.