[Webkit-unassigned] [Bug 271921] New: SVG animate element's begin/end attribute set by js, should animate

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=271921

            Bug ID: 271921
           Summary: SVG animate element's begin/end attribute set by js,
                    should animate
           Product: WebKit
           Version: WebKit Nightly Build
          Hardware: Unspecified
                OS: Unspecified
            Status: NEW
          Keywords: BrowserCompat
          Severity: Normal
          Priority: P2
         Component: SVG
          Assignee: webkit-unassigned at lists.webkit.org
          Reporter: ahmad.saleem792 at gmail.com
                CC: sabouhallawa at apple.com, zimmermann at kde.org

Hi Team,

While going through Blink's commit, I came across another failing test case in Safari / WebKit.

Test Case: https://jsfiddle.net/6j1bx7dh/

^ Passes in Firefox Nightly 126 and Chrome Canary 125 - click on 'black' box and it will finish and show 'PASS' in other while 'Time Out' in Safari.

Blink's Commit: https://src.chromium.org/viewvc/blink?view=revision&revision=195296

If I do changes here, it does not work:

WebKit Source: https://searchfox.org/wubkat/rev/d113ccd097082b7db803786051566b6abd7ceffe/Source/WebCore/svg/animation/SVGSMILElement.cpp#553

Changes (in local build - but didn't fix the test case):

case AttributeNames::beginAttr:
        if (isConnected()) {
            connectConditions();
            beginListChanged(elapsed());
        }
        break;
    case AttributeNames::endAttr:
        if (isConnected()) {
            connectConditions();
            endListChanged(elapsed());
        }
        break;
    default:
        break;

__

It does not work.

Just raising so we can track it.

Thanks!

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-unassigned/attachments/20240330/689ed1a2/attachment-0001.htm>