1543699 - Crash in [@ nsWebBrowser::Create]

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, defect, P1)

Description

[[:philipp]]

This bug is for crash report bp-9b64b64d-23c8-43ba-b14d-47a5f0190411.

Top 10 frames of crashing thread:

0 xul.dll nsWebBrowser::Create toolkit/components/browser/nsWebBrowser.cpp:107
1 xul.dll mozilla::dom::TabChild::Init dom/ipc/TabChild.cpp:526
2 xul.dll mozilla::dom::ContentChild::RecvPBrowserConstructor dom/ipc/ContentChild.cpp:1784
3 xul.dll mozilla::dom::PContentChild::OnMessageReceived ipc/ipdl/PContentChild.cpp:5574
4 xul.dll void mozilla::ipc::MessageChannel::DispatchMessage ipc/glue/MessageChannel.cpp:2078
5 xul.dll nsresult mozilla::ipc::MessageChannel::MessageTask::Run ipc/glue/MessageChannel.cpp:1968
6 xul.dll nsresult mozilla::SchedulerGroup::Runnable::Run xpcom/threads/SchedulerGroup.cpp:295
7 xul.dll nsThread::ProcessNextEvent xpcom/threads/nsThread.cpp:1179
8 xul.dll NS_ProcessNextEvent xpcom/threads/nsThreadUtils.cpp:482
9 xul.dll void mozilla::ipc::MessagePump::Run ipc/glue/MessagePump.cpp:110

crashes with this signature are regressing in volume during the 67 beta cycle.

Comment 1

[Pascal Chevrel:pascalc (PTO until April 26)]

Only one crash on 66, so wontfix for 66, but the 67 volume is noticable even though it went down in the last beta 9.

Comment 2

[Pascal Chevrel:pascalc (PTO until April 26)]

Hsinyi, could we get somebody assigned to this bug? There is an increase in 67 beta and I would like to make sure it doesn't come as a top crash when we reach release. Thanks.

Comment 3

[Peter Van der Beken [:peterv]]

Bug 1541038 comment 8 shows a similar stack, a patch was landed that should have fixed this (see bug 1541038 comment 10 and bug 1541038 comment 17).

Comment 4

[Henri Sivonen (:hsivonen)]

farre, was this already fixed?

Comment 5

[Andreas Farre [:farre]]

The stack looks like what I intended to fix with https://phabricator.services.mozilla.com/D27317. Let's keep track of the crash stats, if they disappear we'll have clear indications of it. Keeping the ni on me to do this.

Comment 6

[Neha Kochar [:neha]]

Andreas, https://phabricator.services.mozilla.com/D27317 was checked into 68. Please use this bug to uplift this patch for 67 beta.

Comment 7

[Neha Kochar [:neha]]

Andreas is on PTO and will be back on 23 Apr.

Comment 8

[Pascal Chevrel:pascalc (PTO until April 26)]

Nika, could you request an uplift of the patch as the reviewer? Andras being on PTO, we may miss the uplift window to beta. Thanks

Comment 9

[Kyle Machulis [:qdot] [:kmachulis] (INACTIVE)]

I'll take this, since both farre and nika are out.

Comment 10

[Kyle Machulis [:qdot] [:kmachulis] (INACTIVE)]

Uplift request at https://bugzilla.mozilla.org/show_bug.cgi?id=1541038#c22 (not quite sure how we're tracking this across 2 bugs)

Comment 11

[Kyle Machulis [:qdot] [:kmachulis] (INACTIVE)]

ni?'ing :pascalc just to make sure request is seen, since this needs to be in soon.

Comment 12

[Pascal Chevrel:pascalc (PTO until April 26)]

https://bugzilla.mozilla.org/show_bug.cgi?id=1541038#c22 will be in today's beta.

Comment 13

[Neha Kochar [:neha]]

https://bugzilla.mozilla.org/show_bug.cgi?id=1541038#c24 has uplifted the fix in beta.