Open Bug 1545439 Opened 8 months ago Updated 7 months ago

Intermittent toolkit/components/extensions/test/xpcshell/test_ext_MessageManagerProxy.js | application crashed [@ mozilla::jni::GetEnvForThread()]

Categories

(WebExtensions :: Android, defect, P5, critical)

defect

Tracking

(Not tracked)

People

(Reporter: intermittent-bug-filer, Unassigned)

References

Details

(Keywords: crash, intermittent-failure, regression)

Crash Data

#[markdown(off)]
Filed by: rob [at] robwu.nl

https://treeherder.mozilla.org/logviewer.html#?job_id=241138384&repo=try

https://queue.taskcluster.net/v1/task/a3WU-STEQPqRUqSC3gDZPQ/runs/0/artifacts/public/logs/live_backing.log

A new unit test for bug 1301837 crashing on Android. The crash is triggered by the `browser.swapDocShells` call at https://hg.mozilla.org/try/file/abd6f9d86c19/toolkit/components/extensions/test/xpcshell/test_ext_MessageManagerProxy.js#l57

I'm going to disable the (new) test on Android, because current extension code doesn't call swapDocShells on Android in practice.

Stack trace:

```
INFO - TEST-PASS | xpcshell.ini:toolkit/components/extensions/test/xpcshell/test_ext_MessageManagerProxy.js | test_message_after_swapdocshells - [test_message_after_swapdocshells : 38] Expected ping-pong - "page2:after setup (to 2)" == "page2:after setup (to 2)"
INFO - xpcshell.ini:toolkit/components/extensions/test/xpcshell/test_ext_MessageManagerProxy.js | Segmentation fault
...
WARNING - PROCESS-CRASH | xpcshell.ini:toolkit/components/extensions/test/xpcshell/test_ext_MessageManagerProxy.js | application crashed [@ mozilla::jni::GetEnvForThread()]

0 libxul.so!mozilla::jni::GetEnvForThread() [Utils.cpp:a17f163841e956c10eb74cdbe27d7073facf6b7f : 0 + 0x6]
1 libxul.so!mozilla::java::GeckoAppShell::UnlockScreenOrientation() [GeneratedJNIWrappers.cpp: : 648 + 0x5]
2 libxul.so!nsDocShell::SetIsActive(bool) [nsDocShell.cpp:a17f163841e956c10eb74cdbe27d7073facf6b7f : 5310 + 0x3]
3 libxul.so!nsDocShell::SetDocLoaderParent(nsDocLoader*) [nsDocShell.cpp:a17f163841e956c10eb74cdbe27d7073facf6b7f : 2604 + 0x9]
4 libxul.so!nsDocLoader::AddChildLoader(nsDocLoader*) [nsDocLoader.cpp:a17f163841e956c10eb74cdbe27d7073facf6b7f : 614 + 0xb]
5 libxul.so!nsDocShell::AddChild(nsIDocShellTreeItem*) [nsDocShell.cpp:a17f163841e956c10eb74cdbe27d7073facf6b7f : 3237 + 0x7]
6 libxul.so!nsFrameLoader::SwapWithOtherLoader(nsFrameLoader*, nsFrameLoaderOwner*, nsFrameLoaderOwner*) [nsFrameLoader.cpp:a17f163841e956c10eb74cdbe27d7073facf6b7f : 1570 + 0x11]
7 libxul.so!mozilla::dom::XULFrameElement::SwapFrameLoaders(nsFrameLoaderOwner*, mozilla::ErrorResult&) [XULFrameElement.cpp:a17f163841e956c10eb74cdbe27d7073facf6b7f : 132 + 0xd]
8 libxul.so!mozilla::dom::XULFrameElement_Binding::swapFrameLoaders(JSContext*, JS::Handle<JSObject*>, mozilla::dom::XULFrameElement*, JSJitMethodCallArgs const&) [XULFrameElementBinding.cpp: : 345 + 0xd]
```

Don't mark intermittent crashes as P5s. We want them to go to triage owners.

Priority: P5 → --

The priority flag is not set for this bug.
:ddurst, could you have a look please?

For more information, please visit auto_nag documentation.

Flags: needinfo?(ddurst)

P5 - This is an Android-only issue, and the test has been disabled. The feature that is tested by the test (swapping browsers) is currently not in use on Fennec.

Flags: needinfo?(ddurst)
Priority: -- → P5
You need to log in before you can comment on or make changes to this bug.