Open Bug 1894162 Opened 2 years ago Updated 1 year ago

Assertion failure: touch->mOriginalTarget, at layout/base/TouchManager.cpp:192

Categories

(Core :: DOM: UI Events & Focus Handling, defect)

Product:
Core
Component:
DOM: UI Events & Focus Handling
Type:
defect

Tracking

()

People

(Reporter: hiro, Unassigned, NeedInfo)

References

(
URL
)

Details

STR

  1. Open https://github.com/webcompat/web-bugs/issues/134949 in RDM on a debug build
  2. Enable touch simulation
  3. Touch "Mar 21" in the first comment header

The null mOriginalTarget comes from this SetTouchTarget call in TouchManager::SetupTarget. In this specific case, targetContent is a text node and its parent is a shadow root, thus targetContent->GetAsElementOrParentElement() is nullptr unfortunately. Should we use the shadow host in such cases?

See Also: → 1887659

This is an assert you added, do you have an idea of what's going on?

Flags: needinfo?(smaug)

Yeah, I guess we should use host.

Flags: needinfo?(smaug)

I'm not sure that I follow?

Flags: needinfo?(smaug)
Severity: -- → S3
You need to log in before you can comment on or make changes to this bug.