Open Bug 1528504 Opened 6 years ago Updated 3 years ago

Code coverage for nsDisplayList.h seems wrong

Categories

(Testing :: Code Coverage, enhancement, P3)

Product:
Testing
Component:
Code Coverage
Type:
enhancement

Tracking

(Not tracked)

People

(Reporter: mstange, Unassigned)

References

Details

Phabricator URL: https://phabricator.services.mozilla.com/D19219#527722

Reviewbot says:

In our previous code coverage analysis run, we found some files which had no coverage and are being modified in this patch:
layout/painting/nsDisplayList.h

This seems hard to believe. nsDisplayList.h contains code that is executed for every paint and it would be hard not to hit it during any test that opens a window.

Yeah, we have a bug with header files. Sorry abour that.

Severity: normal → critical
Priority: -- → P1
Assignee: nobody → mcastelluccio
Status: NEW → ASSIGNED
Priority: P1 → P3
Assignee: mcastelluccio → nobody
Severity: critical → --
Status: ASSIGNED → NEW
Component: Source Code Analysis → Code Coverage
Product: Firefox Build System → Testing
Version: Trunk → unspecified
You need to log in before you can comment on or make changes to this bug.