specifying a bgcolor for a <tr> will immediately be overwritten by any bgcolor attributes within <td> tags.
Perhaps better to use bgcolor within all <td> rather than at <tr> level.
Similarly <tr> only controls the space occupied by the <td>'s and does not colour any cellspacing areas.