Closed Bug 1800963 Opened 3 years ago Closed 3 years ago

High freq dom/base/crashtests/<test> | assertion count 1 is more than expected 0 assertions

Categories

(Core :: DOM: HTML Parser, defect)

defect

Tracking

()

RESOLVED DUPLICATE of bug 1709012

People

(Reporter: intermittent-bug-filer, Unassigned)

References

Details

(Keywords: assertion, intermittent-failure, Whiteboard: [stockwell disable-recommended])

Filed by: ctuns [at] mozilla.com
Parsed log: https://treeherder.mozilla.org/logviewer?job_id=396755526&repo=autoland
Full log: https://firefox-ci-tc.services.mozilla.com/api/queue/v1/task/Hmi8mgzYSEmaKNpCwpUvvg/runs/0/artifacts/public/logs/live_backing.log
Reftest URL: https://hg.mozilla.org/mozilla-central/raw-file/tip/layout/tools/reftest/reftest-analyzer.xhtml#logurl=https://firefox-ci-tc.services.mozilla.com/api/queue/v1/task/Hmi8mgzYSEmaKNpCwpUvvg/runs/0/artifacts/public/logs/live_backing.log&only_show_unexpected=1


[task 2022-11-16T18:54:17.948Z] 18:54:17     INFO - REFTEST TEST-PASS | dom/base/crashtests/830098.html | (LOAD ONLY)
[task 2022-11-16T18:54:17.950Z] 18:54:17     INFO - REFTEST TEST-END | dom/base/crashtests/830098.html
[task 2022-11-16T18:54:18.801Z] 18:54:18     INFO - [Child 1804, Main Thread] ###!!! ASSERTION: Somehow there's stuff in the op queue.: 'NS_FAILED(mBroken) || mOpQueue.IsEmpty()', file /builds/worker/checkouts/gecko/parser/html/nsHtml5TreeOpExecutor.cpp:145
[task 2022-11-16T18:54:18.803Z] 18:54:18     INFO - #01: NS_DebugBreak [xpcom/base/nsDebugImpl.cpp:0]
[task 2022-11-16T18:54:18.804Z] 18:54:18     INFO - #02: nsHtml5TreeOpExecutor::~nsHtml5TreeOpExecutor() [parser/html/nsHtml5TreeOpExecutor.cpp:146]
[task 2022-11-16T18:54:18.805Z] 18:54:18     INFO - #03: nsHtml5TreeOpExecutor::~nsHtml5TreeOpExecutor() [parser/html/nsHtml5TreeOpExecutor.cpp:131]
[task 2022-11-16T18:54:18.807Z] 18:54:18     INFO - #04: SnowWhiteKiller::MaybeKillObject(SnowWhiteKiller::SnowWhiteObject&) [xpcom/base/nsCycleCollector.cpp:0]
[task 2022-11-16T18:54:18.808Z] 18:54:18     INFO - #05: SnowWhiteKiller::~SnowWhiteKiller() [xpcom/base/nsCycleCollector.cpp:2432]
[task 2022-11-16T18:54:18.809Z] 18:54:18     INFO - #06: nsCycleCollector::FreeSnowWhite(bool) [xpcom/base/nsCycleCollector.cpp:0]
[task 2022-11-16T18:54:18.810Z] 18:54:18     INFO - #07: nsCycleCollector::BeginCollection(mozilla::CCReason, ccIsManual, nsICycleCollectorListener*) [xpcom/base/nsCycleCollector.cpp:3616]
[task 2022-11-16T18:54:18.812Z] 18:54:18     INFO - #08: nsCycleCollector::Collect(mozilla::CCReason, ccIsManual, js::SliceBudget&, nsICycleCollectorListener*, bool) [xpcom/base/nsCycleCollector.cpp:0]
[task 2022-11-16T18:54:18.813Z] 18:54:18     INFO - #09: nsCycleCollector_collectSlice(js::SliceBudget&, mozilla::CCReason, bool) [xpcom/base/nsCycleCollector.cpp:3964]
[task 2022-11-16T18:54:18.814Z] 18:54:18     INFO - #10: nsJSContext::RunCycleCollectorSlice(mozilla::CCReason, mozilla::TimeStamp) [dom/base/nsJSEnvironment.cpp:1462]
[task 2022-11-16T18:54:18.815Z] 18:54:18     INFO - #11: mozilla::CCGCScheduler::CCRunnerFired(mozilla::TimeStamp) [dom/base/nsJSEnvironment.cpp:0]
[task 2022-11-16T18:54:18.817Z] 18:54:18     INFO - #12: mozilla::IdleTaskRunner::Run() [xpcom/threads/IdleTaskRunner.cpp:130]
[task 2022-11-16T18:54:18.818Z] 18:54:18     INFO - #13: mozilla::IdleTaskRunnerTask::Run() [xpcom/threads/IdleTaskRunner.cpp:47]
[task 2022-11-16T18:54:18.819Z] 18:54:18     INFO - #14: mozilla::TaskController::DoExecuteNextTaskOnlyMainThreadInternal(mozilla::detail::BaseAutoLock<mozilla::Mutex&> const&) [xpcom/threads/TaskController.cpp:851]
[task 2022-11-16T18:54:18.820Z] 18:54:18     INFO - #15: mozilla::TaskController::ExecuteNextTaskOnlyMainThreadInternal(mozilla::detail::BaseAutoLock<mozilla::Mutex&> const&) [xpcom/threads/TaskController.cpp:725]
[task 2022-11-16T18:54:18.822Z] 18:54:18     INFO - #16: mozilla::TaskController::ProcessPendingMTTask(bool) [xpcom/threads/TaskController.cpp:461]
[task 2022-11-16T18:54:18.823Z] 18:54:18     INFO - #17: mozilla::detail::RunnableFunction<mozilla::TaskController::InitializeInternal()::$_2>::Run() [xpcom/threads/nsThreadUtils.h:547]
[task 2022-11-16T18:54:18.824Z] 18:54:18     INFO - #18: nsThread::ProcessNextEvent(bool, bool*) [xpcom/threads/nsThread.cpp:1208]
[task 2022-11-16T18:54:18.825Z] 18:54:18     INFO - #19: NS_ProcessNextEvent(nsIThread*, bool) [xpcom/threads/nsThreadUtils.cpp:474]
[task 2022-11-16T18:54:18.826Z] 18:54:18     INFO - #20: mozilla::ipc::MessagePump::Run(base::MessagePump::Delegate*) [ipc/glue/MessagePump.cpp:86]
[task 2022-11-16T18:54:18.828Z] 18:54:18     INFO - #21: MessageLoop::RunInternal() [ipc/chromium/src/base/message_loop.cc:0]
[task 2022-11-16T18:54:18.829Z] 18:54:18     INFO - #22: MessageLoop::Run() [ipc/chromium/src/base/message_loop.cc:357]
[task 2022-11-16T18:54:18.830Z] 18:54:18     INFO - #23: nsBaseAppShell::Run() [widget/nsBaseAppShell.cpp:152]
[task 2022-11-16T18:54:18.831Z] 18:54:18     INFO - #24: XRE_RunAppShell() [toolkit/xre/nsEmbedFunctions.cpp:884]
[task 2022-11-16T18:54:18.832Z] 18:54:18     INFO - #25: mozilla::ipc::MessagePumpForChildProcess::Run(base::MessagePump::Delegate*) [ipc/glue/MessagePump.cpp:235]
[task 2022-11-16T18:54:18.834Z] 18:54:18     INFO - #26: MessageLoop::RunInternal() [ipc/chromium/src/base/message_loop.cc:0]
[task 2022-11-16T18:54:18.835Z] 18:54:18     INFO - #27: MessageLoop::Run() [ipc/chromium/src/base/message_loop.cc:357]
[task 2022-11-16T18:54:18.836Z] 18:54:18     INFO - #28: XRE_InitChildProcess(int, char**, XREChildData const*) [toolkit/xre/nsEmbedFunctions.cpp:0]
[task 2022-11-16T18:54:18.837Z] 18:54:18     INFO - #29: content_process_main(mozilla::Bootstrap*, int, char**) [ipc/contentproc/plugin-container.cpp:58]
[task 2022-11-16T18:54:18.838Z] 18:54:18     INFO - #30: main [browser/app/nsBrowserApp.cpp:359]
[task 2022-11-16T18:54:18.839Z] 18:54:18     INFO - #31: __libc_start_main [/lib/x86_64-linux-gnu/libc.so.6 + 0x21b97]
[task 2022-11-16T18:54:18.840Z] 18:54:18     INFO - #32: ??? [/builds/worker/workspace/build/application/firefox/firefox-bin + 0x46ec9]
[task 2022-11-16T18:54:18.841Z] 18:54:18     INFO - #33: ??? (???:???)
[task 2022-11-16T18:54:19.216Z] 18:54:19     INFO - REFTEST TEST-UNEXPECTED-FAIL | dom/base/crashtests/830098.html | assertion count 1 is more than expected 0 assertions
[task 2022-11-16T18:54:19.218Z] 18:54:19     INFO - REFTEST TEST-START | dom/base/crashtests/831287.html
[task 2022-11-16T18:54:19.218Z] 18:54:19     INFO - REFTEST TEST-LOAD | file:///builds/worker/workspace/build/tests/reftest/tests/dom/base/crashtests/831287.html | 268 / 3930 (6%)
[task 2022-11-16T18:54:19.322Z] 18:54:19     INFO - REFTEST TEST-PASS | dom/base/crashtests/831287.html | (LOAD ONLY)

Oh, interesting that there has been similar bug before, bug 1759252.
My patch did change some scheduling, so yes, it is possible it might have triggered the issue which caused also bug 1759252
Looking.

Flags: needinfo?(smaug)

The assertion in bug 1759252 is the same, at least in the most recent logs there from August.

[task 2022-08-19T06:52:54.295Z] 06:52:54 INFO - [Child 1836, Main Thread] ###!!! ASSERTION: Somehow there's stuff in the op queue.: 'NS_FAILED(mBroken) || mOpQueue.IsEmpty()', file /builds/worker/checkouts/gecko/parser/html/nsHtml5TreeOpExecutor.cpp:145
[task 2022-08-19T06:52:54.297Z] 06:52:54 INFO - #01: NS_DebugBreak [xpcom/base/nsDebugImpl.cpp:492]
[task 2022-08-19T06:52:54.298Z] 06:52:54 INFO - #02: nsHtml5TreeOpExecutor::~nsHtml5TreeOpExecutor() [parser/html/nsHtml5TreeOpExecutor.cpp:146]
[task 2022-08-19T06:52:54.299Z] 06:52:54 INFO - #03: nsHtml5TreeOpExecutor::~nsHtml5TreeOpExecutor() [parser/html/nsHtml5TreeOpExecutor.cpp:131]
[task 2022-08-19T06:52:54.301Z] 06:52:54 INFO - #04: SnowWhiteKiller::MaybeKillObject(SnowWhiteKiller::SnowWhiteObject&) [xpcom/base/nsCycleCollector.cpp:2419]

So we're deleting nsHtml5TreeOpExecutor when it has still ops. Does this ring immediately some bells, hsivonen?

Flags: needinfo?(hsivonen)
Severity: S4 → --
Priority: P5 → --
Component: DOM: Core & HTML → DOM: HTML Parser

There is also https://bugzilla.mozilla.org/show_bug.cgi?id=1746047 and
https://bugzilla.mozilla.org/show_bug.cgi?id=1709012

So, we may want to annotate the test to expect 0-1 assertions while investigating the root cause (which has been there for a long time)

https://bugzilla.mozilla.org/show_bug.cgi?id=766792 fixed one instance of this but there is something else happening too.

Duplicate of this bug: 1759252
Duplicate of this bug: 1790008
Duplicate of this bug: 1789237

I have a patch to remove the assertion in bug 1709012. Leaving this open instead of marking as a duplicate in order not to hide this from Treeherder just yet.

Flags: needinfo?(hsivonen)
Whiteboard: [stockwell disable-recommended]
Status: NEW → RESOLVED
Closed: 3 years ago
Duplicate of bug: 1709012
Resolution: --- → DUPLICATE
You need to log in before you can comment on or make changes to this bug.