New issue
Advanced search Search tips

Issue 769340 link

Starred by 1 user
Status: Available
Owner: ----
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Android , Windows , Chrome , Mac
Pri: 3
Type: Bug



Sign in to add a comment

HitTestResult innerNode is not always also in the list result

Project Member Reported by schenney@chromium.org, Sep 27 2017 
It seems like an error to have hit test results that report an InnerNode that was hit yet that node does not appear as one of the hit nodes in the list based result for touch.

For touch adjustment that might be expected, but I would not expect it to happen otherwise. Investigate as these might be sources of bugs. The following tests assert if you check for this situation in PaintLayer::HitTestContents.

fast/events/event-hit-testing-fallback-to-iframe.html
virtual/scroll_customization/fast/events/touch/gesture/gesture-tap-frame-scrollbar.html
touchadjustment/disabled-formelements.html
touchadjustment/pseudo-element.html
touchadjustment/iframe.html
touchadjustment/touch-links-active.html
compositing/gestures/gesture-tapHighlight-no-graphics-layer-region-based-multicol.html
compositing/gestures/gesture-tapHighlight-simple-background.html
compositing/gestures/gesture-tapHighlight-imagemap.html
compositing/gestures/gesture-tapHighlight-simple-navigate.html
compositing/gestures/gesture-tapHighlight-simple-cancel.html
compositing/gestures/gesture-tapHighlight-simple-body.html
compositing/gestures/gesture-tapHighlight-simple-cancel2.html
compositing/gestures/gesture-tapHighlight-pixel-rotated-div.html
compositing/gestures/gesture-tapHighlight-simple-longPress.html
external/wpt/pointerevents/pointerevent_attributes_nohover_pointers-manual.html
fast/events/touch/gesture/gesture-tap-frame-scrollbar.html

Comment 1 by schenney@chromium.org, Sep 27 2017 

Code to add to detect the case.

#if DCHECK_IS_ON()
  else if (result.GetHitTestRequest().ListBased()) {
    // The innerNode must also appear in a list result
    DCHECK(result.ListBasedTestResult().Contains(result.InnerNode()));

  }
#endif
Project Member

Comment 2 by sheriffbot@chromium.org, Sep 28

Labels: Hotlist-Recharge-Cold
Status: Untriaged (was: Available)
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue.

Sorry for the inconvenience if the bug really should have been left as Available.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

Comment 3 by schenney@chromium.org, Sep 28

Status: Available (was: Untriaged)
►

Sign in to add a comment