Attachment 350099 Details for Bug 189738 – Fixes the bug

[patch] Fixes the bug

bug-189738-20180919021447.patch (text/plain), 1.55 KB, created by Ryosuke Niwa on 2018-09-19 02:14:47 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: Ryosuke Niwa

Created: 2018-09-19 02:14:47 PDT

Size: 1.55 KB

patch

obsolete

>Index: Source/WebCore/ChangeLog
>===================================================================
>--- Source/WebCore/ChangeLog	(revision 236165)
>+++ Source/WebCore/ChangeLog	(working copy)
>@@ -1,3 +1,15 @@
>+2018-09-19  Ryosuke Niwa  <rniwa@webkit.org>
>+
>+        REGRESSION(r235917): 2% regression in Dromaeo CSS selector on MacBookPro11,4
>+        https://bugs.webkit.org/show_bug.cgi?id=189738
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        The regression was caused by the regundant walk to the parent element. Removed it to fix the regression.
>+
>+        * cssjit/SelectorCompiler.cpp:
>+        (WebCore::SelectorCompiler::SelectorCodeGenerator::generateNthChildParentCheckAndRelationUpdate):
>+
> 2018-09-19  Philippe Normand <pnormand@igalia.com>
> 
>         [GStreamer] Add support for AV1 decoding
>Index: Source/WebCore/cssjit/SelectorCompiler.cpp
>===================================================================
>--- Source/WebCore/cssjit/SelectorCompiler.cpp	(revision 236165)
>+++ Source/WebCore/cssjit/SelectorCompiler.cpp	(working copy)
>@@ -3551,7 +3551,6 @@ void SelectorCodeGenerator::generateNthC
>     failureCases.append(m_assembler.branchTestPtr(Assembler::Zero, parentNode));
>     Assembler::Jump notElement = DOMJIT::branchTestIsElementFlagOnNode(m_assembler, Assembler::Zero, parentNode);
> 
>-    generateWalkToParentElement(failureCases, parentNode);
>     auto relation = fragmentMatchesRightmostOrAdjacentElement(fragment)
>         ? Style::Relation::ChildrenAffectedByForwardPositionalRules
>         : Style::Relation::DescendantsAffectedByForwardPositionalRules;

Actions: View | Formatted Diff | Diff

Attachments on bug 189738: 350099 | 350143