Created attachment 608909 [details] [diff] [review]
window.content may be null if the tab isn't active. While a page is loading, if it's put in the background, JS will give this error:
Rather than using content.document, I think we can use BrowserApp.selectedBrowser.contentDocument, which doesn't change if the tab is active/inactive.
Please can you set the target milestone when landing on inbound, along the lines of http://blog.bonardo.net/2012/03/23/how-you-can-help-mozilla-inbound-sheriffs-when-pushing :-)
This wasn't marked as a blocker. Should we be working on things like this right now?
(In reply to Damon Sicore (:damons) from comment #3)
> This wasn't marked as a blocker. Should we be working on things like this
> right now?
It was found during work on blockers. JS errors have a tendency to just cause parts of the app to stop working. Brain (and others) have periodically run out of blockers, so I was fine with this fix.
Error is no longer seen in the logs on Nightly 16.0a1 2012-06-12 and Aurora 15.0a2 2012-06-11 on HTC Desire running Android 2.2. Closing the issue as verified.