Closed Bug 1666499 Opened 4 years ago Closed 27 days ago

Perma CDP TEST-UNEXPECTED-PASS | Page Page.close should *not* run beforeunload by default (page.spec.ts) | expected TIMEOUT

Categories

(Remote Protocol :: Agent, defect, P5)

defect

Tracking

(firefox-esr78 unaffected, firefox80 unaffected, firefox81 unaffected, firefox82 unaffected, firefox83 affected)

RESOLVED WONTFIX
Tracking Status
firefox-esr78 --- unaffected
firefox80 --- unaffected
firefox81 --- unaffected
firefox82 --- unaffected
firefox83 --- affected

People

(Reporter: intermittent-bug-filer, Unassigned)

References

(Regression)

Details

(Keywords: intermittent-failure, regression)

Attachments

(1 obsolete file)

Filed by: rmaries [at] mozilla.com
Parsed log: https://treeherder.mozilla.org/logviewer.html#?job_id=316365645&repo=mozilla-central
Full log: https://firefox-ci-tc.services.mozilla.com/api/queue/v1/task/W77dGEcQT5K2pS1lOvvS8A/runs/0/artifacts/public/logs/live_backing.log


[task 2020-09-22T11:54:13.389Z] PID 200 | ["test-start",{"title":"should *not* run beforeunload by default","fullTitle":"Page Page.close should *not* run beforeunload by default","file":"/builds/worker/checkouts/gecko/remote/test/puppeteer/test/page.spec.ts","currentRetry":0}]

[task 2020-09-22T11:54:14.153Z] PID 200 | JavaScript error: chrome://remote/content/domains/parent/page/DialogHandler.jsm, line 103: TypeError: can't access property "listPrompts", this._browser.tabModalPromptBox is undefined
[task 2020-09-22T11:54:15.155Z] PID 200 | 1600775655154	RemoteAgent	TRACE	<-(connection {73cdd381-1b81-4e7b-be4a-3dad2264caad}) {"method":"Target.targetDestroyed","params":{"targetId":"fa5fd86d-089a-4901-9824-4d4f64dfa044"}}
[task 2020-09-22T11:54:15.158Z] PID 200 | 1600775655157	RemoteAgent	TRACE	<-(connection {73cdd381-1b81-4e7b-be4a-3dad2264caad}) {"id":159}
[task 2020-09-22T11:54:15.159Z] TEST-UNEXPECTED-PASS | Page Page.close should *not* run beforeunload by default (page.spec.ts) | expected TIMEOUT
[task 2020-09-22T11:54:15.159Z] TEST-INFO expected TIMEOUT | took 1769ms
[task 2020-09-22T11:54:15.159Z] PID 200 | ["pass",{"title":"should *not* run beforeunload by default","fullTitle":"Page Page.close should *not* run beforeunload by default","file":"/builds/worker/checkouts/gecko/remote/test/puppeteer/test/page.spec.ts","duration":1481,"currentRetry":0}]
[task 2020-09-22T11:54:15.165Z] PID 200 | 1600775655165	RemoteAgent	TRACE	(connection {73cdd381-1b81-4e7b-be4a-3dad2264caad})-> {"method":"Target.disposeBrowserContext","params":{"browserContextId":9},"id":160}
[task 2020-09-22T11:54:15.173Z] PID 200 | 1600775655172	RemoteAgent	TRACE	<-(connection {73cdd381-1b81-4e7b-be4a-3dad2264caad}) {"method":"Target.targetDestroyed","params":{"targetId":"2dac6cbb-4066-44b7-af69-44984345170f"}}
[task 2020-09-22T11:54:15.180Z] PID 200 | 1600775655178	RemoteAgent	TRACE	<-(connection {73cdd381-1b81-4e7b-be4a-3dad2264caad}) {"id":160}```
Summary: Intermittent TEST-UNEXPECTED-PASS | Page Page.close should *not* run beforeunload by default (page.spec.ts) | expected TIMEOUT → Perma tier 2 TEST-UNEXPECTED-PASS | Page Page.close should *not* run beforeunload by default (page.spec.ts) | expected TIMEOUT

I did some retriggers for the last days on central. Looks like its a new regression probably caused by the sub dialog.

Changed from expected timeout to unexpected pass for this mozilla-central merge:
https://treeherder.mozilla.org/#/jobs?repo=mozilla-central&searchStr=pup&revision=fa0bf905d4cb944368ddf8fbe38cca4107692fd4

So most likely bug 1655866 fixed it.

Assignee: nobody → hskupin
Status: NEW → ASSIGNED
Regressed by: 1655866
Fission Milestone: --- → ?
Pushed by hskupin@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/35066da8f3c3 [remote] Updated expectation status for test "Page.close should *not* run beforeunload by default" r=remote-protocol-reviewers,maja_zf
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → 83 Branch

Clearing Fission Milestone "?" flag for this bug that has already been resolved.

Fission Milestone: ? → ---

The unexpected pass was by accident here. It was actually a regression that was fixed by bug 1666670 now.

Please back out so that we can get bug 1668728 fixed.

Status: RESOLVED → REOPENED
Depends on: 1666670
Flags: needinfo?(sheriffs)
Resolution: FIXED → ---

With the backout the test is marked as expected fail again. So lets keep it open until we have figured out what's wrong.

Assignee: hskupin → nobody
Status: REOPENED → NEW
Keywords: leave-open
Assignee: nobody → hskupin
Status: NEW → ASSIGNED
Assignee: hskupin → nobody
Status: ASSIGNED → NEW
Attachment #9177161 - Attachment is obsolete: true
Component: Agent → CDP
Component: CDP → Agent
Severity: normal → S3

This is a CDP failure and marked as SKIP at the moment. We are not going to investigate and fix this.

Status: NEW → RESOLVED
Closed: 4 years ago27 days ago
Keywords: leave-openregression
Resolution: --- → WONTFIX
Summary: Perma tier 2 TEST-UNEXPECTED-PASS | Page Page.close should *not* run beforeunload by default (page.spec.ts) | expected TIMEOUT → Perma CDP TEST-UNEXPECTED-PASS | Page Page.close should *not* run beforeunload by default (page.spec.ts) | expected TIMEOUT
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: