Closed Bug 1697988 Opened 3 years ago Closed 3 years ago

Intermittent dom/ipc/tests/test_browsingcontext_currenturi.html | curURI has expected value - got "https://tls1.example.com/", expected "about:blank"

Categories

(Core :: DOM: Content Processes, defect, P5)

defect

Tracking

()

RESOLVED FIXED
Fission Milestone M7a

People

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

Details

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

Attachments

(2 files)

Filed by: ncsoregi [at] mozilla.com
Parsed log: https://treeherder.mozilla.org/logviewer?job_id=332944413&repo=autoland
Full log: https://firefox-ci-tc.services.mozilla.com/api/queue/v1/task/SdHAlc8tT1qdBPnJH_haLQ/runs/0/artifacts/public/logs/live_backing.log


[task 2021-03-12T02:52:47.557Z] 02:52:47     INFO - TEST-PASS | dom/ipc/tests/test_browsingcontext_currenturi.html | documentURI has the expected value 
[task 2021-03-12T02:52:47.557Z] 02:52:47     INFO - add_task | Leaving test test_frame
[task 2021-03-12T02:52:47.558Z] 02:52:47     INFO - add_task | Entering test test_tls1_frame
[task 2021-03-12T02:52:47.558Z] 02:52:47     INFO - currentWindowGlobal has updated in the parent!
[task 2021-03-12T02:52:47.558Z] 02:52:47     INFO - docURI=about:neterror?e=nssFailure2&u=https%3A//tls1.example.com/&c=UTF-8&d=%20, curURI=about:blank
[task 2021-03-12T02:52:47.559Z] 02:52:47     INFO - Buffered messages finished
[task 2021-03-12T02:52:47.559Z] 02:52:47     INFO - TEST-UNEXPECTED-FAIL | dom/ipc/tests/test_browsingcontext_currenturi.html | curURI has expected value - got "https://tls1.example.com/", expected "about:blank"
[task 2021-03-12T02:52:47.559Z] 02:52:47     INFO -     SimpleTest.is@SimpleTest/SimpleTest.js:500:14
[task 2021-03-12T02:52:47.559Z] 02:52:47     INFO -     test_tls1_frame@dom/ipc/tests/test_browsingcontext_currenturi.html:54:5
[task 2021-03-12T02:52:47.562Z] 02:52:47     INFO - TEST-PASS | dom/ipc/tests/test_browsingcontext_currenturi.html | documentURI starts with about:neterror ```

Neha, could you please redirect this to someone who can take a look?

Flags: needinfo?(nkochar)

This test was recently added by Nika in Bug 1663757, so NI'ing her.

Flags: needinfo?(nkochar) → needinfo?(nika)

this is perma fail (or close to it), we should skip this.

Fission Milestone: --- → ?
Assignee: nobody → apavel
Status: NEW → ASSIGNED
Assignee: apavel → nobody
Status: ASSIGNED → NEW
Keywords: leave-open
Whiteboard: [stockwell disabled]
Pushed by apavel@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/7af304ef1f6d
disable test_browsingcontext_currenturi.html on linux64, mac-qr and win-qr r=jmaher DONTBUILD

Nika's findings:
There's a very small window between the time which the test is listening for and when the actual state in the parent process updates. We probably just need to change the test. Issue is around https://searchfox.org/mozilla-central/rev/d7e344e956d9da2808ea33e1fe0f963ed10c142d/dom/ipc/tests/test_browsingcontext_currenturi.html#40-44 - this condition can fire too early.

Assignee: nobody → kmaglione+bmo
Status: NEW → ASSIGNED
Fission Milestone: ? → M7a
Flags: needinfo?(nika)
Pushed by maglione.k@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/b35470ea968b
Fix race in test_browsingcontext_currenturi.html. r=mccr8

Hey Kris, can this bug be set as fixed now?

Flags: needinfo?(kmaglione+bmo)
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Flags: needinfo?(kmaglione+bmo)
Keywords: leave-open
Resolution: --- → FIXED
Whiteboard: [stockwell disabled]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: