Closed
Bug 1262681
Opened 8 years ago
Closed 8 years ago
crash in mozilla::layers::RotatedBuffer::DrawBufferQuadrant
Categories
(Core :: Graphics: Layers, defect)
Tracking
()
People
(Reporter: alex_mayorga, Assigned: milan)
Details
(Keywords: crash, Whiteboard: [gfx-noted])
Crash Data
Attachments
(1 file, 1 obsolete file)
This bug was filed from the Socorro interface and is report bp-60244d11-07c4-4cfd-b546-115092160407. ============================================================= ¡Hola! 569 crashes (12 startup) in the past month at https://crash-stats.mozilla.com/report/list?product=Firefox&range_unit=days&range_value=28&signature=mozilla%3A%3Alayers%3A%3ARotatedBuffer%3A%3ADrawBufferQuadrant#tab-sigsummary
Reporter | ||
Updated•8 years ago
|
status-firefox45:
--- → affected
status-firefox46:
--- → affected
status-firefox47:
--- → affected
status-firefox48:
--- → affected
Assignee | ||
Comment 1•8 years ago
|
||
All of the ones I've seen call DrawTargetCairo::Snapshot with invalid DrawTarget, returning null , having RotatedBuffer::DrawBufferQuadrant (which got the snapshot through RotatedContentBuffer::GetSourceSurface call) use that snapshot without checking. This perhaps starts with Factory::CreateDrawTargetForData getting called with DIRECT2D1_1 as the BackendType, which is not something we do. Could be an OOM of some type, or a device reset.
Whiteboard: [gfx-noted]
Assignee | ||
Comment 2•8 years ago
|
||
Attachment #8739130 -
Flags: feedback?(bas)
Comment 3•8 years ago
|
||
Comment on attachment 8739130 [details] [diff] [review] Not necessariy a patch that deals with this issue, but we need to worry about the validity, not just the existence, of a draw target. Review of attachment 8739130 [details] [diff] [review]: ----------------------------------------------------------------- I'm not sure how much this adds, but it can't hurt.
Attachment #8739130 -
Flags: feedback?(bas) → feedback+
Assignee | ||
Comment 4•8 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=635c174c197a
Keywords: leave-open
Assignee | ||
Comment 5•8 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=f7decc6bccab
Assignee | ||
Comment 6•8 years ago
|
||
Attachment #8739130 -
Attachment is obsolete: true
Attachment #8741017 -
Flags: review+
Assignee | ||
Updated•8 years ago
|
Keywords: checkin-needed
Comment 8•8 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/7ed46b45eef8
Assignee | ||
Comment 9•8 years ago
|
||
(In reply to Carsten Book [:Tomcat] from comment #8) > https://hg.mozilla.org/mozilla-central/rev/7ed46b45eef8 No crashes of this type since just before this patch landed (20160413030239 is the last one so far.) I did find a (single, linux) crash that has error reports added in the above patch: https://crash-stats.mozilla.com/report/index/b6e49c8d-677d-4b22-99e0-8074f2160417. I doubt this is new, it just would have crashed elsewhere before. Lets wait for more days and data, and see if we get any crashes with the extra info once we go to dev edition with 48.
Flags: needinfo?(milan)
Assignee | ||
Comment 10•8 years ago
|
||
I don't see any of these crashes in 48 or 49. Not enough volume to uplift to 47 beta, looks like we're getting two or so crashes per day, so maybe just call this fixed and let it ride 48.
Assignee: nobody → milan
Status: NEW → RESOLVED
Closed: 8 years ago
Flags: needinfo?(milan)
Keywords: leave-open
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•