1093155 - e10s - Fix browser_417483.js and other browser/base/content/test/general tests relying on using popups with content

Status: NEW

(Firefox :: Menus, defect)

Description

[:Gijs (he/him)]

As in summary. The current comment references about:home; not sure why as the test has nothing to do with about:home, so I'm filing a separate bug and fixing the comment in a bit. Trying to run the test under e10s gives me:

300 INFO TEST-START | chrome://mochitests/content/browser/browser/base/content/test/general/browser_bug417483.js
301 INFO Console message: [JavaScript Error: "The character encoding of the HTML document was not declared. The document will render with garbled text in some browser configurations if the document contains characters from outside the US-ASCII range. The character encoding of the page must be declared in the document or in the transfer protocol." {file: "data:text/html,%20<iframe%20src='data:text/html,text%20text'></iframe>" line: 0}]
302 INFO Console message: [JavaScript Error: "Value being assigned to XULDocument.popupNode does not implement interface Node." {file: "chrome://mochitests/content/browser/browser/base/content/test/general/browser_bug417483.js" line: 20}]


I suspect that really, what needs to happen here is for the test to be converted to mochitest-plain; test_contextmenu.js in the same dir works in e10s mode and has pretty similar requirements.

Comment 1

[:Gijs (he/him)]

List of tests in browser/base/content/test/general which use document.popupNode (all are currently e10s-disabled):

browser_bug417483.js
browser_bug423833.js [perma-disabled everywhere?!]
browser_bug424101.js 
browser_bug734076.js
browser_bug970746.js
browser_plainTextLinks.js

Comment 2

[Neil Deakin]

Going to have separate bugs for each test. Bug 1158805 is for browser_bug970746.js and 1158810 is for browser_bug424101.js.