nsSVGGlyphFrame::UpdateBounds should reuse GetBBoxContribution instead of duplicating code.
Created attachment 640577 [details] [diff] [review]
I'm doing this separately from bug 772017, and passing flags to GetBBoxContribution that are consistent with what the code does now, even though I'm going to change them in bug 772017. This is because - although I think the patch is okay - there are so many layers of indirection in the nsSVGGlyphFrame code that it's hard to be totally sure. I want this separate patch that's as consistent with what we do now as possible so any regressions that might be caused by using GetBBoxContribution can be squarely blamed on this bug, and not flag changes.