The default bug view has changed. See this FAQ.

browser_bug435325.js shouldn't reference nsIDOMWindowUtils, should be moved to /browser/

RESOLVED FIXED in mozilla17

Status

()

Core
Document Navigation
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: Unfocused, Assigned: Unfocused)

Tracking

unspecified
mozilla17
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

Bug 779680 removed usage of nsIDOMWindowUtils.goOnline() in about:neterror. This had been used to go online when clicking "try again" after a page failed to load due to be in Offline Mode. However, browser_bug435325.js, which tests that functionality, still makes reference to nsIDOMWindowUtils in its log output.

Arguably, that test should be in moved to /browser/ (from docshell), since that functionality was moved into browser chrome.
Blocks: 782892

Comment 1

5 years ago
Is there some sort of B2G API that can be used to kick Firefox online so you don't have to rely on something in /browser ?
You mean WebAPI? None that I'm aware of, nor any plans. At least in terms of Firefox, Offline Mode is kinda-sorta being slowly deprecated anyway (as evidenced by its move to the Developer Tools menu).
Created attachment 653657 [details] [diff] [review]
Patch v1
Attachment #653657 - Flags: review?(dolske)
Attachment #653657 - Flags: review?(dolske) → review+
https://hg.mozilla.org/integration/fx-team/rev/bface603ef72
Target Milestone: --- → mozilla17
https://hg.mozilla.org/mozilla-central/rev/bface603ef72
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.