tree be4174cf2a611882e2322e9c87597d5804780c23
parent 03365e3849fbdb7c47e680c45be477ec120c73b9
author thefrog <thefrog@chromium.org> 1687282798 -0700
committer Blink WPT Bot <blink-w3c-test-autoroller@chromium.org> 1687287625 -0700

Revert "[text-fragment] Attempt search for dynamic content"

This reverts commit 6bd222a5ff1b875ac739df526a59799ded6b6676.

Reason for revert: Likely causing recent builder failures.
Example failure: https://ci.chromium.org/ui/p/chromium/builders/ci/Linux%20Tests/131991/overview
Failing: external/wpt/scroll-to-text-fragment/iframes.sub.html

Original change's description:
> [text-fragment] Attempt search for dynamic content
>
> Text fragments currently perform up to two searches, one when document
> parsing completes and a second attempt when document load completes (if
> load wasn't completed at parse complete time). However, pages, often
> load content after document load, when content is "dynamically loaded".
> One popular example is Mobile Wikipedia, which adds `hidden=until-found`
> on collapsed sections in idle tasks after load. This meant
> text-fragments couldn't target pages like these.
>
> This CL attempts to make text fragments work on dynamically loaded pages
> by performing a third attempt, if needed. If all directives haven't
> matched at load time, a delayed task is scheduled for 500ms that will
> request attachment on unmatched directives. TextFragmentAnchor listens
> for relevant changes in the DOM and reschedules this task each time a
> change is made. At 3000ms it gives up and performs the search to avoid
> waiting forever.
>
> At a high level, this CL tries to separate the state tracking of actions
> performed for the first matching directive from the state tracking for
> running multiple searches. This is done by introducing a new iteration_
> enum tracking the latter.
>
> Change-Id: Ie01012935152b12f2d724469010128c7dd34afb8
> Bug: 963045
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4586061
> Reviewed-by: Vladimir Levin <vmpstr@chromium.org>
> Commit-Queue: David Bokan <bokan@chromium.org>
> Cr-Commit-Position: refs/heads/main@{#1160040}

Bug: 963045
Change-Id: I64babfbb5ac1849f3ea3c7bf3fdb41b12390b657
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4629649
Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
Owners-Override: thefrog <thefrog@chromium.org>
Commit-Queue: thefrog <thefrog@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1160118}