Closed Bug 1270387 Opened 8 years ago Closed 8 years ago

Disable Node.rootNode in release builds due to web compatibility issues

Categories

(Core :: DOM: Core & HTML, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla49
Tracking Status
firefox48 --- fixed
firefox49 --- fixed

People

(Reporter: yrliou, Assigned: yrliou)

References

Details

(Keywords: dev-doc-complete, site-compat, Whiteboard: btpp-active)

Attachments

(1 file)

See https://github.com/whatwg/dom/issues/241
As for now, both blink and safari unship this feature.

Before having a new name in the spec, we will use a preference to disable this feature in release builds first.

After spec being revised, remove the preference and adopt the change in Bug 1269155.
Attachment #8749056 - Flags: review?(bugs) → review+
Comment on attachment 8749056 [details]
MozReview Request: Bug 1270387 - Disable Node.rootNode in release builds due to web compatibility issues. r?smaug

https://reviewboard.mozilla.org/r/50727/#review47429

And we need to get this to aurora (FF48) right? So ask approval for the patch to land there too.
Whiteboard: btpp-active
https://hg.mozilla.org/mozilla-central/rev/49dab7b4a276
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla49
Comment on attachment 8749056 [details]
MozReview Request: Bug 1270387 - Disable Node.rootNode in release builds due to web compatibility issues. r?smaug

Approval Request Comment
[Feature/regressing bug #]: Bug 1254956
[User impact if declined]: Web compatibility issues have been brought up and discussed at https://github.com/whatwg/dom/issues/241, both safari and blink unshipped this feature. 
[Describe test coverage new/current, TreeHerder]: There are web platform tests for this feature which are still enabled, we only turn off the preference and disable this feature in release builds.
[Risks and why]: Low, just add a pref and turn it off in release builds
[String/UUID change made/needed]: None
Attachment #8749056 - Flags: approval-mozilla-aurora?
Comment on attachment 8749056 [details]
MozReview Request: Bug 1270387 - Disable Node.rootNode in release builds due to web compatibility issues. r?smaug

Part of deprecation strategy, has some test coverage, please uplift to aurora
Attachment #8749056 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Jocelyn should this have a release note? Or maybe more likely, does it need developer documentation?
(In reply to Liz Henry (:lizzard) (needinfo? me) from comment #7)
> Jocelyn should this have a release note? Or maybe more likely, does it need
> developer documentation?

Yes, I think we should have docs for specifying the new preference.
Documentation of rootNode was written since Bug 1254956, we should probably revise those existing docs and the 48 release note.
See Also: → 1254956
Keywords: site-compat
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.