Closed Bug 1454622 Opened 5 years ago Closed 4 years ago

Change DOMQuad bounds to getBounds() as per specification


(Core :: DOM: Core & HTML, enhancement, P3)




Tracking Status
firefox61 --- wontfix
firefox69 --- fixed


(Reporter: zcorpan, Assigned: saschanaz)


(Keywords: dev-doc-needed, site-compat)


(1 file)


Gecko implements .bounds instead of .getBounds(). Chromium and WebKit implement .getBounds().

Test that needs fixing to change .bounds to .getBounds():

Test that uses getBounds():

Should also add .bounds to
This isn't just a naming matter.  The two getters have totally different semantics ([SameObject] vs [NewObject]).  Would need to carefully audit all code using this getter to see whether it's OK with that change.

In practice, we should probably ship getBounds(), convert internal consumers, then work on telemetry for .bounds or something...
Priority: -- → P3
Keywords: site-compat
In bug 1186265, I'm marking .bounds as deprecated and adding telemetry to give us an idea of how used it is. Hopefully we'll be able to remove it in a few releases.
Component: DOM → DOM: Core & HTML

I suspect once this is fixed we can use [Default] for the toJSON method on DOMQuad.

The bounds attribute has been deprecated and shown zero use, and thus this change removes it.

Should an intent to unship published?

Flags: needinfo?(bzbarsky)
Keywords: checkin-needed

Yes, please!

Note that I tried landing the changes and they failed to apply to tip of autoland, so you may need to merge to tip....

Flags: needinfo?(bzbarsky)
Keywords: checkin-needed
Keywords: checkin-needed
Assignee: nobody → saschanaz

Pushed by
Remove DOMQuad bounds attribute r=bzbarsky

Keywords: checkin-needed
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla69

On MDN we only mention getBounds(), so I think this'll just require a mention in the Fx69 rel notes when the time comes.

Keywords: dev-doc-needed
Created web-platform-tests PR for changes under testing/web-platform/tests
You need to log in before you can comment on or make changes to this bug.