1461092 - Unship GetPropertyCSSValue in beta.

Status: RESOLVED FIXED

(Core :: CSS Parsing and Computation, defect)

Description

[Emilio Cobos Álvarez (:emilio)]

Xidorn pointed out in dev-platform@ that this is probably safer to do in 61, and I think I agree.

Comment 1

[Emilio Cobos Álvarez (:emilio)]

Attachment: Unship getPropertyCSSValue in beta too.

Approval Request Comment
[Feature/Bug causing the regression]: N/A
[User impact if declined]: N/A
[Is this code covered by automated tests?]: N/A
[Has the fix been verified in Nightly?]: N/A
[Needs manual test from QE? If yes, steps to reproduce]: N/A
[List of other uplifts needed for the feature/fix]: none
[Is the change risky?]: not risky
[Why is the change risky/not risky?]: This API has been unshipped in the 61 nightly cycle already without any reported problem (see dependent bug). We're going to unship it in 61 directly.
[String changes made/needed]: none

Comment 2

[Ryan VanderMeulen [:RyanVM]]

Comment on attachment 8975244 [details] [diff] [review]
Unship getPropertyCSSValue in beta too.

I read through the dev-platform thread and it sounds like everyone's on board with flipping this pref on 61. Approved for 61.0b5.

Comment 3

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-beta/rev/8b81ec88ad09

Comment 4

[Kohei]

Updated the site compatibility note: https://www.fxsitecompat.com/en-CA/docs/2018/cssstyledeclaration-getpropertycssvalue-has-been-deprecated/