1465435 - remove reference to old platforms in manifest file

Status: RESOLVED FIXED

(Toolkit :: Places, enhancement, P3)

Description

[Joel Maher ( :jmaher ) (UTC -8)]

we can remove winxp (5.1.2600) from this file:
https://searchfox.org/mozilla-central/source/toolkit/components/places/tests/unit/xpcshell.ini#50

Comment 1

[Eliza Balazs [:ebalazs_]]

Attachment: Removed winxp reference in test_async_transactions.js

Comment 2

[Joel Maher ( :jmaher ) (UTC -8)]

Comment on attachment 8982193 [details] [diff] [review]
Removed winxp reference in test_async_transactions.js

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

::: toolkit/components/places/tests/unit/xpcshell.ini
@@ +47,4 @@
>  [test_annotations.js]
>  [test_asyncExecuteLegacyQueries.js]
>  [test_async_transactions.js]
> +skip-if = (os == "win") # Bug 1158887

previously we skipped only on winXP, now this skips on all windows (win7/win10), so we just need to remove this line completely.

Comment 3

[Eliza Balazs [:ebalazs_]]

Attachment: Removed winxp reference in test_async_transactions.js

Comment 4

[Joel Maher ( :jmaher ) (UTC -8)]

Comment on attachment 8982207 [details] [diff] [review]
Removed winxp reference in test_async_transactions.js

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

thanks

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/516d9db19be3
remove reference for winxp in manifest file. r=jmaher

Comment 6

[Noemi Erli[:noemi_erli]]

https://hg.mozilla.org/mozilla-central/rev/516d9db19be3