Open Bug 1704574 Opened 4 years ago Updated 3 years ago

[SHIP] Fix the `documentHasChildNodes` value for about:blank documents

Categories

(Firefox :: Session Restore, defect)

defect

Tracking

()

Fission Milestone Future

People

(Reporter: u608768, Unassigned)

References

(Blocks 1 open bug)

Details

It's used to cache the document body that is eventually passed to SessionHistory.collectFromParent(), and used here. For the content-history case, we use the content document body, but here we're passing the body of the browser element, which is wrong.

We'll probably have to make it possible for the parent to know if the content body has child nodes.

Note that the patches for bug 1700963 will remove this property from the listener, and stop passing a body parameter to SessionHistory.collectFromParent(). We'll use this bug to correctly add the hasChildNodes() behavior.

Summary: Remove the _lastKnownBody property of the SessionStore SHistoryListener → [SHIP] Fix the `documentHasChildNodes` value for about:blank documents

Only applies to non-blank about:blank documents, so this can be pushed out of M7a.

Fission Milestone: M7a → MVP

Nika says this bug probably doesn't need to block Fission MVP since it is a corner case that hasn't been a problem yet. Tentatively moving from Fission Milestone MVP to Future.

If you disagree, just let me know or reset the Fission Milestone back to MVP.

Agreed that this can be Fission:Future.

Fission Milestone: MVP → Future

Clearing assignee because Kashav is no longer at Mozilla.

Assignee: kshvmdn+bmo → nobody
Status: ASSIGNED → NEW
You need to log in before you can comment on or make changes to this bug.