1957689 - Track SVG local refs separately from SVG

Status: RESOLVED FIXED

(Core :: SVG, defect)

Comment 1

[Emilio Cobos Álvarez [:emilio]]

Attachment: Bug 1957689 - Move URlAndReferrerInfo to SVGObserverUtils.cpp. r=longsonr,#layout,nika

We don't have use for it elsewhere.

Comment 2

[Emilio Cobos Álvarez [:emilio]]

Attachment: Bug 1957689 - Track local SVG references independently. r=longsonr,#layout,nika

Comment 3

[Emilio Cobos Álvarez [:emilio]]

Attachment: Bug 1957689 - Use atoms a bit more in IDTracker. r=smaug,#dom-core

Just drive-by clean-ups.

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/bed9e7a07e7f
Move URlAndReferrerInfo to SVGObserverUtils.cpp. r=longsonr
https://hg.mozilla.org/integration/autoland/rev/3ddb53b6a1ce
Track local SVG references independently. r=longsonr

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/1c27bd6b8786
Fix detection of cross-doc references.

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/b42a36a572dd
Restore commented-out assert that failed locally for me.

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/8c0237947a34
Appease old gcc which doesn't like the nullptr vs NonNull<Encoding*> ternary.

Comment 8

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/38ce7ebff935
Use atoms a bit more in IDTracker. r=smaug

Comment 9

[Cristina Horotan [:chorotan]]

https://hg.mozilla.org/mozilla-central/rev/bed9e7a07e7f
https://hg.mozilla.org/mozilla-central/rev/3ddb53b6a1ce
https://hg.mozilla.org/mozilla-central/rev/1c27bd6b8786
https://hg.mozilla.org/mozilla-central/rev/b42a36a572dd
https://hg.mozilla.org/mozilla-central/rev/8c0237947a34
https://hg.mozilla.org/mozilla-central/rev/38ce7ebff935

Comment 10

[Mayank Bansal]

Increased the metric on Motionmark-suits by 16%.
Not sure if thats an improvement or a regression (perfherder says "higher is better", but IIRC motionmark is a bit weird)