Crash in [@ nsWrapperCache::GetWrapper] under mozilla::dom::Navigator_Binding::get_gpu
Categories
(Core :: Graphics: WebGPU, defect, P3)
Tracking
()
Tracking | Status | |
---|---|---|
firefox-esr68 | --- | unaffected |
firefox75 | --- | unaffected |
firefox76 | + | fixed |
firefox77 | + | fixed |
People
(Reporter: emilio, Assigned: kvark)
References
Details
(Keywords: crash, regression)
Crash Data
This bug is for crash report bp-05ad85cf-38f2-4d76-9b03-852b90200131.
Top 10 frames of crashing thread:
0 libxul.so nsWrapperCache::GetWrapper const dom/base/nsWrapperCacheInlines.h:27
1 libxul.so mozilla::dom::Navigator_Binding::get_gpu dom/bindings/NavigatorBinding.cpp:2299
2 libxul.so bool mozilla::dom::binding_detail::GenericGetter<mozilla::dom::binding_detail::NormalThisPolicy, mozilla::dom::binding_detail::ThrowExceptions> dom/bindings/BindingUtils.cpp:3033
3 libxul.so js::InternalCallOrConstruct js/src/vm/Interpreter.cpp:542
4 libxul.so js::CallGetter js/src/vm/Interpreter.cpp:746
5 libxul.so js::NativeGetProperty js/src/vm/NativeObject.cpp:2631
6 libxul.so js::GetProperty js/src/vm/Interpreter.cpp:4428
7 libxul.so Interpret js/src/vm/Interpreter.cpp:2700
8 libxul.so js::RunScript js/src/vm/Interpreter.cpp:422
9 libxul.so js::ExecuteKernel js/src/vm/Interpreter.cpp:798
I toggled the pref on, went to https://hello-webgpu-compute.glitch.me/, and clicked on the "Chromium/Firefox/Servo Demo".
Reporter | ||
Comment 1•6 years ago
|
||
(Note that this is only reproducible the first time, reloading the page worked)
Assignee | ||
Comment 2•6 years ago
|
||
Thank you for filing! Will look into it.
Assignee | ||
Comment 3•6 years ago
|
||
It's fairly simple: even though the pref is switched to ON, the gfxConfig
isn't updated automatically, and it thinks WebGPU is disabled. When the user tries to get a WebGPU instance, we are returning NULL. This isn't expected by the JS glue code.
Instead, we should probably allow the user to get the instance but not expose any adapters.
Comment 4•6 years ago
|
||
The priority flag is not set for this bug.
:kvark, could you have a look please?
For more information, please visit auto_nag documentation.
Assignee | ||
Updated•6 years ago
|
Comment 5•5 years ago
|
||
The priority flag is not set for this bug.
:kvark, could you have a look please?
For more information, please visit auto_nag documentation.
Assignee | ||
Updated•5 years ago
|
Comment 6•5 years ago
|
||
[Tracking Requested - why for this release]:
these crash reports are spiking up in volume cross-platform after 76.0b5 got released to the beta audience - they are accounting for 2% of content crashes.
could this be related to the uplift in bug 1616741?
Assignee | ||
Comment 7•5 years ago
|
||
No, bug 1616741 is the opposite, it tries to make sure that this never happens in Beta.
I think the spike is caused by more people trying out WebGPU examples in general. They still have to go and flip the pref before they crash. And when they get the version with bug 1616741, the pref will do nothing in Release/Beta, so we'll be in a better place.
Comment 8•5 years ago
•
|
||
We're still getting reports from 76.0b6 with the same-looking stack.
https://crash-stats.mozilla.org/report/index/7fefc164-ef3d-47e8-a7b1-519920200420
Also, the patch from bug 1616741 landed in time to ship with 76.0b5, so how would comment 7 explain the spike in that release?
Assignee | ||
Comment 9•5 years ago
|
||
You are right. I'm requesting the uplift to bug 1629955 fix, which should address this properly.
Comment 10•5 years ago
|
||
No crashes with 76.0b7 so far - looks like that fix worked.
Description
•