1733886 - Don't import `DocumentManager` in extensions tests

Status: RESOLVED FIXED

(WebExtensions :: General, task, P3)

Description

[Mathew Hodson]

+++ This bug was initially created as a clone of Bug #1733851 +++

DocumentManager is used in three tests:

• toolkit/components/extensions/test/xpcshell/test_ext_contentscript_context.js
• toolkit/components/extensions/test/xpcshell/test_ext_contentscript_context_isolation.js
• toolkit/components/extensions/test/xpcshell/test_ext_contexts_gc.js

It can be replaced with a calls to ExtensionProcessScript.getExtensionChild(extensionId); and ExtensionContent.getContext(ext, content);

Comment 1

[Mathew Hodson]

Attachment: Bug 1733886 - Don't import `DocumentManager` in extensions tests. r?robwu

Depends on D127388

Comment 2

[Tomislav Jovanovic :zombie]

Changing severity to NA because it's a task.

Comment 3

[BugBot [:suhaib / :marco/ :calixte]]

There's a r+ patch which didn't land and no activity in this bug for 2 weeks.
:mathew.hodson, could you have a look please?
For more information, please visit auto_nag documentation.

Comment 4

[Rob Wu [:robwu]]

I've just pushed to land

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/926181ea1e20
Don't import `DocumentManager` in extensions tests. r=robwu

Comment 6

[Alexandru Michis [:malexandru]]

https://hg.mozilla.org/mozilla-central/rev/926181ea1e20