1422177 - Add tests to verify that moz-icon is unlinkable from the web

Status: RESOLVED FIXED

(Core :: Security: CAPS, enhancement)

Description

[:Gijs (he/him)]

We're removing access in bug 1222924.

We can probably just add some cases to https://searchfox.org/mozilla-central/source/caps/tests/mochitest/browser_checkloaduri.js .

Comment 1

[:Gijs (he/him)]

Attachment: Bug 1422177 - add tests for moz-icon being non-linkable from the web,

Review commit: https://reviewboard.mozilla.org/r/205132/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/205132/

Comment 2

[Jonathan Kingston [:jkt] he/him]

Comment on attachment 8934188 [details]
Bug 1422177 - add tests for moz-icon being non-linkable from the web,

https://reviewboard.mozilla.org/r/205132/#review210630

LGTM, I'm not a module owner or peer but I suspect that doesn't matter given it's just a test.

Comment 3

[:Gijs (he/him)]

(In reply to Jonathan Kingston [:jkt] from comment #2)
> Comment on attachment 8934188 [details]
> Bug 1422177 - add tests for moz-icon being non-linkable from the web,
> 
> https://reviewboard.mozilla.org/r/205132/#review210630
> 
> LGTM, I'm not a module owner or peer but I suspect that doesn't matter given
> it's just a test.

I... I don't actually know. I guess I can just ask Boris to rubberstamp this, too...

Comment 4

[Boris Zbarsky [:bzbarsky]]

Comment on attachment 8934188 [details]
Bug 1422177 - add tests for moz-icon being non-linkable from the web,

https://reviewboard.mozilla.org/r/205132/#review212016

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/c9670bff5a68
add tests for moz-icon being non-linkable from the web, r=bz,jkt

Comment 6

[Raul Gurzau (:RaulG)]

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