[Webkit-unassigned] [Bug 28268] adjustLineToPixelBoundaries used in platform/GraphicsContext drawLine needs refactoring finished.

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=28268


Eric Seidel <eric at webkit.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #34761|review?                     |review-
               Flag|                            |




--- Comment #2 from Eric Seidel <eric at webkit.org>  2009-08-13 11:42:49 PDT ---
(From update of attachment 34761)
+        static void adjustLineToPixelBoundaries(FloatPoint& p1, FloatPoint&
p2, float strokeWidth, const StrokeStyle& penStyle);

"penStyle" name is not needed.

KHTML has little to do with WebKit anymore:
+    // works out.  For example, with a border width of 3, KHTML will pass us
(y1+y2)/2, e.g.,
That should be "WebKIt"

If you were a committer, I would have you fix those tiny nits when you landed,
but since you aren't (yet), please post a revised patch.

It's not clear when this function should be used.  I suspect that CoreGraphics
takes care of all this automatically internally?  Is this function used to work
around missing features in graphics libraries, or is this something that all
ports should be using all the time?

Please add some comments to the function declaration to help others understand
when (if ever) this should be used.  If it's only needed by Qt and CAiro,
consider wrapping it in an #ifdef

Otherwise looks fine.

Thanks!

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.