Closed Bug 1030843 Opened 7 years ago Closed 7 years ago

crash in nsNativeThemeCocoa::GetMinimumWidgetSize(nsRenderingContext*, nsIFrame*, unsigned char, nsIntSize*, bool*)


(Core :: Widget: Cocoa, defect)

33 Branch
Not set





(Reporter: rnewman, Assigned: jwatt)



(Keywords: crash, reproducible)

Crash Data


(2 files)

This bug was filed from the Socorro interface and is 
report bp-e05bb53e-5488-4b1b-8e17-86c8f2140626.

Nightly, retina MBP. Crash when printing a page to PDF.

Looks just like Bug 877085.
Component: Layout → Widget: Cocoa
I haven't been able to reproduce this yet, but the first report appears to show a build of 6/21/2014. This might be a recent regression and unrelated to bug 877085.
Ditto :-)

These appear to have started very recently (the earliest report I can find is for the 2014-06-21 m-c nightly).

I can't reproduce these.  Since it seems that you can, Richard, can you find the regression range (in m-c nightlies)?
This only happens for my personal profile. Even copying my prefs.js into another profile (which should mean the same gfx and print settings) doesn't allow for repro.

I'll try to get a regression range the hard way.
I was able to reproduce with an older profile. Let me see what I can find out. A regression range would still be much appreciated.
2014-06-18 Nightly doesn't crash.
Fits with the next interval in my manual regression search -- 20140623030201 crashes. Great!
Attached patch patchSplinter Review
I'm not sure that it really makes sense to be asking for a minimum size with no frame, but I guess it could do. It seems the nsRenderingContext argument isn't actually used other than to get unit conversion ratios, so we might as well change it to an nsPresContext argument instead. That happens to allow us to stop needlessly creating nsRenderingContexts in a couple of locations as an added bonus.
Assignee: nobody → jwatt
Attachment #8446698 - Flags: review?(roc)
I should also update NS_ITHEME_IID and NS_THEMERENDERER_CID of course. Done locally.
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla33
You need to log in before you can comment on or make changes to this bug.