Closed Bug 1147017 Opened 9 years ago Closed 3 years ago

Many files under browser/base/content/test/ use gBrowser.contentDocument and gBrowser.contentWindow, which doesn't exist in e10s browsers

Categories

(Firefox :: General, defect, P5)

Product:
Firefox
Component:
General
Platform:
x86
macOS
Type:
defect

Tracking

()

Status:
RESOLVED WONTFIX
Tracking Flags:
Tracking Status
e10s later ---

People

(Reporter: mconley, Unassigned)

References

(Blocks 1 open bug)

Details

Bug 1144149 will add contentDocumentAsCPOW and contentWindowAsCPOW shortcuts to gBrowser.

The tests in browser_aboutHome.js should be updated to use those instead.
tracking-e10s: --- → ?
Blocks: e10s-tests
It turns out a number of tests use these accessors. Note that this probably isn't high-priority, since the tests will be using the add-on shims, so they'll actually get the CPOWs automatically through contentWindow and contentDocument.

Test files with gBrowser.contentDocument and / or gBrowser.contentWindow:

browser/base/content/test/general/browser.ini
browser/base/content/test/general/browser_aboutHome.js
browser/base/content/test/general/browser_addKeywordSearch.js
browser/base/content/test/general/browser_bug329212.js
browser/base/content/test/general/browser_bug331772_xul_tooltiptext_in_html.js
browser/base/content/test/general/browser_bug423833.js
browser/base/content/test/general/browser_bug424101.js
browser/base/content/test/general/browser_bug431826.js
browser/base/content/test/general/browser_bug435325.js
browser/base/content/test/general/browser_bug460146.js
browser/base/content/test/general/browser_bug517902.js
browser/base/content/test/general/browser_bug561623.js
browser/base/content/test/general/browser_bug561636.js
browser/base/content/test/general/browser_bug581947.js
browser/base/content/test/general/browser_bug592338.js
browser/base/content/test/general/browser_bug633691.js
browser/base/content/test/general/browser_bug676619.js
browser/base/content/test/general/browser_bug839103.js
browser/base/content/test/general/browser_bug970746.js
browser/base/content/test/general/browser_keywordSearch_postData.js
browser/base/content/test/general/browser_locationBarCommand.js
browser/base/content/test/general/browser_locationBarExternalLoad.js
browser/base/content/test/general/browser_readerMode.js
browser/base/content/test/general/browser_save_private_link_perwindowpb.js
browser/base/content/test/general/browser_save_video.js
browser/base/content/test/general/browser_urlbarStop.js
browser/base/content/test/newtab/head.js
browser/base/content/test/social/browser_aboutHome_activation.js
browser/base/content/test/social/browser_social_activation.js
Summary: content/test/general/browser_aboutHome.js uses gBrowser.contentDocument and gBrowser.contentWindow, which doesn't exist in e10s browsers → Many files under content/test/general/ use gBrowser.contentDocument and gBrowser.contentWindow, which doesn't exist in e10s browsers
Summary: Many files under content/test/general/ use gBrowser.contentDocument and gBrowser.contentWindow, which doesn't exist in e10s browsers → Many files under content/test/ use gBrowser.contentDocument and gBrowser.contentWindow, which doesn't exist in e10s browsers
Summary: Many files under content/test/ use gBrowser.contentDocument and gBrowser.contentWindow, which doesn't exist in e10s browsers → Many files under browser/base/content/test/ use gBrowser.contentDocument and gBrowser.contentWindow, which doesn't exist in e10s browsers
Priority: -- → P5

CPOWs have been removed.

Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.