Closed Bug 1599232 Opened 5 years ago Closed 5 years ago

Crash in [@ mozilla::ipc::WriteIPDLParam<T> | mozilla::dom::PSHistoryChild::SendAddEntry]

Categories

(Core :: DOM: Navigation, defect)

Unspecified
macOS
defect
Not set
normal

Tracking

()

RESOLVED WONTFIX
Tracking Status
firefox71 --- unaffected
firefox72 --- affected

People

(Reporter: marcia, Unassigned)

References

(Regression)

Details

(Keywords: crash, regression)

Crash Data

This bug is for crash report bp-5da3c1d2-d458-4cbd-9140-4a6000191108.

macOS specific nightly signature which dates back to 20191108095936: https://bit.ly/2QRiG7k

The other similiar signature started on the same date: https://bit.ly/34gixhs

Possible regression range based on Build ID: https://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=d271c572a9bcd008ed14bf104b2eb81949952e4c&tochange=f414b9e6d85710f92649566c1c6511265dadd476

Top 10 frames of crashing thread:

0 XUL void mozilla::ipc::WriteIPDLParam<mozilla::dom::PSHEntryChild*&> ipc/glue/IPDLParamTraits.h:60
1 XUL mozilla::dom::PSHistoryChild::SendAddEntry ipc/ipdl/PSHistoryChild.cpp:586
2 XUL non-virtual thunk to mozilla::dom::SHistoryChild::AddEntry docshell/shistory/SHistoryChild.cpp
3 XUL nsDocShell::AddToSessionHistory docshell/base/nsDocShell.cpp:11504
4 XUL nsDocShell::UpdateURLAndHistory docshell/base/nsDocShell.cpp:11116
5 XUL nsDocShell::AddState docshell/base/nsDocShell.cpp:11069
6 XUL nsHistory::PushState dom/base/nsHistory.cpp:216
7 XUL mozilla::dom::History_Binding::pushState dom/bindings/HistoryBinding.cpp:363
8 XUL bool mozilla::dom::binding_detail::GenericMethod<mozilla::dom::binding_detail::NormalThisPolicy, mozilla::dom::binding_detail::ThrowExceptions> dom/bindings/BindingUtils.cpp:3218
9 XUL js::InternalCallOrConstruct js/src/vm/Interpreter.cpp:548

Component: IPC → DOM: Navigation

Two URLs shows up between the 2 signatures - https://discordapp.com and https://hg.mozilla.org/mozilla-central/rev/055e0a0e4bf0. Around 12 macOS crashes only since this was reported.

perhaps regressed by bug 1588491?

Flags: needinfo?(peterv)
Regressed by: 1588491

This code can only run when enabling the fission.sessionHistoryInParent pref. We're doing a redesign of that code, so I think this is just a WONTFIX for now.

Status: NEW → RESOLVED
Closed: 5 years ago
Flags: needinfo?(peterv)
Resolution: --- → WONTFIX
Has Regression Range: --- → yes
You need to log in before you can comment on or make changes to this bug.