User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-GB; rv:18.104.22.168) Gecko/20100722 Firefox/3.6.8 Build Identifier: N900 maemo browser When there is no active network connection and the Maemo browser is started and then closed with excatcly right timing, the "Select connection" dialog may appear. If you get the timing right, the browser attempts to close, but it is not properly shut down as nsMaemoNetworkManager is Waiting on a monitor that will no longer be Notified. Reproducible: Always Steps to Reproduce: PRECONDITIONS: - No internet connnection, WLAN as automatic selection mode - no any paired WLANs awailable STEPS LEADING TO PROBLEM: 1. Pick up N900. Tap some browser shortcut on desktop 2. Wait for a moment when browser opens and starts to load page 3. Tap browser to desktop mode (right down corner) (or set it up to open in non-full screen mode) 4. Close browser (upper right corner) Do this before connection selection dialog pops up (step 6)! (Timing this is quite tricky, repeat until you get it right) 5. Browser closes down 6. Connection selection dialog opens 7. Do not select a connection just tap outside dialog to cancel it Actual Results: Browserd is not closed cleanly. Expected Results: Browserd is closed cleanly. Also on current Maemo builds the browser UI process takes all the CPU it can get. But that is not Mozilla issue.
Created attachment 467364 [details] [diff] [review] Patch With this patch we listen to quit-application instead of xpcom-shutdown and when shutdown is detected, change the gInternalState on indicate that. And then we can break out of the while-loop and not wait for something that will never again be notified. The reason for the bug is basically that in the test sequence, xpcon-shutdown is never sent so the nsMaemoNetworkManager::Shutdown is never called and the Monitor never notified.
Comment on attachment 467364 [details] [diff] [review] Patch Not sure if this is still relevant but r=biesi
Attachment #467364 - Flags: review?(cbiesinger) → review+
Status: UNCONFIRMED → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.