1909216 - Crash in [@ mozilla::PresShell::GetMobileViewportManager] while printing

Status: RESOLVED DUPLICATE of bug 1908660

(Core :: Layout, defect)

Description

[Andrew McCreight [:mccr8]]

Crash report: https://crash-stats.mozilla.org/report/index/06f37def-66a1-4902-b1e8-c91850240722

Reason: SIGSEGV / SEGV_MAPERR

Top 10 frames:

0  libxul.so  RefPtr<MobileViewportManager>::RefPtr(RefPtr<MobileViewportManager> const&)  mfbt/RefPtr.h:92
0  libxul.so  mozilla::PresShell::GetMobileViewportManager() const  layout/base/PresShell.cpp:11148
1  libxul.so  nsPresContext::AdjustSizeForViewportUnits()  layout/base/nsPresContext.cpp:3010
1  libxul.so  nsPresContext::SetVisibleArea(nsRect const&)  layout/base/nsPresContext.cpp:2966
2  libxul.so  nsPrintJob::ReflowPrintObject(mozilla::UniquePtr<nsPrintObject, mozilla::Defa...  layout/printing/nsPrintJob.cpp:1331
3  libxul.so  nsPrintJob::ReflowDocList(mozilla::UniquePtr<nsPrintObject, mozilla::DefaultD...  layout/printing/nsPrintJob.cpp:959
4  libxul.so  nsPrintJob::InitPrintDocConstruction(bool)  layout/printing/nsPrintJob.cpp:1000
5  libxul.so  nsPrintJob::DoCommonPrint(bool, nsIPrintSettings*, nsIWebProgressListener*, m...  layout/printing/nsPrintJob.cpp:446
6  libxul.so  nsPrintJob::CommonPrint(bool, nsIPrintSettings*, nsIWebProgressListener*, moz...  layout/printing/nsPrintJob.cpp:335
7  libxul.so  nsDocumentViewer::Print(nsIPrintSettings*, mozilla::layout::RemotePrintJobChi...  layout/base/nsDocumentViewer.cpp:2912

Huge crash spike, all on the 20240719095541 build. Looks printing related. I'm not sure what component this should go in. Very odd that it is only in a single build from a few days ago. Maybe something got backed out for another reason and it happened to fix this? I'm not sure what though. The new commits in that build are here and it looks okay. I guess this is Android so it might also include more commits.

Comment 1

[Andrew McCreight [:mccr8]]

Sean, I noticed you've looked at nsPrintJob recently. Is this a crash that bug 1881096 might have fixed?

Comment 2

[Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)]

Maybe from bug 1788504? Regression bug 1908660 mention Print and got fixed.
This is a recent Nightly crash, bug 1881096 landed on 129.0a1 (now in beta) a month ago.

Comment 3

[Andrew McCreight [:mccr8]]

Ah, oops! Somehow I read 06-20 as 07-20. Thanks for looking.