Rebased and added a comment to explain what VIRTUAL_OFFSET is for. I'll do a fresh try run with compositor enabled before landing this.

Code analysis found 28 defects in the diff 214519:

• 28 defects found by clang-format

You can run this analysis locally with:

• ./mach clang-format -s -p gfx/webrender_bindings/DCLayerTree.cpp gfx/webrender_bindings/DCLayerTree.h (C/C++)

For your convenience, here is a patch that fixes all the source-test-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.

In our previous code coverage analysis run, we found some files which had no coverage and are being modified in this patch:

• gfx/webrender_bindings/DCLayerTree.cpp

Should they have tests, or are they dead code ?

• You can file a bug blocking Bug 1415824 for untested files that should be tested.
• You can file a bug blocking Bug 1415819 for untested files that should be removed.

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

In our previous code coverage analysis run, we found some files which had no coverage and are being modified in this patch:

• gfx/webrender_bindings/DCLayerTree.cpp

Should they have tests, or are they dead code ?

• You can file a bug blocking Bug 1415824 for untested files that should be tested.
• You can file a bug blocking Bug 1415819 for untested files that should be removed.

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

In our previous code coverage analysis run, we found some files which had no coverage and are being modified in this patch:

• gfx/webrender_bindings/DCLayerTree.cpp

Should they have tests, or are they dead code ?

• You can file a bug blocking Bug 1415824 for untested files that should be tested.
• You can file a bug blocking Bug 1415819 for untested files that should be removed.

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