Closed Bug 1700719 Opened 4 years ago Closed 4 years ago

Crash in [@ mozilla::dom::CanonicalBrowsingContext::RequestRestoreTabContent]

Categories

(Firefox :: Session Restore, defect, P1)

defect

Tracking

()

RESOLVED FIXED
89 Branch
Fission Milestone M7a
Tracking Status
firefox-esr78 --- unaffected
firefox86 --- unaffected
firefox87 --- unaffected
firefox88 --- unaffected
firefox89 + fixed

People

(Reporter: aryx, Assigned: u608768)

References

(Regression)

Details

(Keywords: crash, regression)

Crash Data

Attachments

(1 file)

3 crashes on 2+ machines in code landed in bug 1597499.

Maybe Fission related. (DOMFissionEnabled=1)

Crash report: https://crash-stats.mozilla.org/report/index/835d067d-3401-4524-afdc-d2fcb0210324

MOZ_CRASH Reason: MOZ_DIAGNOSTIC_ASSERT(!context->IsDiscarded())

Top 10 frames of crashing thread:

0 XUL mozilla::dom::CanonicalBrowsingContext::RequestRestoreTabContent docshell/base/CanonicalBrowsingContext.cpp:1752
1 XUL mozilla::dom::WindowGlobalParent::RecvRequestRestoreTabContent dom/ipc/WindowGlobalParent.cpp:1092
2 XUL mozilla::dom::PWindowGlobalParent::OnMessageReceived ipc/ipdl/PWindowGlobalParent.cpp:1501
3 XUL mozilla::dom::PContentParent::OnMessageReceived ipc/ipdl/PContentParent.cpp:6484
4 XUL mozilla::ipc::MessageChannel::DispatchMessage ipc/glue/MessageChannel.cpp:2078
5 XUL mozilla::ipc::MessageChannel::MessageTask::Run ipc/glue/MessageChannel.cpp:1957
6 XUL mozilla::TaskController::DoExecuteNextTaskOnlyMainThreadInternal xpcom/threads/TaskController.cpp:754
7 XUL mozilla::detail::RunnableFunction<mozilla::TaskController::InitializeInternal xpcom/threads/nsThreadUtils.h:534
8 XUL nsThread::ProcessNextEvent xpcom/threads/nsThread.cpp:1155
9 XUL NS_ProcessPendingEvents xpcom/threads/nsThreadUtils.cpp:496
Flags: needinfo?(kmadan)

We're discard-checking the top-level context but not WindowGlobalParent::mBrowsingContext.

Assignee: nobody → kmadan
Status: NEW → ASSIGNED
Flags: needinfo?(kmadan)
Pushed by kmadan@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/51fc422f096f Discard-check both the top-level and current context before restoring, r=nika

Backed out for causing browser-chrome failures in CanonicalBrowsingContext.

Backout link: https://hg.mozilla.org/integration/autoland/rev/2b5850c18e69bcde444d19e189a806642dcff9b8

Push with failures

Failure log

Flags: needinfo?(kmadan)
Flags: needinfo?(kmadan)
Attachment #9211497 - Attachment description: Bug 1700719 - Discard-check both the top-level and current context before restoring, r?nika → Bug 1700719 - Discard-check all ancestors before restoring, r?nika
Pushed by kmadan@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/0dadbfb3feca Discard-check all ancestors before restoring, r=nika
Severity: -- → S2
Fission Milestone: --- → M7a
Priority: -- → P1
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → 89 Branch
Has Regression Range: --- → yes
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: