querySelector*(":target") doesn't work in 'load' event handler |
||||
Issue descriptionChrome Version: 59 canary OS: All but iOS What steps will reproduce the problem? (1) Open http://w3c-test.org/dom/nodes/ParentNode-querySelector-All.html What is the expected result? No FAILs in :target-related tests What happens instead? There are FAILs in :target-related tests Please use labels and text to provide additional information. We dispatches load event, then call Document::SetCSSTarget(). So :target doesn't work in load event handler. Issue 569511 resolves this issue. Edge, Firefox, and Safari work correctly.
,
Apr 14 2017
Issue 549590 is a related but different issue.
,
Apr 14 2017
According to the HTML standard, It seems dispatching "load" event then "scroll to the fragment" matches to the standard. The test depends on rendering timing, and I found the test sometimes produced a different result. https://test-results.appspot.com/dashboards/flakiness_dashboard.html#testType=webkit_tests&tests=ParentNode-querySelector-All.html
,
Apr 18 2017
,
Apr 18 2017
Keeping to a single component - this looks like the Loader team are handling this.
,
Apr 18 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/a377281d4100e91855312b96e98cea4c8fb34791 commit a377281d4100e91855312b96e98cea4c8fb34791 Author: tkent <tkent@chromium.org> Date: Tue Apr 18 09:08:25 2017 [dom] Fix flakiness of ParentNode-querySelector-All*.*. :target doesn't work before the first page rendering in some browsers. This tries to fix it by running tests in an animation frame. BUG= 711575 Review-Url: https://codereview.chromium.org/2821243002 Cr-Commit-Position: refs/heads/master@{#465182} [modify] https://crrev.com/a377281d4100e91855312b96e98cea4c8fb34791/third_party/WebKit/LayoutTests/external/wpt/dom/nodes/ParentNode-querySelector-All-xht-expected.txt [modify] https://crrev.com/a377281d4100e91855312b96e98cea4c8fb34791/third_party/WebKit/LayoutTests/external/wpt/dom/nodes/ParentNode-querySelector-All-xht.xht [modify] https://crrev.com/a377281d4100e91855312b96e98cea4c8fb34791/third_party/WebKit/LayoutTests/external/wpt/dom/nodes/ParentNode-querySelector-All.html
,
Apr 18 2017
|
||||
►
Sign in to add a comment |
||||
Comment 1 by tkent@chromium.org
, Apr 14 2017