Created attachment 468363 [details] [diff] [review]
I don't see any reason for storing this frame pointer,
it's only used in two MouseClick methods and it's not
a performance problem to do the lookup there, AFAIK.
Removing it eliminates the risk of it becoming a stale pointer.
It also allows us to remove nsFileControlFrame::Reflow()
which now doesn't do anything besides calling the super class.
Created attachment 575363 [details] [diff] [review]
Comment on attachment 575363 [details] [diff] [review]