1395870 - Changes in bug 1361639 caused some deadcode in SVGTransformableElement::GetAnimatedTransformList

Status: RESOLVED FIXED

(Core :: SVG, defect)

Description

[Sylvestre Ledru [:Sylvestre]]

intervals: At condition flags == 0U, the value of flags must be in one of the following intervals: {[512,513], [516,517], [528,529], [532,533], [544,545], [548,549], [560,561], [564,565]}.
   dead_error_condition: The condition flags == 0U cannot be true.
208    if (flags == 0) {
   CID 1417207 (#1 of 2): Logically dead code (DEADCODE)dead_error_line: Execution cannot reach this statement: <temporary>.already_AddRefe....
209      return NS_NewSVGRect(this,0,0,0,0);

Comment 1

[Sylvestre Ledru [:Sylvestre]]

Attachment: Bug 1395870 - Changes in bug 1361639 caused some deadcode in SVGTransformableElement::GetAnimatedTransformList

Review commit: https://reviewboard.mozilla.org/r/175334/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/175334/

Comment 2

[u459114]

Comment on attachment 8903502 [details]
Bug 1395870 - Changes in bug 1361639 caused some deadcode in SVGTransformableElement::GetAnimatedTransformList

https://reviewboard.mozilla.org/r/175334/#review180374

thank you for fixing it.

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/8b08fc013700
Changes in bug 1361639 caused some deadcode in SVGTransformableElement::GetAnimatedTransformList r=cjku

Comment 4

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/8b08fc013700