1635082 - Fix crossorigin relevant mutation tests.

Status: RESOLVED FIXED

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

Comment 1

[Emilio Cobos Álvarez (:emilio)]

Attachment: Bug 1635082 - Fix crossorigin relevant mutation tests. r=edgar

referrerpolicy and crossorigin changes were handled in
AfterMaybeChangeAttr, but were only handling the case where the
attribute changed (that is, the AfterSetAttr caller). However
AfterSetAttr only calls AfterMaybeChangeAttr if there's a new value,
which means that we don't handle attribute removal.

So their handling can just move to AfterSetAttr, and we can simplify
AfterMaybeChangeAttr to just deal with src changes which is what it was
intended to deal with anyhow.

Comment 2

[Emilio Cobos Álvarez (:emilio)]

Attachment: Bug 1635082 - Remove nsImageLoadingContent::GetImageReferrerPolicy. r=edgar

It's useless now, the relevant check lives in
ReferrerInfo::InitFromNode.

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/716bd7fea38c
Fix crossorigin relevant mutation tests. r=edgar

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/249fc32acae5
Remove nsImageLoadingContent::GetImageReferrerPolicy. r=edgar

Comment 5

[Cristina Coroiu [:ccoroiu]]

https://hg.mozilla.org/mozilla-central/rev/716bd7fea38c
https://hg.mozilla.org/mozilla-central/rev/249fc32acae5