[Webkit-unassigned] [Bug 41576] [chromium] Fix drawing of inset box shadows

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Sun Jul 4 09:56:22 PDT 2010


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


mitz at webkit.org changed:

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




--- Comment #5 from mitz at webkit.org  2010-07-04 09:56:22 PST ---
(From update of attachment 60467)
A method called GraphicsContext::canvasClip() sounds like a bad idea. According to GraphicsContext.h,

// This clip function is used only by <canvas> code

but this patch makes that comment false and illustrates why having this method is a bad idea. Moreover, the change log (which mentions a bunch of files that the patch doesn’t touch) does a very poor job of explaining the change.

I suggest eliminating canvasClip() and devising a better solution to the platform-specific problem it was intended to solve; ideally, such a solution will not require people writing cross-platform code to have to think about the platform-specific problem. As things stand, there is no way for someone writing rendering code to tell whether they should call clip() or canvasClip().

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


More information about the webkit-unassigned mailing list