Open Bug 1816238 Opened 3 years ago Updated 6 days ago

Intermittent comm/mail/base/test/browser/browser_browserContext.js | single tracking bug

Categories

(Thunderbird :: Mail Window Front End, defect, P5)

Product:
Thunderbird
Component:
Mail Window Front End
Type:
defect

Tracking

(Not tracked)

Status:
REOPENED
Milestone:
111 Branch

People

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

References

Details

(Keywords: intermittent-failure, intermittent-testcase)

Attachments

(3 files)

Bug 1816238 - Fix testMessagePane of browser_browserContext.js by waiting for the correct page being loaded. r=#thunderbird-reviewers
48 bytes, text/x-phabricator-request
Details | Review
Bug 1816238 - Attempt to fix browser_browserContext.js. r=#thunderbird-reviewers
48 bytes, text/x-phabricator-request
Details | Review
Bug 1816238 - Use readyState to wait for content loads in tests. r=#thunderbird-reviewers
48 bytes, text/x-phabricator-request
Details | Review

Filed by: john [at] thunderbird.net
Parsed log: https://treeherder.mozilla.org/logviewer?job_id=405322078&repo=comm-central
Full log: https://firefox-ci-tc.services.mozilla.com/api/queue/v1/task/JXSDQgvHQV-3NSN8F9dQvA/runs/0/artifacts/public/logs/live_backing.log

Timing of the webBrowser loading a URL has changed with new mail tabs. Need to make sure the actual page has loaded, not about:blank
Assignee: nobody → john
Status: NEW → ASSIGNED
Keywords: checkin-needed-tb
Target Milestone: --- → 111 Branch

Pushed by mkmelin@iki.fi:
https://hg.mozilla.org/comm-central/rev/4642ddd0ec24
Fix testMessagePane of browser_browserContext.js by waiting for the correct page being loaded. r=mkmelin

Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Keywords: checkin-needed-tb
Resolution: --- → FIXED

Intermittent failures are still happening for this file.

Status: RESOLVED → REOPENED
Resolution: FIXED → ---

Bug 1860255 may have helped, but there's now - https://treeherder.mozilla.org/logviewer?job_id=455513724&repo=comm-central&lineNumber=2228 - Console message: [JavaScript Error: "TypeError: doc is null" {file: "resource:///actors/ContextMenuChild.sys.mjs" line: 559}] which obviously is the cause of the timeout in this case at least

See Also: → 1478596
Flags: needinfo?(john)

This is based on a comparison between the following tests:

  • /mail/base/test/browser/browser_browserContext.js
  • /mail/base/test/widgets/browser_formPickers.js

The two tests look similar. While the first one fails often,
the second one seems to run fine. The only difference I could
find is the second one is waiting for focus before clicking the
extension button. This does seem to fix the first test as well.

Pushed by john@thunderbird.net:
https://hg.mozilla.org/comm-central/rev/75ab9fac7c8d
Attempt to fix browser_browserContext.js. r=thunderbird-reviewers,babolivier

Status: REOPENED → RESOLVED
Closed: 3 years ago → 11 months ago
Resolution: --- → FIXED

grrr

Status: RESOLVED → REOPENED
Resolution: FIXED → ---

While investigating timeouts of browser_browserContext.js, it was
observed that the condition to wait for a load event was triggered by
browser.webProgress.isLoadingDocument being true, but the load event
was never seen.

The readyState however was already complete. This patch therefore
changes the wait-for-load condition of content loads, here and in
similar tests.

Attachment #9436372 - Attachment description: Bug 1816238 - Use readyState to wait for content loads in tests. r=mkmelin → Bug 1816238 - Use readyState to wait for content loads in tests. r=#thunderbird-reviewers
Flags: needinfo?(john)
Keywords: checkin-needed-tb

Pushed by arschmitz@thunderbird.net:
https://hg.mozilla.org/comm-central/rev/9cb48a799aa6
Use readyState to wait for content loads in tests. r=mkmelin

Status: REOPENED → RESOLVED
Closed: 11 months ago → 11 months ago
Keywords: checkin-needed-tb
Resolution: --- → FIXED

Seems still to occur, maybe a different cause.

https://treeherder.mozilla.org/jobs?repo=comm-central&selectedTaskRun=PLVEjy9fQW2VeHlkmTgwkA.0

[task 2025-03-04T11:41:00.572Z] 11:41:00     INFO - Entering test bound testExtensionMessageDisplayAction
[task 2025-03-04T11:41:00.572Z] 11:41:00     INFO - Extension loaded
[task 2025-03-04T11:41:00.573Z] 11:41:00     INFO - Buffered messages logged at 11:39:40
[task 2025-03-04T11:41:00.573Z] 11:41:00     INFO - Console message: [JavaScript Warning: "initNSMouseEvent() is deprecated. Use the MouseEvent() constructor instead." {file: "chrome://messenger/content/browserPopups.js" line: 115}]
[task 2025-03-04T11:41:00.574Z] 11:41:00     INFO - TEST-PASS | comm/mail/base/test/browser/browser_browserContext.js | "undefined" != "closed" - 
[task 2025-03-04T11:41:00.575Z] 11:41:00     INFO - TEST-PASS | comm/mail/base/test/browser/browser_browserContext.js | ["browserContext-back","browserContext-forward","browserContext-reload"] deepEqual ["browserContext-back","browserContext-forward","browserContext-reload"] - 
[task 2025-03-04T11:41:00.575Z] 11:41:00     INFO - TEST-PASS | comm/mail/base/test/browser/browser_browserContext.js | "open" != "closed" - 
[task 2025-03-04T11:41:00.576Z] 11:41:00     INFO - TEST-PASS | comm/mail/base/test/browser/browser_browserContext.js | ["context-navigation","browserContext-selectall"] deepEqual ["context-navigation","browserContext-selectall"] - 
[task 2025-03-04T11:41:00.577Z] 11:41:00     INFO - TEST-PASS | comm/mail/base/test/browser/browser_browserContext.js | "open" != "closed" - 
[task 2025-03-04T11:41:00.578Z] 11:41:00     INFO - TEST-PASS | comm/mail/base/test/browser/browser_browserContext.js | ["browserContext-openLinkInBrowser","browserContext-selectall","browserContext-copylink","browserContext-savelink"] deepEqual ["browserContext-openLinkInBrowser","browserContext-selectall","browserContext-copylink","browserContext-savelink"] - 
[task 2025-03-04T11:41:00.579Z] 11:41:00     INFO - TEST-PASS | comm/mail/base/test/browser/browser_browserContext.js | "Link to a web page" == "Link to a web page" - 
[task 2025-03-04T11:41:00.580Z] 11:41:00     INFO - TEST-PASS | comm/mail/base/test/browser/browser_browserContext.js | "open" != "closed" - 
[task 2025-03-04T11:41:00.581Z] 11:41:00     INFO - TEST-PASS | comm/mail/base/test/browser/browser_browserContext.js | ["browserContext-undo","browserContext-cut","browserContext-copy","browserContext-paste","browserContext-selectall","browserContext-spell-check-enabled"] deepEqual ["browserContext-undo","browserContext-cut","browserContext-copy","browserContext-paste","browserContext-selectall","browserContext-spell-check-enabled"] - 
[task 2025-03-04T11:41:00.581Z] 11:41:00     INFO - Buffered messages finished
[task 2025-03-04T11:41:00.582Z] 11:41:00     INFO - TEST-UNEXPECTED-FAIL | comm/mail/base/test/browser/browser_browserContext.js | Test timed out -
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: