Closed
Bug 1560977
Opened 5 years ago
Closed 5 years ago
Fission-enabled profiles should be clearly marked on crash-stats
Categories
(Core :: DOM: Content Processes, enhancement, P2)
Core
DOM: Content Processes
Tracking
()
Tracking | Status | |
---|---|---|
firefox69 | --- | verified |
People
(Reporter: nika, Assigned: mccr8)
References
Details
Attachments
(2 files)
(deleted),
text/x-phabricator-request
|
Details | |
(deleted),
text/plain
|
chutten
:
data-review+
|
Details |
Fission is enabled by setting the useRemoteSubframes
bit on a nsILoadContext. The easiest way to do this currently is by setting the fission.autostart
pref.
We should clearly mark crashes in profiles which have set this bit as fission-enabled.
Reporter | ||
Updated•5 years ago
|
Fission Milestone: --- → M4
Priority: -- → P2
Assignee | ||
Updated•5 years ago
|
Assignee: nobody → continuation
Updated•5 years ago
|
Status: NEW → ASSIGNED
Assignee | ||
Comment 1•5 years ago
|
||
Assignee | ||
Comment 2•5 years ago
|
||
I've also got a data collection review thing that I'll upload after I get some feedback from Nika.
Assignee | ||
Comment 3•5 years ago
|
||
Attachment #9075395 -
Flags: data-review?(chutten)
Comment 4•5 years ago
|
||
Comment on attachment 9075395 [details]
datareview.txt
DATA COLLECTION REVIEW RESPONSE:
Is there or will there be documentation that describes the schema for the ultimate data set available publicly, complete and accurate?
Yes. This collection is a Crash Annotation so is documented in its definitions file [CrashAnnotations.yaml](https://hg.mozilla.org/mozilla-central/file/tip/toolkit/crashreporter/CrashAnnotations.yaml).
Is there a control mechanism that allows the user to turn the data collection on and off?
Yes. This collection is part of crash reports so can be controlled through Firefox's Preferences for crash reports.
If the request is for permanent data collection, is there someone who will monitor the data over time?
Yes, Nika Layzell is responsible.
Using the category system of data types on the Mozilla wiki, what collection type of data do the requested measurements fall under?
Category 2, Interaction. (We could argue that it's Category 1 because the value of the pref at the time of a crash is more technical than interaction, but let's leave it at Category 2 because that's what the request asked for)
Is the data collection request for default-on or default-off?
Default off. The user must opt-in to sending crash reports.
Does the instrumentation include the addition of any new identifiers?
No.
Is the data collection covered by the existing Firefox privacy notice?
Yes.
Does there need to be a check-in in the future to determine whether to renew the data?
No. This collection is permanent.
---
Result: datareview+
Attachment #9075395 -
Flags: data-review?(chutten) → data-review+
Pushed by amccreight@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/cc9b37d2e954
Annotate crash reports if Fission is enabled. r=nika
Comment 6•5 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
status-firefox69:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla69
Updated•5 years ago
|
Flags: qe-verify+
QA Contact: vlad.lucaci
Comment 8•5 years ago
|
||
Comparing two different crashes encountered with fission enabled:
- Encountered before this fix -> bp-d3edca6f-7830-4980-ae4c-a1b320190619 (The DOMFissionEnabled is not displayed)
- Encountered after this fix -> bp-40090b34-f6c0-442f-97d0-da1d20190820 (The DOMFissionEnabled is successfully displayed and has the value of 1)
Marking this as verified fixed.
You need to log in
before you can comment on or make changes to this bug.
Description
•