1492091 - Remove a wrong WPT .ini file for subresource-integrity

Status: RESOLVED FIXED

(Core :: DOM: Security, enhancement)

Description

[Andrea Marchesini [:baku]]

We don't need testing/web-platform/meta/subresource-integrity/idlharness.window.js.ini

Comment 1

[Andrea Marchesini [:baku]]

Attachment: subresource.patch

Comment 2

[Christoph Kerschbaumer [:ckerschb]]

(In reply to Andrea Marchesini [:baku] from comment #0)
> We don't need
> testing/web-platform/meta/subresource-integrity/idlharness.window.js.ini

How come that's not causing any problems on TRY at the moment? I mean how come we can remove that file without any actual code updates?

Comment 3

[Andrea Marchesini [:baku]]

Because that test doesn't exist. If you add in any .ini file [something that doesn't exist], this is ignored.

Comment 4

[Christoph Kerschbaumer [:ckerschb]]

Comment on attachment 9009898 [details] [diff] [review]
subresource.patch

Review of attachment 9009898 [details] [diff] [review]:
-----------------------------------------------------------------

(In reply to Andrea Marchesini [:baku] from comment #3)
> Because that test doesn't exist. If you add in any .ini file [something that
> doesn't exist], this is ignored.

Ah, the test itself doesn't even exist. How did you figure that?

Comment 5

[Andrea Marchesini [:baku]]

> Ah, the test itself doesn't even exist. How did you figure that?

Running them locally and controlling that the result was 6/6 (or something similar).

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/16265d1e65ee
Remove a wrong WPT .ini file for subresource-integrity, r=ckerschb

Comment 7

[Arthur Iakab [arthur_iakab]]

https://hg.mozilla.org/mozilla-central/rev/16265d1e65ee