The aText parameter in this function:
Is unused in the body of the function, and should probably be removed.
Yup, it can go - thanks for catching that. It was made redundant by bug 703100, but I forgot to remove it there.
Created attachment 605669 [details] [diff] [review]
eliminate unused parameter in the gfxTextRun constructor