As a security precaution, we have turned on the setting "Require API key authentication for API requests" for everyone. If this has broken something, please contact
Last Comment Bug 931389 - Deprecate and remove CanvasRenderingContext2D.mozDash/mozDashOffset
: Deprecate and remove CanvasRenderingContext2D.mozDash/mozDashOffset
: addon-compat, dev-doc-needed, site-compat
Product: Core
Classification: Components
Component: Canvas: 2D (show other bugs)
: unspecified
: All All
: -- normal (vote)
: mozilla52
Assigned To: Thomas Wisniewski
: Milan Sreckovic [:milan]
: 931643 (view as bug list)
Depends on:
Blocks: unprefix 768067
  Show dependency treegraph
Reported: 2013-10-26 10:09 PDT by :Ms2ger (⌚ UTC+1/+2)
Modified: 2016-10-06 05:22 PDT (History)
7 users (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

931389-remove-mozDash-and-mozDashOffset.diff (13.58 KB, patch)
2016-09-15 06:07 PDT, Thomas Wisniewski
gwright: review+
Details | Diff | Splinter Review
931389-remove-mozDash-and-mozDashOffset.diff (13.58 KB, patch)
2016-10-04 11:29 PDT, Thomas Wisniewski
amarchesini: review+
Details | Diff | Splinter Review

Description User image :Ms2ger (⌚ UTC+1/+2) 2013-10-26 10:09:44 PDT
Originally part of bug 768067, but didn't happen there.
Comment 1 User image :Ms2ger (⌚ UTC+1/+2) 2016-08-11 01:26:47 PDT
*** Bug 931643 has been marked as a duplicate of this bug. ***
Comment 2 User image Thomas Wisniewski 2016-09-15 06:07:42 PDT
Created attachment 8791582 [details] [diff] [review]

Since the unprefixed versions of these features have been in Firefox since version 27, the MDN page for the standard version has noted that they're deprecated since at least July of last year, and no addons on DXR are only using the prefixed versions, I think it's probably safe to go ahead and remove them now. Here's a patch which does so. Try seems fine:

Intent to unship:!topic/
Comment 3 User image George Wright (:gw280) (:gwright) (needinfo me!) 2016-09-15 12:11:18 PDT
Comment on attachment 8791582 [details] [diff] [review]

Review of attachment 8791582 [details] [diff] [review]:

I'm ok with this, but let's give dev-platform a little more time to raise any possible objections.
Comment 4 User image Thomas Wisniewski 2016-10-03 12:10:26 PDT
I still haven't received any feedback on the dev-platform thread, George. Would you still like to wait a while longer?
Comment 5 User image George Wright (:gw280) (:gwright) (needinfo me!) 2016-10-03 13:29:19 PDT
Nope, let's go ahead with this.
Comment 6 User image Thomas Wisniewski 2016-10-04 11:29:11 PDT
Created attachment 8797719 [details] [diff] [review]

Alright then. Here's a rebased version of the patch. A fresh try run* seems fine, so I'm carrying over r+ and requesting check-in.

Comment 7 User image Kohei Yoshino [:kohei] 2016-10-04 11:31:22 PDT
Posted the site compatibility doc:
Comment 8 User image Ryan VanderMeulen [:RyanVM] 2016-10-04 11:58:59 PDT
The webidl changes need DOM peer review.
Comment 9 User image Pulsebot 2016-10-05 05:34:59 PDT
Pushed by
remove CanvasRenderingContext2D.mozDash/mozDashOffset. r=gw280, r=baku
Comment 10 User image Carsten Book [:Tomcat] 2016-10-06 03:03:56 PDT

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