we have 2 references in this manifest: https://searchfox.org/mozilla-central/source/testing/web-platform/meta/content-security-policy/generic/generic-0_10.html.ini#5 when removing, if there are no remaining conditions, remove the test name

there are 2 additional manifests: https://searchfox.org/mozilla-central/source/testing/web-platform/meta/content-security-policy/media-src/media-src-7_1.html.ini#5 https://searchfox.org/mozilla-central/source/testing/web-platform/meta/content-security-policy/media-src/media-src-redir-bug.sub.html.ini#3

Comment on attachment 8982169 [details] [diff] [review] remove reference to old winxp in generic-0_10.html.ini Review of attachment 8982169 [details] [diff] [review]: ----------------------------------------------------------------- ::: testing/web-platform/meta/content-security-policy/generic/generic-0_10.html.ini @@ +6,3 @@ > [Violation report status OK.] > expected: > + if not debug and (os == "win") : TIMEOUT this now applies to win7/win10 whereas before this applied to winxp only. We need to remove the entire clause (and delete 'expected' and [<name>] with no conditions)

Comment on attachment 8982176 [details] [diff] [review] remove reference to old winxp in generic-0_10.html.ini Review of attachment 8982176 [details] [diff] [review]: ----------------------------------------------------------------- perfect. As you can see the test is already disabled for all of win/!debug- now it is easier to read

:apavel, there are 2 other files (comment 1) that need attention as well- they were the same bugzilla component so I put them in the same bug. lets do checkin-needed after all patches are ready

Comment on attachment 8982191 [details] [diff] [review] removed media-src-redir-bug.sub.html.ini manifest file Review of attachment 8982191 [details] [diff] [review]: ----------------------------------------------------------------- I like removed files

Comment on attachment 8982192 [details] [diff] [review] removed win XP references in media-src-7_1.html.ini Review of attachment 8982192 [details] [diff] [review]: ----------------------------------------------------------------- well done

Pushed by apavel@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/af54112146f9 remove reference to old winxp in manifest file r=jmaher https://hg.mozilla.org/integration/mozilla-inbound/rev/cd597e745065 removed manifest file r=jmaher https://hg.mozilla.org/integration/mozilla-inbound/rev/f443ef59e6da remove reference to old winxp in manifest file r=jmaher

https://hg.mozilla.org/mozilla-central/rev/af54112146f9 https://hg.mozilla.org/mozilla-central/rev/cd597e745065 https://hg.mozilla.org/mozilla-central/rev/f443ef59e6da