904659 - Some of the Iframe sandbox mochitests crash on b2g

Status: RESOLVED WORKSFORME

(Core :: Security, defect)

Description

[Martijn Wargers (dead)]

These mochitests are crashing in the b2g emulator:
http://mxr.mozilla.org/mozilla-central/source/content/html/content/test/test_iframe_sandbox_inheritance.html?force=1
http://mxr.mozilla.org/mozilla-central/source/content/html/content/test/test_iframe_sandbox_navigation.html?force=1
http://mxr.mozilla.org/mozilla-central/source/content/html/content/test/test_iframe_sandbox_navigation2.html?force=1

With the url, I can see the crash also occuring when using the browser on b2g.

Comment 1

[Ian Melven :imelven]

The easiest fix is to disable these tests on B2G.

The downside is that some B2G apps use iframe sandbox to provide security guarantees.

I won't have any spare time to look into these for at least a few weeks, I'm afraid.

Comment 2

[Martijn Wargers (dead)]

These tests are currently disabled on B2G, the point is to get them enabled.

Comment 3

[Boris Zbarsky [:bzbarsky]]

Is there breakpad on b2g or some other way to get the stack for these crashes?

Comment 4

[Martijn Wargers (dead)]

You should be able to get breakpad crashes, like described here: https://wiki.mozilla.org/B2G/QA/Tips_And_Tricks#Getting_crashes_off_the_Device
I tried to get stacktraces for browser crashes in the b2g emulator, but failed, see bug 902271, comment 2.

Comment 5

[Bob Owen (:bobowen)]

I thought that for these tests the main problem with B2G was that they open multiple new browsing contexts at once (using window.open), which causes problems on B2G ... is this a different problem?

Comment 6

[Martijn Wargers (dead)]

Attachment: 904659.diff

Ok, it seems like these tests don't crash anymore on b2g, this patch would enable those tests: https://tbpl.mozilla.org/?tree=Try&rev=2992a3a6648b

Comment 7

[Martijn Wargers (dead)]

I've filed bug 1037976 for enabling those tests on b2g emulator, I'm going to mark this bug as worksforme.