Closed Bug 1250538 Opened 9 years ago Closed 9 years ago

"ASSERTION: shouldn't have observed an animation being removed twice" when running devtools animation-inspector mochitests

Categories

(Core :: DOM: Animation, defect)

Product:
Core
Component:
DOM: Animation
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED DUPLICATE of bug 1189015

People

(Reporter: pbro, Unassigned)

Details

I just realized today that when running the tests in devtools/client/animationinspector/test/ on windows debug e10s (not sure about other platforms/settings), a *lot* of exceptions are displayed in the logs. See this try push for example: https://treeherder.mozilla.org/#/jobs?repo=try&revision=898af952fc11&group_state=expanded&selectedJob=17028740 Pretty much every test that runs triggers the following assertion: 05:59:08 INFO - [Child 340] ###!!! ASSERTION: shouldn't have observed an animation being removed twice: 'Not Reached', file c:\builds\moz2_slave\try-w32-d-00000000000000000000\build\src\dom\base\nsDOMMutationObserver.h, line 859 05:59:08 INFO - #01: nsAnimationReceiver::RecordAnimationMutation(mozilla::dom::Animation *,nsAnimationReceiver::AnimationMutation) [dom/base/nsDOMMutationObserver.cpp:416] 05:59:08 INFO - #02: nsAnimationReceiver::AnimationRemoved(mozilla::dom::Animation *) [dom/base/nsDOMMutationObserver.cpp:459] 05:59:08 INFO - #03: nsNodeUtils::AnimationRemoved(mozilla::dom::Animation *) [dom/base/nsNodeUtils.cpp:290] 05:59:08 INFO - #04: mozilla::dom::Animation::UpdateRelevance() [dom/animation/Animation.cpp:689] 05:59:08 INFO - #05: mozilla::dom::CSSAnimation::UpdateTiming(mozilla::dom::Animation::SeekFlag,mozilla::dom::Animation::SyncNotifyFlag) [layout/style/nsAnimationManager.cpp:269] 05:59:08 INFO - #06: mozilla::dom::Animation::Tick() [dom/animation/Animation.cpp:519] 05:59:08 INFO - #07: mozilla::dom::CSSAnimation::Tick() [layout/style/nsAnimationManager.cpp:123] 05:59:08 INFO - #08: nsRefreshDriver::Tick(__int64,mozilla::TimeStamp) [layout/base/nsRefreshDriver.cpp:1691] 05:59:08 INFO - #09: mozilla::RefreshDriverTimer::TickDriver(nsRefreshDriver *,__int64,mozilla::TimeStamp) [layout/base/nsRefreshDriver.cpp:273] 05:59:08 INFO - #10: mozilla::RefreshDriverTimer::TickRefreshDrivers(__int64,mozilla::TimeStamp,nsTArray<RefPtr<nsRefreshDriver> > &) [layout/base/nsRefreshDriver.cpp:244] 05:59:08 INFO - #11: mozilla::RefreshDriverTimer::Tick(__int64,mozilla::TimeStamp) [layout/base/nsRefreshDriver.cpp:265] 05:59:08 INFO - #12: mozilla::VsyncRefreshDriverTimer::RunRefreshDrivers(mozilla::TimeStamp) [layout/base/nsRefreshDriver.cpp:587] 05:59:08 INFO - #13: mozilla::VsyncRefreshDriverTimer::RefreshDriverVsyncObserver::TickRefreshDriver(mozilla::TimeStamp) [layout/base/nsRefreshDriver.cpp:508] 05:59:08 INFO - #14: mozilla::VsyncRefreshDriverTimer::RefreshDriverVsyncObserver::NotifyVsync(mozilla::TimeStamp) [layout/base/nsRefreshDriver.cpp:427] 05:59:08 INFO - #15: mozilla::layout::VsyncChild::RecvNotify(mozilla::TimeStamp const &) [layout/ipc/VsyncChild.cpp:67] 05:59:08 INFO - #16: mozilla::layout::PVsyncChild::OnMessageReceived(IPC::Message const &) [obj-firefox/ipc/ipdl/PVsyncChild.cpp:233] 05:59:08 INFO - #17: mozilla::ipc::PBackgroundChild::OnMessageReceived(IPC::Message const &) [obj-firefox/ipc/ipdl/PBackgroundChild.cpp:1721] 05:59:08 INFO - #18: mozilla::ipc::MessageChannel::DispatchAsyncMessage(IPC::Message const &) [ipc/glue/MessageChannel.cpp:1445] 05:59:08 INFO - #19: mozilla::ipc::MessageChannel::DispatchMessageW(IPC::Message const &) [ipc/glue/MessageChannel.cpp:1385] 05:59:08 INFO - #20: mozilla::ipc::MessageChannel::OnMaybeDequeueOne() [ipc/glue/MessageChannel.cpp:1355] 05:59:08 INFO - #21: MessageLoop::RunTask(Task *) [ipc/chromium/src/base/message_loop.cc:365] 05:59:08 INFO - #22: MessageLoop::DeferOrRunPendingTask(MessageLoop::PendingTask const &) [ipc/chromium/src/base/message_loop.cc:375] 05:59:08 INFO - #23: MessageLoop::DoWork() [ipc/chromium/src/base/message_loop.cc:459] 05:59:08 INFO - #24: mozilla::ipc::DoWorkRunnable::Run() [ipc/glue/MessagePump.cpp:221] 05:59:08 INFO - #25: nsThread::ProcessNextEvent(bool,bool *) [xpcom/threads/nsThread.cpp:1018] 05:59:08 INFO - #26: NS_ProcessNextEvent(nsIThread *,bool) [xpcom/glue/nsThreadUtils.cpp:297] 05:59:08 INFO - #27: mozilla::ipc::MessagePump::Run(base::MessagePump::Delegate *) [ipc/glue/MessagePump.cpp:127] 05:59:08 INFO - #28: mozilla::ipc::MessagePumpForChildProcess::Run(base::MessagePump::Delegate *) [ipc/glue/MessagePump.cpp:289] 05:59:08 INFO - #29: MessageLoop::RunInternal() [ipc/chromium/src/base/message_loop.cc:234] 05:59:08 INFO - #30: MessageLoop::RunHandler() [ipc/chromium/src/base/message_loop.cc:228] 05:59:08 INFO - #31: MessageLoop::Run() [ipc/chromium/src/base/message_loop.cc:202] 05:59:08 INFO - #32: nsBaseAppShell::Run() [widget/nsBaseAppShell.cpp:158] 05:59:08 INFO - #33: nsAppShell::Run() [widget/windows/nsAppShell.cpp:259] 05:59:08 INFO - #34: XRE_RunAppShell [toolkit/xre/nsEmbedFunctions.cpp:789] 05:59:08 INFO - #35: mozilla::ipc::MessagePumpForChildProcess::Run(base::MessagePump::Delegate *) [ipc/glue/MessagePump.cpp:259] 05:59:08 INFO - #36: MessageLoop::RunInternal() [ipc/chromium/src/base/message_loop.cc:234] 05:59:08 INFO - #37: MessageLoop::RunHandler() [ipc/chromium/src/base/message_loop.cc:228] 05:59:08 INFO - #38: MessageLoop::Run() [ipc/chromium/src/base/message_loop.cc:202] 05:59:08 INFO - #39: XRE_InitChildProcess [toolkit/xre/nsEmbedFunctions.cpp:629] 05:59:08 INFO - #40: content_process_main(int,char * * const) [ipc/contentproc/plugin-container.cpp:238] 05:59:08 INFO - #41: wmain [toolkit/xre/nsWindowsWMain.cpp:138] 05:59:08 INFO - #42: __tmainCRTStartup [f:/dd/vctools/crt/crtw32/startup/crt0.c:255] 05:59:08 INFO - #43: kernel32.dll + 0x53c45 05:59:08 INFO - #44: ntdll.dll + 0x637f5 05:59:08 INFO - #45: ntdll.dll + 0x637c8
Thank you, Patrick. This assertion might be more frequent than before. I have to revisit bug 1189015.
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → DUPLICATE
You need to log in before you can comment on or make changes to this bug.