stylo: Remove eRestyle_SVGAttrAnimations declartion.

RESOLVED FIXED in Firefox 55

Status

()

Core
CSS Parsing and Computation
P1
normal
RESOLVED FIXED
a year ago
a year ago

People

(Reporter: mantaroh, Assigned: boris)

Tracking

unspecified
mozilla55
Points:
---

Firefox Tracking Flags

(firefox55 fixed)

Details

MozReview Requests

()

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(2 attachments, 2 obsolete attachments)

(Reporter)

Description

a year ago
The stylo build failed because we removed the SVGAttrAnimationRuleProcessor on bug 1062106(bug 1062106, comment 81). So then I decided that I leave related restyle hint and remove this restyle hint from gecko and servo at this bug.

Note that we should remove string definition from RestyleManager when we remove this restyle hint.[1]

[1] https://dxr.mozilla.org/mozilla-central/rev/e03e0c60462c775c7558a1dc9d5cf2076c3cd1f9/layout/base/RestyleManager.cpp#411
(Assignee)

Updated

a year ago
Assignee: nobody → boris.chiou
(Assignee)

Comment 1

a year ago
OK, Looks like we only need to remove eRestyle_SVGAttrAnimations on stylo. I suppose we need to wait for the patches of bug 1062106 merged into m-c first. Fortunately, they are in autoland alredy.
(Assignee)

Updated

a year ago
Summary: Remove eRestyle_SVGAttrAnimations declartion. → stylo: Remove eRestyle_SVGAttrAnimations declartion.
(Assignee)

Updated

a year ago
Component: Layout → CSS Parsing and Computation
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
(Assignee)

Updated

a year ago
Attachment #8850818 - Flags: review?(boris.chiou)

Comment 5

a year ago
mozreview-review
Comment on attachment 8850816 [details]
Bug 1349834 - Remove eRestyle_SVGAttrAnimations declartion.

https://reviewboard.mozilla.org/r/123322/#review125848
Attachment #8850816 - Flags: review?(xidorn+moz) → review+

Comment 6

a year ago
mozreview-review
Comment on attachment 8850817 [details]
Bug 1349834 - Update RESTYLE_STYLE_ATTRIBUTE to sync Gecko change.

https://reviewboard.mozilla.org/r/123324/#review125850
Attachment #8850817 - Flags: review?(xidorn+moz) → review+
Comment hidden (mozreview-request)
(Assignee)

Updated

a year ago
Attachment #8850817 - Attachment is obsolete: true
(Assignee)

Updated

a year ago
Attachment #8850818 - Attachment is obsolete: true
Comment hidden (mozreview-request)
(Assignee)

Comment 9

a year ago
Created attachment 8850967 [details] [review]
Servo PR, #16119

Comment 10

a year ago
Pushed by bchiou@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/b2a34dd3df40
Remove eRestyle_SVGAttrAnimations declartion. r=xidorn

Comment 11

a year ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/b2a34dd3df40
Status: NEW → RESOLVED
Last Resolved: a year ago
status-firefox55: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
Thank you Boris!
Priority: -- → P1
You need to log in before you can comment on or make changes to this bug.