Open Bug 1782759 Opened 2 years ago Updated 2 years ago

Add tests to ensure that browsing context flags are carried over across navigations in GV apps

Categories

(GeckoView :: Core, task, P2)

All
Android

Tracking

(Not tracked)

People

(Reporter: twisniewski, Assigned: owlish)

References

Details

(Whiteboard: [geckoview:m106] [geckoview:m107])

We have just fixed breakage related to this in bug #1781936, but that patch did not contain any tests to make sure it won't break again without us noticing. We should ensure that our tests check not only the initial browsing context on a tab, but also navigate to a new URL (to trigger a new browsing context) and ensure that it receives the correct flags as well.

Thomas, Irene says you wanted to add this GV test. If you need help writing the test, Irene can help.

Severity: -- → N/A
Type: defect → task
Priority: -- → P1

I'm afraid that I don't really have the time to write one right now, I was hoping she could find someone who does. I'll keep an eye on this in case I do find time, but I'm also inexperienced with writing these kinds of tests, so it might be better to get someone more familiar to write it.

No problem. I just wanted to confirm. I'll add this bug to the top of the GV team's backlog.

Priority: P1 → P2
Whiteboard: [geckoview:m105] → [geckoview:m106?]
Priority: P2 → P1
Whiteboard: [geckoview:m106?] → [geckoview:m106]
Assignee: nobody → bugzeeeeee

107

Whiteboard: [geckoview:m106] → [geckoview:m106] [geckoview:m107]
Priority: P1 → P2
Whiteboard: [geckoview:m106] [geckoview:m107] → [geckoview:m106] [geckoview:m107] [geckoview:m109?]
Whiteboard: [geckoview:m106] [geckoview:m107] [geckoview:m109?] → [geckoview:m106] [geckoview:m107]
You need to log in before you can comment on or make changes to this bug.