Bug 1537588 Comment 4 Edit History

Note: The actual edited comment in the bug view page will always show the original commentor’s name and original timestamp.

Apparently this event isn't reliably fired in this scenario, even in normal mozilla-central. My crashtest was passing reliably yesterday, locally, and it passes on Android (it's shown as passing in the log for [Android 4.3 opt C1](https://treeherder.mozilla.org/#/jobs?repo=try&revision=725f99eea2ed38e7b40191c34a740dd1b873f116&selectedJob=235089294) and debug C1 in my try run above).

But it failed on all other Try platforms (specifically, it never fired pageshow, so it timed out).

I tried to rewrite the test with the sync XHR + navigation happening in an iframe, but that doesn't seem to help -- that still makes pageshow fire quite rarely (with "###!!! ASSERTION: Destroying a currently-showing document" being spammed most of the time).

I'm experimenting with an iframe-based pagehide test now, though, which seems more reliable locally (despite comment 2). 
If that works, I might just settle for that at the moment, and hopefully we can extend it to cover pageshow as well once bug 1535388 or another followup is addressed.
Apparently this event isn't reliably fired in this scenario, even in normal mozilla-central. My crashtest was passing reliably yesterday, locally, and it passes on Android (it's shown as passing in the log for [Android 4.3 opt C1](https://treeherder.mozilla.org/#/jobs?repo=try&revision=725f99eea2ed38e7b40191c34a740dd1b873f116&selectedJob=235089294) and debug C1 in my try run above) -- but it failed on all other Try platforms (specifically, it never fired pageshow, so it timed out).

I tried to rewrite the test with the sync XHR + navigation happening in an iframe, but that doesn't seem to help -- that still makes pageshow fire quite rarely (with "###!!! ASSERTION: Destroying a currently-showing document" being spammed most of the time).

I'm experimenting with an iframe-based pagehide test now, though, which seems more reliable locally (despite comment 2). 
If that works, I might just settle for that at the moment, and hopefully we can extend it to cover pageshow as well once bug 1535388 or another followup is addressed.

Back to Bug 1537588 Comment 4