1551030 - Merge all SVGRect classes

Status: RESOLVED FIXED

(Core :: SVG, task, P3)

Description

[Robert Longson [:longsonr]]

This is a step on the way to implementing bug 1439685 as we'll only have one class to deal with there and also helps with bug 1242048 as there are fewer classes to deal with and fewer wrapper caches.

Comment 1

[Robert Longson [:longsonr]]

Attachment: Bug 1551030 Part 1 - Remove some obsolete typedefs

Comment 2

[Robert Longson [:longsonr]]

Attachment: Bug 1551030 Part 2 - Merge all SVGRect classes r=dholbert

Comment 3

[Robert Longson [:longsonr]]

The form of the implementation is similar to https://dxr.mozilla.org/mozilla-central/source/dom/svg/DOMSVGAngle.h and https://dxr.mozilla.org/mozilla-central/source/dom/svg/DOMSVGAngle.cpp now.

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/9e59023d2e69
Part 1 - Remove some obsolete typedefs r=dholbert

Comment 5

[Cristina Coroiu [:ccoroiu]]

https://hg.mozilla.org/mozilla-central/rev/9e59023d2e69

Comment 6

[Daniel Holbert [:dholbert]]

ticking needinfo to be sure this doesn't fall off your radar -- part 2 is unlanded but has r+ with minor nits, per https://phabricator.services.mozilla.com/D30804#983174 . Thanks!

Comment 7

[Robert Longson [:longsonr]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=33ab2ebac71a058ee74a18f2ac711b18ea604895&selectedJob=252618160

Comment 8

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/b848ca90888e
Merge all SVGRect classes r=dholbert

Comment 9

[Arthur Iakab [arthur_iakab]]

https://hg.mozilla.org/mozilla-central/rev/b848ca90888e

Comment 10

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/mozilla-central/rev/b848ca90888e