Closed Bug 1582113 Opened 4 months ago Closed 3 months ago

test_autofill_from_bfcache.html fails with fission enabled

Categories

(Toolkit :: Password Manager, defect, P2)

defect

Tracking

()

RESOLVED FIXED
mozilla71
Fission Milestone M4
Tracking Status
firefox71 --- fixed

People

(Reporter: enndeakin, Assigned: MattN)

References

(Blocks 1 open bug)

Details

Attachments

(2 files)

The test:

  1. The pageHideEvent.persisted is false instead of true
  2. Cannot navigate back in history near the end of the test
Priority: -- → P2

(In reply to Neil Deakin from comment #0)

  1. The pageHideEvent.persisted is false instead of true

Is there a DOM bug filed on this? It sounds like we'll need a DOM fix.

  1. Cannot navigate back in history near the end of the test

That can easily be solved with SpecialPowers.spawnTask AFAICT so I think we're mostly blocked on the above.

Flags: needinfo?(enndeakin)

(In reply to Matthew N. [:MattN] (PM me if requests are blocking you) from comment #1)

(In reply to Neil Deakin from comment #0)

  1. The pageHideEvent.persisted is false instead of true

Is there a DOM bug filed on this? It sounds like we'll need a DOM fix.

I don't think I filed one.

Flags: needinfo?(enndeakin)

(In reply to Neil Deakin from comment #2)

(In reply to Matthew N. [:MattN] (PM me if requests are blocking you) from comment #1)

(In reply to Neil Deakin from comment #0)

  1. The pageHideEvent.persisted is false instead of true

Is there a DOM bug filed on this? It sounds like we'll need a DOM fix.

I don't think I filed one.

Hi Peter, is it intentional with Fission that a page doesn't get persisted in bfcache (or maybe just pageHideEvent.persisted lies?) when navigating cross-origin? Or will this get fixed by a dependency of bug 1467221? The test that relies on testing bfcache is pretty straightforward.

Assignee: nobody → MattN+bmo
Status: NEW → ASSIGNED
Flags: needinfo?(peterv)
Pushed by mozilla@noorenberghe.ca:
https://hg.mozilla.org/integration/autoland/rev/655fc6e0528b
Fix test_autofill_from_bfcache usage of history.back() for Fission. r=sfoster
Pushed by csabou@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/2830baa75b96
Enable test_autofill_highlight.html on fission cause now it's fixed. r=sfoster
Status: ASSIGNED → RESOLVED
Closed: 3 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla71

(In reply to Matthew N. [:MattN] (PM me if requests are blocking you) from comment #3)

Hi Peter, is it intentional with Fission that a page doesn't get persisted in bfcache (or maybe just pageHideEvent.persisted lies?) when navigating cross-origin? Or will this get fixed by a dependency of bug 1467221? The test that relies on testing bfcache is pretty straightforward.

At least nsDocShell::CanSavePresentation will need to be fixed for Fission (it uses GetInProcessSameTypeParent), but I don't know if that will fix this. But I'm a bit confused, the test is marked as completely fixed now, did the problem with persisted go away somehow?

Flags: needinfo?(peterv)
Blocks: 1592920

The test isn't fixed. Only part 2 was fixed by patches in this bug which erroneously closed the bug. Also, the second part appears to have enabled a different test.

I filed a new bug 1592920 on the remaining and marked it as dependent on the CanSavePresentation change.

Retroactively moving fixed bugs whose summaries mention "Fission" (or other Fission-related keywords) but are not assigned to a Fission Milestone to an appropriate Fission Milestone.

This will generate a lot of bugmail, so you can filter your bugmail for the following UUID and delete them en masse:

0ee3c76a-bc79-4eb2-8d12-05dc0b68e732

Fission Milestone: --- → M4
You need to log in before you can comment on or make changes to this bug.