1629477 - Crash in [@ mozilla::dom::ContentParent::GetUsedBrowserProcess]

Status: RESOLVED FIXED

(Core :: DOM: Content Processes, defect, P3)

Description

[Gabriele Svelto [:gsvelto]]

This bug is for crash report bp-6a1aa861-bbe4-494f-935f-08c1d0200412.

Top 10 frames of crashing thread:

0 libxul.so mozilla::dom::ContentParent::GetUsedBrowserProcess dom/ipc/ContentParent.cpp:877
1 libxul.so mozilla::dom::ContentParent::GetNewOrUsedBrowserProcessInternal dom/ipc/ContentParent.cpp:906
2 libxul.so mozilla::dom::ContentParent::GetNewOrUsedBrowserProcess dom/ipc/ContentParent.cpp:993
3 libxul.so mozilla::dom::ContentParent::CreateBrowser dom/ipc/ContentParent.cpp:1236
4 libxul.so nsFrameLoader::TryRemoteBrowserInternal dom/base/nsFrameLoader.cpp:2580
5 libxul.so nsFrameLoader::TryRemoteBrowser dom/base/nsFrameLoader.cpp:2642
6 libxul.so nsFrameLoader::ShowRemoteFrame dom/base/nsFrameLoader.cpp:1032
7 libxul.so nsFrameLoader::Show dom/base/nsFrameLoader.cpp:903
8 libxul.so nsSubDocumentFrame::ShowViewer layout/generic/nsSubDocumentFrame.cpp:192
9 libxul.so AsyncFrameInit::Run layout/generic/nsSubDocumentFrame.cpp:95

We're tripping over this assertion which IIUC was added in bug 1582318.

Comment 1

[Andrew McCreight [:mccr8]]

It looks like bug 1582318 got backed out, but this might still be of interest to Kris.

Comment 2

[Chris Peterson [:cpeterson]]

Assigning to kmag. He thinks he's fixed a related crash.

P3 M6 because the crash volume is low.

Comment 3

[Kris Maglione [:kmag]]

Attachment: Bug 1629477: Remove failing assertion. r=nika

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/241ea498604b
Remove failing assertion. r=nika

Comment 5

[Natalia Csoregi [:nataliaCs]]

https://hg.mozilla.org/mozilla-central/rev/241ea498604b