Title: [284755] trunk/Source/WebCore
- Revision
- 284755
- Author
- [email protected]
- Date
- 2021-10-24 07:50:04 -0700 (Sun, 24 Oct 2021)
Log Message
[css-contain] Support contain:style for counters
https://bugs.webkit.org/show_bug.cgi?id=226458
Patch by Rob Buis <[email protected]> on 2021-10-24
Reviewed by Antti Koivisto.
Remove unneeded logic from r284642 change.
* rendering/RenderCounter.cpp:
(WebCore::previousInPreOrder):
Modified Paths
Diff
Modified: trunk/Source/WebCore/ChangeLog (284754 => 284755)
--- trunk/Source/WebCore/ChangeLog 2021-10-24 04:31:56 UTC (rev 284754)
+++ trunk/Source/WebCore/ChangeLog 2021-10-24 14:50:04 UTC (rev 284755)
@@ -1,3 +1,15 @@
+2021-10-24 Rob Buis <[email protected]>
+
+ [css-contain] Support contain:style for counters
+ https://bugs.webkit.org/show_bug.cgi?id=226458
+
+ Reviewed by Antti Koivisto.
+
+ Remove unneeded logic from r284642 change.
+
+ * rendering/RenderCounter.cpp:
+ (WebCore::previousInPreOrder):
+
2021-10-23 Rob Buis <[email protected]>
Null check in traverseNodesForSerialization
Modified: trunk/Source/WebCore/rendering/RenderCounter.cpp (284754 => 284755)
--- trunk/Source/WebCore/rendering/RenderCounter.cpp 2021-10-24 04:31:56 UTC (rev 284754)
+++ trunk/Source/WebCore/rendering/RenderCounter.cpp 2021-10-24 14:50:04 UTC (rev 284755)
@@ -79,18 +79,11 @@
Element* previous = ElementTraversal::previousIncludingPseudo(*renderer.element());
Element* styleContainAncestor = ancestorStyleContainmentObject(*renderer.element());
- while (true) {
- while (previous && !previous->renderer())
- previous = ElementTraversal::previousIncludingPseudo(*previous, styleContainAncestor);
- if (!previous)
- return nullptr;
- Element* previousStyleContainAncestor = ancestorStyleContainmentObject(*previous);
- if (previousStyleContainAncestor == styleContainAncestor)
- return previous->renderer();
- if (!previousStyleContainAncestor)
- return nullptr;
- previous = previousStyleContainAncestor;
- }
+ while (previous && !previous->renderer())
+ previous = ElementTraversal::previousIncludingPseudo(*previous, styleContainAncestor);
+ if (!previous)
+ return nullptr;
+ return previous->renderer();
}
static inline Element* parentOrPseudoHostElement(const RenderElement& renderer)
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
