Since we can't overload std::abs() directly, adding an inline absoluteValue method instead.
Created attachment 139485 [details] Patch
Comment on attachment 139485 [details] Patch Darin Adler is a better reviewer for this than I am. But this is OK for me. I'd like you to wait a few hours and see if you can get him to comment as well though.
The reason why I suggest Darin, is that he might have a clever suggestion for your std::abs() problem other than this one.
(In reply to comment #3) > The reason why I suggest Darin, is that he might have a clever suggestion for your std::abs() problem other than this one. Deal! I'll wait till near EOD PST.
(In reply to comment #4) > (In reply to comment #3) > > The reason why I suggest Darin, is that he might have a clever suggestion for your std::abs() problem other than this one. > > Deal! I'll wait till near EOD PST. Unless there's argument, I'll move forward on landing this.
Committed r115691: <http://trac.webkit.org/changeset/115691>
Comment on attachment 139485 [details] Patch Clearing flags
Comment on attachment 139485 [details] Patch Only thing I might have done differently would have been to overload it for int and FractionalLayoutUnit from the start rather than defining it as a function that takes LayoutUnit.