Closed Bug 868681 Opened 8 years ago Closed 8 years ago
Offline apps doorhanger not shown
Copied the lazy getter code used in aboutAddons.js -- works for me and allows robocop's (currently disabled) testDoorHanger to succeed.
Attachment #745453 - Flags: review?(mark.finkle)
Comment on attachment 745453 [details] [diff] [review] define gChromeWin This was a bad review on my part. OfflineApps is injected into the browser.js scope, so BrowserApp.manifest should work fine without the need for a "chromeWindow". Try this line: let requestor = BrowserApp.manifest ? "'" + BrowserApp.manifest.name + "'" : host;
Attachment #745453 - Flags: review?(mark.finkle) → review-
That's better, and works just as well - thanks!
Attachment #745667 - Flags: review?(mark.finkle) → review+
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 23
Verified fixed on: -build: Firefox for Android 23.0a1(2013-05-12) -device: Samsung Galaxy Nexus -OS: Android 4.1.1
Status: RESOLVED → VERIFIED
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.