(In reply to comment #6) > Probably simpler to use nsIntRect(...).ToAppUnits(...);
Thanks, I didn't realise that existed. In any case, as aPoint is already in app units, it doesn't seem to make things any easier.
I'm attaching an updated patch now, slightly tidied and with a test case
(In reply to comment #6) ...).ToAppUnits (...);
> Probably simpler to use nsIntRect(
Thanks, I didn't realise that existed. In any case, as aPoint is already in app units, it doesn't seem to make things any easier.
I'm attaching an updated patch now, slightly tidied and with a test case