Closed Bug 1233182 Opened 4 years ago Closed 4 years ago

crash in mozilla::layers::ContentClientDoubleBuffered::FinalizeFrame - 44 release only


(Core :: Graphics, defect)

43 Branch
Not set



Tracking Status
firefox44 + fixed
b2g-v2.5 --- fixed


(Reporter: milan, Assigned: milan)




(2 files)

Bug 1200021 landed on 45 and 43, but 44 wasn't done.  We were waiting to see if the results were conclusive.  Along the way, bug 1200021 exposed a problem fixed in bug 1230740, currently only in 45.

The purpose of this bug is to track us doing the right thing in 44.  That should probably be patches from both bug 1200021 and bug 1230740.
Assignee: nobody → milan
See Also: → 1200021, 1230740
Milan, would you be able to help nominate the relevant patches for uplift to Beta44?
Flags: needinfo?(milan)
Will do - bug 1230740 is on inbound, once it's on central for a few days, we can see what exactly we need.
OK, looks like, for 44, we will want to uplift bug 1200021, combined with bug 1230740.  Because it's somewhat confusing, I'll put up a patch on this bug that is specific to 44, rather than try to uplift things form those two bugs.
Flags: needinfo?(milan)
Approval Request Comment
See discussion in bug 1200021, bug 1230740.
Attachment #8703757 - Flags: review+
Attachment #8703757 - Flags: approval-mozilla-beta?
Comment on attachment 8703757 [details] [diff] [review]
Beta only patch for bug 1200021 and 1230740.  Carry r=bas

Both the fixes have been in Nightly and Aurora for a few weeks and the crash data looks promising, beta44+
Attachment #8703757 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
This may be a cause for new crashes in bug 1238257, I will investigate in that bug.
Bad merge to produce the above patch, causing bug 1238257.
Resolution: FIXED → ---
There is a problem with the first patch - this fixes it.
Attachment #8706514 - Flags: review?(botond)
Attachment #8706514 - Flags: approval-mozilla-beta?
Attachment #8706514 - Flags: review?(botond) → review+
Comment on attachment 8706514 [details] [diff] [review]
Part 2 - two lines were reversed in the previous patch .  r=botond

Fixes a top crash, taking it. Beta44+
Attachment #8706514 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
Reverting to fx44 status to "affected" so it shows up on the queries Sheriff's use to look for beta+'d patches that still need to land.
Closed: 4 years ago4 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.