978426 - /builds/slave/test/build/tests/xpcshell/tests/security/manager/ssl/tests/unit/test_sts_preloadlist_perwindowpb.js | application crashed [@ mozilla::storage::Connection::~Connection] on B2G ICS Emulator Debug only

Status: RESOLVED FIXED

(Core :: Security: PSM, defect)

Description

[Brian Smith (:briansmith, :bsmith, use NEEDINFO?)]

Attachment: stack trace

Comment 1

[Brian Smith (:briansmith, :bsmith, use NEEDINFO?)]

https://hg.mozilla.org/integration/mozilla-inbound/rev/8b52006f691b

I disabled test_sts_preloadlist_perwindowpb.js on B2G/ARM due to perma-orange. I filed bug 978426 to re-enable it.

Also, there was a repeated typo in my original patch:

    fail-if = os == "android" || buildapp == "b2g""

I replaced this with:

    fail-if = os == "android" || buildapp == "b2g"

(Notice one less double-quote at the end.)

Comment 2

[:Cykesiopka]

Attachment: bug978426_reenable-test_sts_preloadlist_perwindowpb.js_v1.patch

No idea what has changed in the past months, but this test looks fine to re-enable so far:
https://treeherder.mozilla.org/ui/#/jobs?repo=try&revision=71e761002490

Comment 3

[Dana Keeler (she/her) (use needinfo) (:keeler for reviews)]

Comment on attachment 8533886 [details] [diff] [review]
bug978426_reenable-test_sts_preloadlist_perwindowpb.js_v1.patch

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

Cool - thanks!

Comment 4

[:Cykesiopka]

Thanks for the review.

(Try link is in Comment 2.)

Comment 5

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/c1accea055e5

Comment 6

[Ryan VanderMeulen [:RyanVM]]

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