Closed Bug 1673952 Opened 4 years ago Closed 5 months ago

[gv-junit-fis] Run ContentDelegateMultipleSessionsTest on Fission

Categories

(GeckoView :: Sandboxing, task, P2)

Unspecified
All

Tracking

(firefox121 fixed)

RESOLVED FIXED
121 Branch
Tracking Status
firefox121 --- fixed

People

(Reporter: agi, Assigned: owlish)

References

(Blocks 1 open bug)

Details

(Whiteboard: [fission:android:m2] [geckoview:2022q3][fxdroid])

Attachments

(1 file)

This bug tracks enabling all ContentDelegateMultipleSessionsTest to run on Fission.

Severity: -- → N/A
Priority: -- → P2
Fission Milestone: --- → Future

Tracking this bug for Android Fission milestone M2 (pass tests with Fission enabled).

Whiteboard: [fission:android:m2]
Fission Milestone: Future → ---
Whiteboard: [fission:android:m2] → [fission:android:m2] [geckoview:2022q3?]
Whiteboard: [fission:android:m2] [geckoview:2022q3?] → [fission:android:m2] [geckoview:2022q3]

Moving Android Fission bugs to the new GeckoView::Sandboxing component.

Component: General → Sandboxing
Summary: Run ContentDelegateMultipleSessionsTest on Fission → [gv-junit-fis] Run ContentDelegateMultipleSessionsTest on Fission
Assignee: nobody → bugzeeeeee
Assignee: bugzeeeeee → nobody
Whiteboard: [fission:android:m2] [geckoview:2022q3] → [fission:android:m2] [geckoview:2022q3][fxdroid]
Assignee: nobody → bugzeeeeee

In Fission, when this particular navigation happens, we switch process. It's an expected Fission behavior. However, because we switch process, we can no longer test if multiple sessions sharing the same process receive the onCrash or onKill calls. Originally, the navigation was added to the test to get rid of a race condition, when the first tab would crash while the second was still initializing. That is no longer a danger for this test as it has evolved and now we have lots of things happening between initializing the second session and crashing the first one. So to fix the test, I am removing the navigation to make sure the tab stays in the same content process.

Pushed by istorozhko@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/0408d1b0f762
Remove navigation from crashContentMultipleSessions test r=geckoview-reviewers,boek
Status: NEW → RESOLVED
Closed: 5 months ago
Resolution: --- → FIXED
Target Milestone: --- → 121 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: