New issue
Advanced search Search tips

Issue 725469 link

Starred by 3 users

Issue metadata

Status: Verified
Owner:
Closed: May 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Android , Windows , Chrome , Mac , Fuchsia
Pri: 2
Type: Bug-Regression



Sign in to add a comment

SMIL animation marker-end property fails to animate

Reported by j...@jacobeanrnd.com, May 23 2017

Issue description

UserAgent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_12_0) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/57.0.2987.133 Safari/537.36

Steps to reproduce the problem:
1.  Animate the marker-end CSS property of an SVG path element using declarative SMIL animation. 

What is the expected behavior?
The marker-end will animate, changing values over time.

What went wrong?
The marker-end does not animate. It is stuck at the first animation value.

Did this work before? Yes Chrome 56.0.2924.76

Does this work in other browsers? Yes

Chrome version: 57.0.2987.133  Channel: stable
OS Version: OS X 10.12.0
Flash Version: 

Also verified as broken in Chrome Canary version 60.0.3108.0.
 
test.html
2.6 KB View Download

Comment 1 by f...@opera.com, May 23 2017

Components: -Blink>Animation Blink>SVG
Labels: -OS-Mac
Owner: f...@opera.com
Status: Assigned (was: Unconfirmed)
Bisected to db478183363dad5b460070122b991a8302a1f98a.
Labels: BugSource-User ReleaseBlock-Beta M-60 PaintTeamTriaged-20170523
Please tag with all affected OSs.  Thanks.

Comment 4 by f...@opera.com, May 24 2017

Labels: OS-Android OS-Chrome OS-Fuchsia OS-Linux OS-Mac OS-Windows

Comment 5 by ajha@chromium.org, May 25 2017

Just to update, as per schedule, M-60 will branched today(05/25) and promoted to Beta during 1st week of June.

fs@: Since this is marked as Beta blocker, please plan the fix accordingly.

Thank you!

Comment 6 by f...@opera.com, May 25 2017

Status: Started (was: Assigned)
Project Member

Comment 7 by bugdroid1@chromium.org, May 25 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/77584b488e6f147a4a0b5d75a1ceda38ff80dd08

commit 77584b488e6f147a4a0b5d75a1ceda38ff80dd08
Author: fs <fs@opera.com>
Date: Thu May 25 18:33:04 2017

Stricter equality check for local refs in CSSURIValue

'local' references ("#foo") needn't even consider the absolute URL(s)
when being compared. In this particular case, the (redundant) comparison
ended up comparing equal because a base-URL-less parser context is used
when parsing the property (will be fixed separately.)
When |is_local_| is set, only consider the relative URLs during the
comparison.

BUG= 725469 

Review-Url: https://codereview.chromium.org/2905033003
Cr-Commit-Position: refs/heads/master@{#474717}

[add] https://crrev.com/77584b488e6f147a4a0b5d75a1ceda38ff80dd08/third_party/WebKit/LayoutTests/svg/animations/animate-local-url-expected.html
[add] https://crrev.com/77584b488e6f147a4a0b5d75a1ceda38ff80dd08/third_party/WebKit/LayoutTests/svg/animations/animate-local-url.html
[modify] https://crrev.com/77584b488e6f147a4a0b5d75a1ceda38ff80dd08/third_party/WebKit/Source/core/css/CSSURIValue.cpp

Cc: kkaluri@chromium.org
Labels: TE-Verified-M61 TE-Verified-61.0.3114.0
Unable to reproduce this issue on Windows 10, Ubuntu 14.04 and Mac 10.12.4 with chrome #61.0.3114.0

Adding TE-verified labels

Attaching a screen-cast for reference.



Issue 725469.mp4
310 KB View Download
SohagNerob
sohagnerob1993@gmail.com
If there is no pending work, can we tag this as fixed?

Comment 11 by f...@opera.com, May 30 2017

Status: Fixed (was: Started)
Labels: Merge-TBD
[Auto-generated comment by a script] We noticed that this issue is targeted for M-60; it appears the fix may have landed after branch point, meaning a merge might be required. Please confirm if a merge is required here - if so add Merge-Request-60 label, otherwise remove Merge-TBD label. Thanks.

Comment 13 by f...@opera.com, May 30 2017

Labels: -Merge-TBD
According to the branch announcement, M60 was cut at 474934, which means the fix should be on it.
Status: Verified (was: Fixed)

Sign in to add a comment