Closed
Bug 1411685
Opened 7 years ago
Closed 6 years ago
Crash in <T>::operator() | mozilla::detail::RunnableFunction<T>::Run
Categories
(Core :: Disability Access APIs, defect, P2)
Tracking
()
RESOLVED
FIXED
mozilla61
People
(Reporter: davidb, Assigned: Jamie)
References
Details
(Keywords: crash, regression, Whiteboard: [aes+])
Crash Data
Attachments
(1 file)
https://crash-stats.mozilla.com/signature/?product=Firefox&signature=%3CT%3E%3A%3Aoperator%28%29%20%7C%20mozilla%3A%3Adetail%3A%3ARunnableFunction%3CT%3E%3A%3ARun Content crash. This is a follow on to bug 1383501.
Comment 1•7 years ago
|
||
This is a top crash on Windows, though it looks like it doesn't come from many installations.
Reporter | ||
Comment 2•7 years ago
|
||
> This is a top crash on Windows, though it looks like it doesn't come from > many installations. Yeah, I'm not sure what to make of this. I think there are a few kinds of stacks. Some that I noticed: Ones that crash at null and are related to cookies, like: https://crash-stats.mozilla.com/report/index/793571ad-8b7f-4ed8-841d-1e34f0171027 And ones where accessibility is active, and related to COM IPC like: https://crash-stats.mozilla.com/report/index/361b1776-f8f4-4743-bf08-c06090171027 I'm pretty sure it is the latter that is spiking.(In reply to Andrew McCreight [:mccr8] from comment #1)
Comment 3•6 years ago
|
||
There are 542 crashes for 373 installs in 58.0b4 and 191 crashes for 97 installs in 59.0a1.
status-firefox59:
--- → affected
Comment 4•6 years ago
|
||
[Tracking Requested - why for this release]: the signature is currently the top browser crash on 58.0b4 / 11% of all browser crashes. a small share of the reports come with MOZ_RELEASE_ASSERT(marshalResult != ((HRESULT)0x80070057L)) from bug 1383501
tracking-firefox58:
--- → ?
Keywords: regression
Comment 6•6 years ago
|
||
This should not be tracked -- those assertions are on Nightly and Dev-Ed only.
Comment 7•6 years ago
|
||
nearly 80% of crashes on 58.0b4 are from users on the beta channel though, the rest of them are devedition users.
Comment 8•6 years ago
|
||
(In reply to [:philipp] from comment #7) > nearly 80% of crashes on 58.0b4 are from users on the beta channel though, > the rest of them are devedition users. The problem is that this signature is pretty generic. The beta 4 crashes are the same signature but a different cause that has nothing to do with a11y code.
Comment 9•6 years ago
|
||
For example: https://crash-stats.mozilla.com/report/index/17503ff3-417f-47eb-8225-a65540171121 That's in nsCookieService, which is Necko
Reporter | ||
Comment 11•6 years ago
|
||
Marcia sounds like we may need to track the spin off bug (comment 10).
Flags: needinfo?(dbolter)
Updated•6 years ago
|
Flags: needinfo?(dbolter)
Reporter | ||
Comment 12•6 years ago
|
||
Aaron, we should probably stop the crashing on beta at this point(?)
Flags: needinfo?(dbolter) → needinfo?(aklotz)
Comment 14•6 years ago
|
||
Given https://hg.mozilla.org/releases/mozilla-beta/rev/64ea10c053272a0f0e208dfada147ab9d4134379, I think we shouldn't track this anymore. If someone disagrees, please set the flags back -- thanks!
Assignee | ||
Comment 16•6 years ago
|
||
We decided in bug 1431404 comment 15 to stop these diagnostic crashes even on nightly, since they aren't getting us any closer to fixing the underlying problem and cause pain for users who may not even care. Unfortunately, I missed this one in the patch for bug 1431404.
Assignee: nobody → jteh
Comment hidden (mozreview-request) |
Comment 18•6 years ago
|
||
mozreview-review |
Comment on attachment 8961903 [details] Bug 1411685: Remove another accessibility related diagnostic crash on Nightly. https://reviewboard.mozilla.org/r/230726/#review239868
Attachment #8961903 -
Flags: review?(aklotz) → review+
Comment 19•6 years ago
|
||
Pushed by aklotz@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/9cc46ceede59 Remove another accessibility related diagnostic crash on Nightly. r=aklotz
Comment 20•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/9cc46ceede59
Status: NEW → RESOLVED
Closed: 6 years ago
status-firefox61:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla61
Comment 21•6 years ago
|
||
Please nominate this for Beta approval when you get a chance.
Assignee | ||
Comment 22•6 years ago
|
||
(In reply to Ryan VanderMeulen [:RyanVM] from comment #21) > Please nominate this for Beta approval when you get a chance. This does not affect beta, since these were diagnostic asserts that only ran in nightly builds.
Flags: needinfo?(jteh)
Updated•6 years ago
|
You need to log in
before you can comment on or make changes to this bug.
Description
•