Closed Bug 1786565 Opened 2 years ago Closed 2 years ago

Fix and re-enable testNotificationDotOnMultipleWindows

Categories

(Firefox :: Firefox View, task, P2)

task
Points:
2

Tracking

()

RESOLVED FIXED
106 Branch
Tracking Status
firefox106 --- fixed

People

(Reporter: dao, Assigned: mconley)

References

(Blocks 1 open bug)

Details

(Whiteboard: [fidefe-firefox-view])

Attachments

(1 file)

When updating browser_notification_dot.js for bug 1786293, I ran into a weird failure in testNotificationDotOnMultipleWindows which times out waiting for the content docshell to become inactive. In order to unblock enabling in Nightly, I'm going to disable that part of the test and look into it later.

Summary: Re-enable testNotificationDotOnMultipleWindows → Fix and re-enable testNotificationDotOnMultipleWindows
See Also: → 1782556
Severity: -- → S3
Priority: -- → P2
Assignee: nobody → mconley
Status: NEW → ASSIGNED

When I re-enabled this test locally, it passed for me on both macOS and Linux. I have a try push here where I've just gone ahead and re-enabled it without any other changes. If it comes back without any suspicious failures in browser_notification_dot.js, then whatever was originally causing it to fail might have gone away and we might be good to re-enable.

Looking at the try push from comment 1, looks like we may have gotten a freebie here. No timeouts from any of the retriggers, so I suspect the test can just be re-enabled.

Pushed by mconley@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/9449d37e02c2
Re-enable testNotificationDotOnMultipleWindows test in browser_notification_dot.js. r=dao
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 106 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: