ref: Drop errored in favor of crashed

[bruno-garcia]

We've discussed in today's TSC that it wont' be possible to introduce errored and have crashed mean that in fact the app was terminated (what most people understand crash means). That's mainly due to backwards compat (old SDKs will send crashed=true meaning erroed, for years). So we're accepting that crashed=true means errored and not necessarily a hard crash.

We'll compensate in the server normalization or worst case in the UI to indicate that this flag means this was the errored thread as opposed to this thread crashed the process

[codecov-io]

Codecov Report

Merging #187 into master will decrease coverage by 0.09%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##             master    #187     +/-   ##
==========================================
- Coverage        58%   57.9%   -0.1%     
+ Complexity      551     543      -8     
==========================================
  Files            72      72             
  Lines          2617    2599     -18     
  Branches        235     228      -7     
==========================================
- Hits           1518    1505     -13     
+ Misses          978     977      -1     
+ Partials        121     117      -4

Impacted Files	Coverage Δ	Complexity Δ
...c/main/java/io/sentry/core/MainEventProcessor.java	70% <ø> (+6.58%)	6 <0> (-1)	⬇️
...ain/java/io/sentry/core/protocol/SentryThread.java	70.37% <ø> (-2.97%)	12 <0> (-2)
.../main/java/io/sentry/core/SentryThreadFactory.java	100% <100%> (ø)	6 <0> (-4)	⬇️
...core/src/main/java/io/sentry/core/SentryEvent.java	66.66% <0%> (-1.02%)	38% <0%> (-1%)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4b34e5f...f80a4e8. Read the comment docs.