Closed Bug 1816898 Opened 2 years ago Closed 2 years ago

comm Perma docshell/test/unit/test_allowJavascript.js | xpcshell return code: 0

Categories

(Thunderbird :: Upstream Synchronization, defect, P5)

Thunderbird 112

Tracking

(Not tracked)

RESOLVED FIXED
112 Branch

People

(Reporter: intermittent-bug-filer, Assigned: jdescottes)

References

(Regression)

Details

(Keywords: intermittent-failure, intermittent-testcase, regression, Whiteboard: [stockwell disable-recommended])

Attachments

(1 file)

Filed by: martin [at] humanoids.be
Parsed log: https://treeherder.mozilla.org/logviewer?job_id=405711493&repo=comm-central
Full log: https://firefox-ci-tc.services.mozilla.com/api/queue/v1/task/QUAHgAwmTo2bGqjOkVGvbQ/runs/0/artifacts/public/logs/live_backing.log


ERROR Unexpected exception TypeError: can't access property "wrappedJSObject", this.contentWindow is null at resource://test/AllowJavascriptChild.sys.mjs:7
receiveMessage@resource://test/AllowJavascriptChild.sys.mjs:7:9

Version: unspecified → Thunderbird 112

Julian, you added this part of the test. Any idea why it would be failing on Thunderbird only? Some preference, perhaps?

Flags: needinfo?(jdescottes)

No idea, it seems to throw around:

  await page.loadURL("http://example.org/?other");

  await assertLoadFired(
    bc,
    undefined,
    "top BC with scripts disabled after navigation"
  );

which is not directly related to what I want to test.

I use the loadURL helper:

  async loadURL(url, redirectUrl = undefined) {
    await this.browserReady;

    this.browser.fixupAndLoadURIString(url, {
      triggeringPrincipal: Services.scriptSecurityManager.getSystemPrincipal(),
    });
    return promiseBrowserLoaded(this.browser, url, redirectUrl);
  }

I wonder if the browsing context gets replaced after this navigation on TB?

Could you try to add a bc = page.browsingContext; at L276 in this test and see if this helps (I don't have a dev environment setup for thunderbird, sorry :/)

Flags: needinfo?(jdescottes) → needinfo?(geoff)

Adding that does fix the test.

Flags: needinfo?(geoff)
Assignee: nobody → jdescottes
Status: NEW → ASSIGNED

Thanks for checking I can ask for a quick review

The browsing context might be replaced after the navigation

Pushed by jdescottes@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/0d780b3bf959 Fix docshell/test/unit/test_allowJavascript.js for thunderbird r=nika
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 112 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: