nsRect is for areas in app units and it uses nscoord which may not be
PRInt32 in the future. We should propagate table damage using nsIntRect
to prevent mistakes and to make the code somewhat more future proof.
Arguably, we should a distinct type (eg TableDamageRect) to avoid the
mistake of passing a nsIntRect in pixel coordinates, but nsIntRect isn't
all that common on layout code so doesn't seem worth the effort right now.
Created attachment 590490 [details] [diff] [review]
Use nsIntRect for table damage.
Created attachment 590491 [details] [diff] [review]
Only calculate the border half we actally use.
Created attachment 590492 [details] [diff] [review]
Avoid calling GetFirstInFlow() on the row group twice.
That's all for this bug.