Closed Bug 1299738 Opened 4 years ago Closed 3 years ago
Surface Incompatible crash in mozilla::gfx::Get Bitmap For Surface
This bug was filed from the Socorro interface and is report bp-7f0c305d-fc17-420f-ae1f-b03732160901. ============================================================= Low volume crash at , SourceSurfaceIncompatible: "Non-Skia SourceSurfaces need to be DataSourceSurfaces". Crashes seem to have started with the 8-27 nightly (with regression range ), and there are consistently a few crashes per nightly.  https://hg.mozilla.org/mozilla-unified/annotate/1a5b53a831e5/gfx/2d/DrawTargetSkia.cpp#l215  https://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=a551f534773cf2d6933f78ce7d82a7a33a99643e&tochange=c724ae8bb7b867b6bc65efa1049c0322457ff4b0
Version: unspecified → 51 Branch
We have stale SourceSurfaceD2D1s lying around temporarily during a device reset till the browser notices to clean them up. These will fail to map/convert to data surfaces. Normally this was not a problem in DrawTargetD2D1 or DrawTargetCairo since we would not crash, but only just bail out or at worst warn. So, we're a little overzealous in always crashing in this case in Skia. So let's just warn here to be consistent. Also clean up some old cruft that is unnecessary in DrawTargetSkia checking SourceSurface types.
Assignee: nobody → lsalzman
Status: NEW → ASSIGNED
Attachment #8787303 - Flags: review?(mchang)
Attachment #8787303 - Flags: review?(mchang) → review+
Pushed by email@example.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/657ba2f2611c warn instead of crash if DrawTargetSkia's GetBitmapForSurface fails. r=mchang
You need to log in before you can comment on or make changes to this bug.