Closed
Bug 1307809
Opened 8 years ago
Closed 7 years ago
Crash in mozilla::layers::BufferDescriptor::AssertSanity
Categories
(Core :: Graphics: Layers, defect)
Tracking
()
RESOLVED
FIXED
mozilla53
People
(Reporter: anton.kochkov, Assigned: sotaro)
Details
(Keywords: crash, crashreportid, Whiteboard: [gfx-noted])
Crash Data
Attachments
(1 file)
724 bytes,
patch
|
nical
:
review+
jcristau
:
approval-mozilla-beta+
|
Details | Diff | Splinter Review |
This bug was filed from the Socorro interface and is report bp-1b3b61fb-3f88-46ae-aef6-5214f2161005. ============================================================= Crash after switching to tab with Twitter.
Updated•8 years ago
|
status-firefox52:
--- → affected
Component: Untriaged → Graphics: Layers
Product: Firefox → Core
Version: Trunk → 52 Branch
Updated•8 years ago
|
Keywords: crashreportid
Sotaro, can you think of a case where this assert can happen with the basic compositor?
Flags: needinfo?(sotaro.ikeda.g)
Whiteboard: [gfx-noted]
Assignee | ||
Updated•7 years ago
|
Assignee: nobody → sotaro.ikeda.g
Flags: needinfo?(sotaro.ikeda.g)
Assignee | ||
Comment 2•7 years ago
|
||
There might be a case that mFirstSource is not owned by the compositing TextureHost.
Assignee | ||
Comment 3•7 years ago
|
||
Assignee | ||
Comment 4•7 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=9b8fba227079a7584ccbb99883cf3ea18762ad64
Assignee | ||
Updated•7 years ago
|
Attachment #8823924 -
Flags: review?(nical.bugzilla)
Updated•7 years ago
|
Attachment #8823924 -
Flags: review?(nical.bugzilla) → review+
Pushed by sikeda@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/928cb4767284 Add IsOwnedBy() check r=nical
Comment 6•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/928cb4767284
Status: UNCONFIRMED → RESOLVED
Closed: 7 years ago
status-firefox53:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla53
Comment 7•7 years ago
|
||
Please nominate this for Aurora approval when you get a chance.
status-firefox51:
--- → wontfix
Flags: needinfo?(sotaro.ikeda.g)
Assignee | ||
Comment 8•7 years ago
|
||
Comment on attachment 8823924 [details] [diff] [review] patch - Add IsOwnedBy() check Approval Request Comment [Feature/Bug causing the regression]: Bug 1254011 [User impact if declined]:Firefox might crash during video playback without hw acceleration. [Is this code covered by automated tests?]: Yes [Has the fix been verified in Nightly?]:There's no real STR. [Needs manual test from QE? If yes, steps to reproduce]: No [List of other uplifts needed for the feature/fix]: No [Is the change risky?]: No. Low risk. [Why is the change risky/not risky?]: It added simple status check. [String changes made/needed]: None
Flags: needinfo?(sotaro.ikeda.g)
Attachment #8823924 -
Flags: approval-mozilla-aurora?
Comment 9•7 years ago
|
||
Comment on attachment 8823924 [details] [diff] [review] patch - Add IsOwnedBy() check crash fix, beta52+
Attachment #8823924 -
Flags: approval-mozilla-aurora? → approval-mozilla-beta+
Comment 10•7 years ago
|
||
bugherder uplift |
https://hg.mozilla.org/releases/mozilla-beta/rev/3ae45b6e6eb1
You need to log in
before you can comment on or make changes to this bug.
Description
•