>+ PRInt32 offsetForContextMenuCSS = mAdjustOffsetForContextMenu ? 2 : 0;
>+ PRInt32 offsetForContextMenuDev =
>+ nsPresContext::CSSPixelsToAppUnits(offsetForContextMenuCSS) / factor;
>+ offsetForContextMenu = presContext->DevPixelsToAppUnits(offsetForContextMenuDev);
Any reason not to do this in an if rather than possibly calculating with zero?
Comment on attachment 317116
fix v3
>+ nscoord offsetForContex tMenu = 0;
>+ PRInt32 offsetForContex tMenuCSS = mAdjustOffsetFo rContextMenu ? 2 : 0; tMenuDev = :CSSPixelsToApp Units(offsetFor ContextMenuCSS) / factor; tMenu = presContext- >DevPixelsToApp Units(offsetFor ContextMenuDev) ;
>+ PRInt32 offsetForContex
>+ nsPresContext:
>+ offsetForContex
Any reason not to do this in an if rather than possibly calculating with zero?