Closed Bug 1250538 Opened 8 years ago Closed 8 years ago

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

Categories

(Core :: DOM: Animation, defect)

defect
Not set
normal

Tracking

()

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: 8 years ago
Resolution: --- → DUPLICATE
You need to log in before you can comment on or make changes to this bug.