Closed Bug 1349834 Opened 3 years ago Closed 3 years ago
stylo: Remove e
Restyle _SVGAttr Animations declartion .
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.  https://dxr.mozilla.org/mozilla-central/rev/e03e0c60462c775c7558a1dc9d5cf2076c3cd1f9/layout/base/RestyleManager.cpp#411
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.
Summary: Remove eRestyle_SVGAttrAnimations declartion. → stylo: Remove eRestyle_SVGAttrAnimations declartion.
Attachment #8850818 - Flags: review?(boris.chiou)
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 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+
Attachment #8850817 - Attachment is obsolete: true
Attachment #8850818 - Attachment is obsolete: true
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/autoland/rev/b2a34dd3df40 Remove eRestyle_SVGAttrAnimations declartion. r=xidorn
Thank you Boris!
You need to log in before you can comment on or make changes to this bug.