Code analysis found 5 defects in the diff 5119:

• 1 defect found by clang-format
• 1 defect found by file-whitespace (Mozlint)
• 3 defects found by clang-tidy

You can run this analysis locally with:

• ./mach clang-format -p dom/animation/Animation.cpp
• ./mach lint --warnings --outgoing
• ./mach static-analysis check --outgoing (C/C++)

For your convenience, here is a patch that fixes all the clang-format defects (use it in your repository with hg import or git apply -p0).

The analysis task source-test-coverity-coverity failed, but we could not detect any issue.
Please check this task manually.

If you see a problem in this automated review, please report it here.

You can view these defects on the code-review frontend and on Treeherder.

Code analysis found 8 defects in the diff 5125:

• 1 defect found by clang-format
• 1 defect found by file-whitespace (Mozlint)
• 3 defects found by Coverity
• 3 defects found by clang-tidy

You can run this analysis locally with:

• ./mach clang-format -p dom/animation/Animation.cpp
• ./mach lint --warnings --outgoing
• ./mach static-analysis check --outgoing (C/C++)

For your convenience, here is a patch that fixes all the clang-format defects (use it in your repository with hg import or git apply -p0).

If you see a problem in this automated review, please report it here.

You can view these defects on the code-review frontend and on Treeherder.