1952681 - Fix up content script tests that are skipped when sessionHistoryInParent

Status: NEW

(WebExtensions :: General, task, P2)

Description

[Rob Wu [:robwu]]

Some content script tests are skipped with the condition sessionHistoryInParent. But SHIP is enabled by default, which means that these tests are effectively skipped on all configurations that match what users actually see.

I'm refactoring content script logic that affect timing (bug 1951688), and these tests cover relevant scenarios that are sensitive to timing.

I'm going to fix test_ext_contentscript_context.js and test_ext_contentscript_context_isolation.js and re-enable them (in xpcshell-common.toml).

While looking at the issue, I saw that test_ext_web_accessible_resources.js is also skipped when sessionHistoryInParent, and will fix that test too.

The skip-if of these tests are currently associated with bug 1762638.

Comment 1

[Rob Wu [:robwu]]

Attachment: Bug 1952681 - Fix and re-enable some xpcshell tests with SHIP

Some tests are skipped when sessionHistoryInParent. But SHIP is enabled
by default, so effectively these tests were skipped.

Fix the underlying issues in the tests and re-enable them.
This is mostly fixing the logic so that test state between
ContentPage.spawn calls are correctly carried over.

Several location.href navigations were replaced with
contentPage.loadURL (which awaits the load) because without it, the
tests would fail when run with --disable-fission.