Closed Bug 624630 Opened 9 years ago Closed 9 years ago
[JS Error] browser
_addons .js message listener is stuck and UIReady Delayed is busted
I also see issues with FormHelper and FindHelper. It looks like we are not making it all the way through the BrowserUI.UIReadyDelayed code.
Summary: [JS Error] browser_addons.js message listener is stuck → [JS Error] browser_addons.js message listener is stuck and UIReadyDelayed is busted
Found the UIReadyDelayed issue. Bustage from bug 620975. nsIAlertsProgressListener.onCancel is impl'ed on other platforms, buts throws. We need to wrap the code in #ifdef ANDROID. The browser_adons.js stuck message handler is fixed if we add the tab first and use a setTimeout(...0) for the callback.
Fixes. Tests are green locally with this.
Assignee: nobody → mark.finkle
Comment on attachment 502729 [details] [diff] [review] patch I'm landing this to fix the bustage from bug 620975, but please review for sanity.
Attachment #502729 - Flags: review?(21)
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Attachment #502729 - Flags: review?(21) → review+
Verified fixed according to Tinderboxpushlog results
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.