Bug 41466

Summary:

[Qt] Canvas: clip() should use non-zero winding rule

Product:

WebKit

Reporter:

Andreas Kling <kling>

Component:

WebCore Misc.

Assignee:

Nobody <webkit-unassigned>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

commit-queue, mdelaney7

Priority:

Keywords:

HTML5, Qt, QtTriaged

Version:

528+ (Nightly build)

Hardware:

All

OS:

All

Attachments:

Description	Flags
Screenshot of web inspector in QtTestBrowser	none
Proposed patch	none

Andreas Kling

Reported 2010-07-01 09:03:57 PDT

Created attachment 60256 [details] Screenshot of web inspector in QtTestBrowser This causes the "pill" in the web inspector to look strange (see attached screenshot.) Quoth HTML5: "The clip() method must create a new clipping region by calculating the intersection of the current clipping region and the area described by the current path, using the non-zero winding number rule." Spec link: http://www.whatwg.org/specs/web-apps/current-work/#dom-context-2d-clip

Attachments
Screenshot of web inspector in QtTestBrowser (67.31 KB, image/png) 2010-07-01 09:03 PDT, Andreas Kling	no flags	Details
Proposed patch (1.44 KB, patch) 2010-07-01 09:06 PDT, Andreas Kling	no flags	Details Formatted Diff Diff
View All Add attachment proposed patch, testcase, etc.

Andreas Kling

Comment 1 2010-07-01 09:06:22 PDT

Created attachment 60257 [details] Proposed patch

Oliver Hunt

Comment 2 2010-07-01 09:15:11 PDT

Comment on attachment 60257 [details] Proposed patch r=me

WebKit Commit Bot

Comment 3 2010-07-01 11:28:33 PDT

Comment on attachment 60257 [details] Proposed patch Clearing flags on attachment: 60257 Committed r62276: <http://trac.webkit.org/changeset/62276>

WebKit Commit Bot

Comment 4 2010-07-01 11:28:38 PDT

All reviewed patches have been landed. Closing bug.

Note You need to log in before you can comment on or make changes to this bug.