New issue
Advanced search Search tips

Issue 729636 link

Starred by 1 user

Issue metadata

Status: Untriaged
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug

Blocking:
issue 745501



Sign in to add a comment

Resolve inconsistency between DOM and Flat tree position canonicalization

Project Member Reported by xiaoche...@chromium.org, Jun 5 2017

Issue description

There are some cases where we canonicalize a Position to a non-null VisiblePosition, but the position in fact does't participate in the flat tree, and hence, should be canonicalized to null.

Example:

<div id=host>
  <span id=anchor>foo</span>
  bar
</div>

where DIV#host is a shadow host, and span#anchor is not distributed into the shadow DOM.

In this case, Position (SPAN#anchor, Before/AfterAnchor) doesn't participate in the flat tree, and hence, should be canonicalized to null.

We need to fully understand the implementation of |CanonicalPosition| to resolve this issue.
 
 Issue 728441  has been merged into this issue.
Found another case reproducing the inconsistency, but in the other way that flat tree canonicalization gives non-null result but DOM tree canonicalization gives null.

Use the test case in https://bugs.chromium.org/p/chromium/issues/detail?id=745501#c11.

After triple clicking under the table, the frame selection is set to
start = TABLE@0
end = TABLE@AfterAnchor

In selection canonicalization, we get VSInFlatTree being null but VSInDOMTree not null.
Blocking: 745501

Comment 4 by yosin@chromium.org, Jan 10 2018

Labels: Pri-3
Project Member

Comment 5 by sheriffbot@chromium.org, Jan 10

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

Sign in to add a comment