Include more information in the panic for [@ swgl::swgl_fns::{{impl}}::shader_source]
Categories
(Core :: Graphics: WebRender, task, P3)
Tracking
()
Tracking | Status | |
---|---|---|
firefox86 | --- | fixed |
People
(Reporter: aosmond, Assigned: aosmond)
References
Details
Attachments
(2 files)
(deleted),
text/x-phabricator-request
|
Details | |
(deleted),
text/plain
|
chutten
:
data-review+
|
Details |
We don't really have an idea of where we went wrong in bug 1680452. Is the shader source empty? Or missing the header?
Comment 1•4 years ago
|
||
For bug 1609191 we wanted to annotate crash reports with which shader was being compiled during the crash. Not sure if the solution to this could help with that too
Assignee | ||
Comment 2•4 years ago
|
||
Yep looking at ways to do that. I also want to make sure they aren't doing something like setting WR_RESOURCE_PATH and overriding the default shaders.
Assignee | ||
Comment 3•4 years ago
|
||
This patch adds infrastructure for crash reporter annotations to
WebRender. This is used to expose the new annotation,
GraphicsCompileShader, to indicate which shader we are in the process of
compiling. We often see crash reports when compiling shaders, and it
would be useful to know which one it is compiling.
This also adds another annotation, IsWebRenderResourcePathOverridden,
which is useful to know if someone overrode the shader resource path for
testing purposes. We can likely ignore any crash reports that have this
annotation set.
Assignee | ||
Comment 4•4 years ago
|
||
Comment 5•4 years ago
|
||
Comment on attachment 9193285 [details]
Data review, v1
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.
Is there a control mechanism that allows the user to turn the data collection on and off?
Yes. This collection is Telemetry so can be controlled through Firefox's Preferences.
If the request is for permanent data collection, is there someone who will monitor the data over time?
Yes, :aosmond 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 1, Technical.
Is the data collection request for default-on or default-off?
Default on for all channels.
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+
Comment 7•4 years ago
|
||
bugherder |
Description
•