:Ms2ger (he/him; ⌚ UTC+1/+2) Reporter
	Description • 12 years ago

Added to the spec in <http://html5.org/tools/web-apps-tracker?from=7634&to=7635>; need to check if our implementation matches that.

	Masatoshi Kimura [:emk]
	Comment 1 • 11 years ago

The fillRule property has been superseded by CanvasWindingRule and no longer defined in the latest spec. We should just remove mozFillRule.

	Thomas Wisniewski Assignee
	Comment 2 • 8 years ago

Here's a patch which removes mozFillRule, assuming that's still what's desirable. A try run seems fine: https://treeherder.mozilla.org/#/jobs?repo=try&revision=8e4f85e4c5fc

	Thomas Wisniewski Assignee
	Comment 3 • 8 years ago

:emk, should I go ahead and mark this as checkin-needed, or has the situation changed since you wrote comment 1?

	Masatoshi Kimura [:emk]
	Comment 4 • 8 years ago

No changes except that CanvasWindingRule has been renamed to CanvasFillRule. But a DOM peer should sign-off the WebIDL change.

	Thomas Wisniewski Assignee
	Comment 5 • 8 years ago

:mrbkap, mind signing off on the WebIDL change?

	Blake Kaplan (:mrbkap) (inactive)
	Comment 6 • 8 years ago

r=me on the WebIDL change.

	Thomas Wisniewski Assignee
	Comment 7 • 8 years ago

Thanks!

Requesting check-in.

	Pulsebot
	Comment 8 • 8 years ago

Pushed by cbook@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/0ca0282fe48b
Remove Canvas mozFillRule. r=mrbkap r=bas

	Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)
	Comment 9 • 8 years ago
bugherder

https://hg.mozilla.org/mozilla-central/rev/0ca0282fe48b

	Kohei Yoshino
	Comment 10 • 8 years ago

Posted the site compatibility doc: https://www.fxsitecompat.com/en-CA/docs/2016/canvasrenderingcontext2d-mozfillrule-has-been-removed/

	Jean-Yves Perrier [:teoli]
	Comment 11 • 8 years ago

mozFillRule removed from https://developer.mozilla.org/en-US/docs/Web/API/CanvasRenderingContext2D
and
https://developer.mozilla.org/en-US/Firefox/Releases/51#Canvas mentions it.