Closed Bug 1670327 Opened 4 years ago Closed 4 years ago

Assertion failure: IsAncestorFrameCrossDoc(aAncestor.mFrame, aFrame) (Fix the caller), at src/layout/base/nsLayoutUtils.cpp:2486

Categories

(Core :: Layout, defect)

defect

Tracking

()

VERIFIED FIXED
83 Branch
Tracking Status
firefox-esr78 --- unaffected
firefox81 --- unaffected
firefox82 --- unaffected
firefox83 --- verified

People

(Reporter: tsmith, Assigned: emilio)

References

(Blocks 1 open bug, Regression)

Details

(Keywords: assertion, regression, testcase, Whiteboard: [bugmon:bisected,confirmed], [wptsync upstream])

Attachments

(3 files)

Attached file testcase.html

Assertion failure: IsAncestorFrameCrossDoc(aAncestor.mFrame, aFrame) (Fix the caller), at src/layout/base/nsLayoutUtils.cpp:2486

#0 0x7f95128ec39e in nsLayoutUtils::TransformFrameRectToAncestor(nsIFrame const*, nsRect const&, mozilla::RelativeTo, bool*, mozilla::Maybe<mozilla::gfx::Matrix4x4TypedFlagged<mozilla::gfx::UnknownUnits, mozilla::gfx::UnknownUnits> >*, bool, nsIFrame**) src/layout/base/nsLayoutUtils.cpp:2485:3
#1 0x7f950fa8039a in TransformFrameRectToAncestor src/layout/base/nsLayoutUtils.h:804:12
#2 0x7f950fa8039a in ComputeTheIntersection src/dom/base/DOMIntersectionObserver.cpp:379:7
#3 0x7f950fa8039a in mozilla::dom::DOMIntersectionObserver::Update(mozilla::dom::Document*, double) src/dom/base/DOMIntersectionObserver.cpp:604:26
#4 0x7f950fc2b231 in mozilla::dom::Document::UpdateIntersectionObservations(mozilla::TimeStamp) src/dom/base/Document.cpp:15151:17
#5 0x7f951282e627 in nsRefreshDriver::UpdateIntersectionObservations(mozilla::TimeStamp) src/layout/base/nsRefreshDriver.cpp:1811:10
#6 0x7f951282cf32 in nsRefreshDriver::Tick(mozilla::layers::BaseTransactionId<mozilla::VsyncIdType>, mozilla::TimeStamp) src/layout/base/nsRefreshDriver.cpp:2267:3
#7 0x7f9512834931 in TickDriver src/layout/base/nsRefreshDriver.cpp:374:13
#8 0x7f9512834931 in mozilla::RefreshDriverTimer::TickRefreshDrivers(mozilla::layers::BaseTransactionId<mozilla::VsyncIdType>, mozilla::TimeStamp, nsTArray<RefPtr<nsRefreshDriver> >&) src/layout/base/nsRefreshDriver.cpp:353:7
#9 0x7f951283481c in mozilla::RefreshDriverTimer::Tick(mozilla::layers::BaseTransactionId<mozilla::VsyncIdType>, mozilla::TimeStamp) src/layout/base/nsRefreshDriver.cpp:368:5
#10 0x7f9512833dc8 in RunRefreshDrivers src/layout/base/nsRefreshDriver.cpp:829:5
#11 0x7f9512833dc8 in mozilla::VsyncRefreshDriverTimer::RefreshDriverVsyncObserver::TickRefreshDriver(mozilla::layers::BaseTransactionId<mozilla::VsyncIdType>, mozilla::TimeStamp) src/layout/base/nsRefreshDriver.cpp:747:16
#12 0x7f95128336d0 in mozilla::VsyncRefreshDriverTimer::RefreshDriverVsyncObserver::NotifyParentProcessVsync() src/layout/base/nsRefreshDriver.cpp:649:7
#13 0x7f9512833149 in mozilla::VsyncRefreshDriverTimer::RefreshDriverVsyncObserver::NotifyVsync(mozilla::VsyncEvent const&) src/layout/base/nsRefreshDriver.cpp:570:9
#14 0x7f9512bf08b7 in mozilla::layout::VsyncChild::RecvNotify(mozilla::VsyncEvent const&) src/layout/ipc/VsyncChild.cpp:55:16
#15 0x7f950eed4d65 in mozilla::layout::PVsyncChild::OnMessageReceived(IPC::Message const&) /builds/worker/workspace/obj-build/ipc/ipdl/PVsyncChild.cpp:187:54
#16 0x7f950ec8b3ed in mozilla::ipc::PBackgroundChild::OnMessageReceived(IPC::Message const&) /builds/worker/workspace/obj-build/ipc/ipdl/PBackgroundChild.cpp:6080:32
#17 0x7f950e95393e in mozilla::ipc::MessageChannel::DispatchAsyncMessage(mozilla::ipc::ActorLifecycleProxy*, IPC::Message const&) src/ipc/glue/MessageChannel.cpp:2150:25
#18 0x7f950e9500ff in mozilla::ipc::MessageChannel::DispatchMessage(IPC::Message&&) src/ipc/glue/MessageChannel.cpp:2074:9
#19 0x7f950e951506 in mozilla::ipc::MessageChannel::RunMessage(mozilla::ipc::MessageChannel::MessageTask&) src/ipc/glue/MessageChannel.cpp:1922:3
#20 0x7f950e95212b in mozilla::ipc::MessageChannel::MessageTask::Run() src/ipc/glue/MessageChannel.cpp:1953:13
#21 0x7f950e04f75f in mozilla::RunnableTask::Run() src/xpcom/threads/TaskController.cpp:245:16
#22 0x7f950e04cbba in mozilla::TaskController::DoExecuteNextTaskOnlyMainThreadInternal(mozilla::detail::BaseAutoLock<mozilla::Mutex&> const&) src/xpcom/threads/TaskController.cpp:515:26
#23 0x7f950e04ba94 in mozilla::TaskController::ExecuteNextTaskOnlyMainThreadInternal(mozilla::detail::BaseAutoLock<mozilla::Mutex&> const&) src/xpcom/threads/TaskController.cpp:374:15
#24 0x7f950e04bc47 in mozilla::TaskController::ProcessPendingMTTask(bool) src/xpcom/threads/TaskController.cpp:171:36
#25 0x7f950e052ea9 in operator() src/xpcom/threads/TaskController.cpp:88:37
#26 0x7f950e052ea9 in mozilla::detail::RunnableFunction<mozilla::TaskController::InitializeInternal()::$_4>::Run() /builds/worker/workspace/obj-build/dist/include/nsThreadUtils.h:577:5
#27 0x7f950e064e9f in nsThread::ProcessNextEvent(bool, bool*) src/xpcom/threads/nsThread.cpp:1234:14
#28 0x7f950e06a95a in NS_ProcessNextEvent(nsIThread*, bool) src/xpcom/threads/nsThreadUtils.cpp:513:10
#29 0x7f950e959144 in mozilla::ipc::MessagePump::Run(base::MessagePump::Delegate*) src/ipc/glue/MessagePump.cpp:109:5
#30 0x7f950e8cb913 in MessageLoop::RunInternal() src/ipc/chromium/src/base/message_loop.cc:334:10
#31 0x7f950e8cb82d in RunHandler src/ipc/chromium/src/base/message_loop.cc:327:3
#32 0x7f950e8cb82d in MessageLoop::Run() src/ipc/chromium/src/base/message_loop.cc:309:3
#33 0x7f9512587de8 in nsBaseAppShell::Run() src/widget/nsBaseAppShell.cpp:137:27
#34 0x7f9513d7a1e3 in XRE_RunAppShell() src/toolkit/xre/nsEmbedFunctions.cpp:913:20
#35 0x7f950e959f59 in mozilla::ipc::MessagePumpForChildProcess::Run(base::MessagePump::Delegate*) src/ipc/glue/MessagePump.cpp:237:9
#36 0x7f950e8cb913 in MessageLoop::RunInternal() src/ipc/chromium/src/base/message_loop.cc:334:10
#37 0x7f950e8cb82d in RunHandler src/ipc/chromium/src/base/message_loop.cc:327:3
#38 0x7f950e8cb82d in MessageLoop::Run() src/ipc/chromium/src/base/message_loop.cc:309:3
#39 0x7f9513d79dc8 in XRE_InitChildProcess(int, char**, XREChildData const*) src/toolkit/xre/nsEmbedFunctions.cpp:744:34
#40 0x5592aed87917 in content_process_main src/browser/app/../../ipc/contentproc/plugin-container.cpp:56:28
#41 0x5592aed87917 in main src/browser/app/nsBrowserApp.cpp:304:18
#42 0x7f952292b0b2 in __libc_start_main /build/glibc-ZN95T4/glibc-2.31/csu/../csu/libc-start.c:308:16
#43 0x5592aed656c9 in _start (/home/worker/builds/m-c-20201008094950-fuzzing-debug/firefox-bin+0x176c9)
Flags: in-testsuite?

Bugmon Analysis:
Verified bug as reproducible on mozilla-central 20201009214545-cad2c1678593.
The bug appears to have been introduced in the following build range:

Start: a3e34680e841fb4e55a5cf17622daf62eb1c673f (20201002012040)
End: c54385f18d19901f0846dea0c4cee45676c8cf97 (20201002021059)
Pushlog: https://hg.mozilla.org/integration/autoland/pushloghtml?fromchange=a3e34680e841fb4e55a5cf17622daf62eb1c673f&tochange=c54385f18d19901f0846dea0c4cee45676c8cf97

Whiteboard: [bugmon:bisected,confirmed]
Flags: needinfo?(emilio)
Regressed by: 1668156
Has Regression Range: --- → yes
Assignee: nobody → emilio

(That doesn't rely on the PiP intersection observer).

Two spec issues here.

https://github.com/w3c/IntersectionObserver/issues/457: I think this is
just a spec bug and I've made us match other browsers, but since the
tests don't match the spec for now I've added them as .tentative.html

https://github.com/w3c/IntersectionObserver/issues/456: I've aligned
with WebKit here. There was a (disabled) test for this which tests
chrome behavior and which after this patch shouldn't be flaky. This is
what was causing the assertion.

Flags: needinfo?(emilio)
Flags: needinfo?(emilio)

Set release status flags based on info from the regressing bug 1668156

Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → 83 Branch

Bugmon Analysis:
Verified bug as fixed on rev mozilla-central 20201012152719-61618dc9e8bf.
Removing bugmon keyword as no further action possible.
Please review the bug and re-add the keyword for further analysis.

Status: RESOLVED → VERIFIED
Keywords: bugmon
Whiteboard: [bugmon:bisected,confirmed] → [bugmon:bisected,confirmed], [wptsync upstream]
Upstream PR merged by moz-wptsync-bot
Regressions: 1682915
Duplicate of this bug: 1497420
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: