| Summary: | Layout tests outside of the WPT import should not use resources from it | ||||||||
|---|---|---|---|---|---|---|---|---|---|
| Product: | WebKit | Reporter: | Carlos Alberto Lopez Perez <clopez> | ||||||
| Component: | Tools / Tests | Assignee: | Carlos Alberto Lopez Perez <clopez> | ||||||
| Status: | RESOLVED FIXED | ||||||||
| Severity: | Normal | CC: | cdumez, dbarton, ews-watchlist, fred.wang, koivisto, rniwa, simon.fraser, webkit-bug-importer, youennf | ||||||
| Priority: | P2 | Keywords: | InRadar | ||||||
| Version: | WebKit Nightly Build | ||||||||
| Hardware: | Unspecified | ||||||||
| OS: | Unspecified | ||||||||
| See Also: | https://bugs.webkit.org/show_bug.cgi?id=212613 | ||||||||
| Attachments: |
|
||||||||
|
Description
Carlos Alberto Lopez Perez
2020-06-02 14:11:10 PDT
Created attachment 400858 [details]
Patch
Comment on attachment 400858 [details]
Patch
Do we know if these tests should be upstreamed to WPT or already have equivalent WPT tests?
"try resources" ? (In reply to youenn fablet from comment #2) > Comment on attachment 400858 [details] > Patch > > Do we know if these tests should be upstreamed to WPT or already have > equivalent WPT tests? I have checked, this is what I found: > canvas/philip/tests/2d.drawImage.incomplete.emptysrc.html > canvas/philip/tests/2d.drawImage.incomplete.nosrc.html > canvas/philip/tests/2d.drawImage.incomplete.removedsrc.html > canvas/philip/tests/2d.pattern.image.incomplete.emptysrc.html > canvas/philip/tests/2d.pattern.image.incomplete.removedsrc.html ^^ All of this are duplicated and can be removed in favor of the ones inside LayoutTests/imported/w3c/web-platform-tests/html/canvas/element > fast/css/shadow-parts/exportparts-syntax.html > fast/css/shadow-parts/invalidation-class-before-after.html > fast/css/shadow-parts/invalidation-class-descendant-combinator-export.html > fast/css/shadow-parts/invalidation-class-descendant-combinator.html > fast/css/shadow-parts/invalidation-class-sibling-combinator-export.html > fast/css/shadow-parts/invalidation-class-sibling-combinator.html ^^ I couldn't find duplicates for this tests in WPT. And according to git log, all of them were added by Antti (cc'ing). Maybe this ones can be upstreamed to WPT. > fast/custom-elements/enqueue-custom-element-callback-reactions-inside-another-callback.html > fast/custom-elements/perform-microtask-checkpoint-before-construction.html > fast/custom-elements/throw-on-dynamic-markup-insertion-counter-construct.html > fast/custom-elements/throw-on-dynamic-markup-insertion-counter-reactions.html ^^ This 4 are upstreamed in WPT, but we still have not re-imported the WPT ones back. So we can't still de-duplicate them. > fast/custom-elements/DOMImplementation-createDocument.html > fast/custom-elements/adopting-from-frameless-document.html > fast/custom-elements/disconnected-callback-in-detached-iframe.html > fast/custom-elements/document-createElementNS.html > fast/custom-elements/reactions-for-webkit-extensions.html ^^ I couldn't find this on WPT. Maybe some of them can be upstreamed > fast/images/sizes-dynamic-001.html > fast/images/sizes-dynamic-002.html ^^ This two ones are upstreamed to WPT, and we have reimported them back. But the WPT version of this tests its not working for us due to we still don't support reftest-wait. See bug 186045 > fast/shadow-dom/capturing-and-bubbling-event-listeners-across-shadow-trees.html ^ This one its upsteamed to WPT and we have re-imported it back already. We can de-duplicate it. > mathml/non-core/frac-linethickness-001.html > mathml/non-core/frac-linethickness-002.html > mathml/non-core/lengths-1.html > mathml/non-core/lengths-2.html > mathml/non-core/lengths-3.html ^ I understand this are upstreamed to WPT but we want to keep the older version for WebKit. See r249713 > mathml/presentation/radical-bar-visibility.html ^ This one its not upstreamed to WPT, it was added in r193743 ... not sure if it makes sense to do it or not. CC'ed Fred. Created attachment 401534 [details]
Patch
Updated patch (for landing) de-duplicating fast/shadow-dom/capturing-and-bubbling-event-listeners-across-shadow-trees.html and canvas/philip/tests/*
Committed r262840: <https://trac.webkit.org/changeset/262840> All reviewed patches have been landed. Closing bug and clearing flags on attachment 401534 [details]. |