Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Revert [253530@main] :has(:lang(~)) doesn't get invalidated #3461

Conversation

HerculesH
Copy link
Contributor

@HerculesH HerculesH commented Aug 18, 2022

5c847be

Revert [253530@main] :has(:lang(~)) doesn't get invalidated
https://bugs.webkit.org/show_bug.cgi?id=243172
rdar://98834362

Reverting commit due to causing bot opensource API test to crash on all queues.

Unreviewed revert.

* LayoutTests/fast/css/lang-pseudo-container-query-invalidation-expected.html: Removed.
* LayoutTests/fast/css/lang-pseudo-container-query-invalidation-xhtml-expected.html: Removed.
* LayoutTests/fast/css/lang-pseudo-container-query-invalidation-xhtml.xhtml: Removed.
* LayoutTests/fast/css/lang-pseudo-container-query-invalidation.html: Removed.
* Source/WebCore/css/SelectorCheckerTestFunctions.h:
(WebCore::matchesLangPseudoClass):
* Source/WebCore/dom/Element.cpp:
(WebCore::Element::attributeChanged):
(WebCore::Element::insertedIntoAncestor):
(WebCore::Element::removedFromAncestor):
(WebCore::effectiveLangFromAttribute): Deleted.
(WebCore::Element::effectiveLang const): Deleted.
* Source/WebCore/dom/Element.h:
* Source/WebCore/dom/ElementRareData.cpp:
* Source/WebCore/dom/ElementRareData.h:
(WebCore::ElementRareData::useTypes const):
(WebCore::ElementRareData::effectiveLang const): Deleted.
(WebCore::ElementRareData::setEffectiveLang): Deleted.
* Source/WebCore/dom/NodeRareData.h:
* Source/WebCore/style/PseudoClassChangeInvalidation.cpp:
(WebCore::Style::PseudoClassChangeInvalidation::computeInvalidation):
(WebCore::Style::PseudoClassChangeInvalidation::collectRuleSets):
* Source/WebCore/style/PseudoClassChangeInvalidation.h:
(WebCore::Style::PseudoClassChangeInvalidation::PseudoClassChangeInvalidation):
(): Deleted.

Canonical link: https://commits.webkit.org/253566@main

b2dc0ff

Misc iOS, tvOS & watchOS macOS Linux Windows
βœ… πŸ§ͺ style βœ… πŸ›  ios βœ… πŸ›  mac βœ… πŸ›  wpe βœ… πŸ›  πŸ§ͺ win
βœ… πŸ§ͺ bindings βœ… πŸ›  ios-sim βœ… πŸ›  mac-debug βœ… πŸ›  gtk βœ… πŸ›  wincairo
βœ… πŸ§ͺ webkitperl βœ… πŸ§ͺ ios-wk2 βœ… πŸ›  mac-AS-debug βœ… πŸ§ͺ gtk-wk2
βœ… πŸ§ͺ api-ios βœ… πŸ§ͺ api-mac βœ… πŸ§ͺ api-gtk
βœ… πŸ›  tv βœ… πŸ§ͺ mac-wk1
βœ… πŸ›  tv-sim βœ… πŸ§ͺ mac-wk2
βœ… πŸ›  watch βœ… πŸ§ͺ mac-AS-debug-wk2
βœ… πŸ›  πŸ§ͺ unsafe-merge βœ… πŸ›  watch-sim βœ… πŸ§ͺ mac-wk2-stress

@HerculesH HerculesH self-assigned this Aug 18, 2022
@HerculesH HerculesH added CSS Cascading Style Sheets implementation WebKit Nightly Build unsafe-merge-queue Applied to send a pull request to merge-queue, but skip building and testing labels Aug 18, 2022
https://bugs.webkit.org/show_bug.cgi?id=243172
rdar://98834362

Reverting commit due to causing bot opensource API test to crash on all queues.

Unreviewed revert.

* LayoutTests/fast/css/lang-pseudo-container-query-invalidation-expected.html: Removed.
* LayoutTests/fast/css/lang-pseudo-container-query-invalidation-xhtml-expected.html: Removed.
* LayoutTests/fast/css/lang-pseudo-container-query-invalidation-xhtml.xhtml: Removed.
* LayoutTests/fast/css/lang-pseudo-container-query-invalidation.html: Removed.
* Source/WebCore/css/SelectorCheckerTestFunctions.h:
(WebCore::matchesLangPseudoClass):
* Source/WebCore/dom/Element.cpp:
(WebCore::Element::attributeChanged):
(WebCore::Element::insertedIntoAncestor):
(WebCore::Element::removedFromAncestor):
(WebCore::effectiveLangFromAttribute): Deleted.
(WebCore::Element::effectiveLang const): Deleted.
* Source/WebCore/dom/Element.h:
* Source/WebCore/dom/ElementRareData.cpp:
* Source/WebCore/dom/ElementRareData.h:
(WebCore::ElementRareData::useTypes const):
(WebCore::ElementRareData::effectiveLang const): Deleted.
(WebCore::ElementRareData::setEffectiveLang): Deleted.
* Source/WebCore/dom/NodeRareData.h:
* Source/WebCore/style/PseudoClassChangeInvalidation.cpp:
(WebCore::Style::PseudoClassChangeInvalidation::computeInvalidation):
(WebCore::Style::PseudoClassChangeInvalidation::collectRuleSets):
* Source/WebCore/style/PseudoClassChangeInvalidation.h:
(WebCore::Style::PseudoClassChangeInvalidation::PseudoClassChangeInvalidation):
(): Deleted.

Canonical link: https://commits.webkit.org/253566@main
@webkit-early-warning-system webkit-early-warning-system force-pushed the eng/REGRESSION253530main--iOS-macOS--TestWebKitAPI-DocumentOrder-Positions-is-a-constant-crash branch from b2dc0ff to 5c847be Compare August 18, 2022 18:18
@webkit-early-warning-system webkit-early-warning-system merged commit 5c847be into WebKit:main Aug 18, 2022
@webkit-commit-queue
Copy link
Collaborator

Committed 253566@main (5c847be): https://commits.webkit.org/253566@main

Reviewed commits have been landed. Closing PR #3461 and removing active labels.

@webkit-commit-queue webkit-commit-queue removed the unsafe-merge-queue Applied to send a pull request to merge-queue, but skip building and testing label Aug 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CSS Cascading Style Sheets implementation
Projects
None yet
3 participants