Closed
Bug 1369677
Opened 8 years ago
Closed 7 years ago
Handle the SHMEM map-failed with WR
Categories
(Core :: Graphics: WebRender, defect)
Core
Graphics: WebRender
Tracking
()
RESOLVED
FIXED
mozilla57
Tracking | Status | |
---|---|---|
firefox57 | --- | fixed |
People
(Reporter: jerry, Assigned: jerry)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
3.52 KB,
patch
|
nical
:
review+
|
Details | Diff | Splinter Review |
Some canvas tests create a large shmem buffer and the parent side might not be able to map that large buffer.
The log will be:
###!!! [Parent][DispatchAsyncMessage] Error: SHMEM_CREATED_MESSAGE Payload error: message could not be deserialized
[GFX1-]: Failed to create a valid ShmemTextureHost
There is no problem for gecko with this situation. The gecko just skips the rendering call for that bad buffer. But with WR, we already put the display items at client side. We can't cancel the display item rendering at parent side. So, we will hit some crash when we try to map that bad-shmem during rendering.
I plan to skip the mapping call for the bad-shmem, and bypass the nullptr to WR. Then, provide a whole black buffer(rgba(0,0,0,0)) or a 0 gl handle in the wr-binding. We will see a black area for that bad-shmem display item.
Updated•7 years ago
|
Blocks: stage-wr-nightly
Assignee | ||
Comment 1•7 years ago
|
||
MozReview-Commit-ID: Ghez1SJ6ghf
Attachment #8903095 -
Flags: review?(nical.bugzilla)
Updated•7 years ago
|
Attachment #8903095 -
Flags: review?(nical.bugzilla) → review+
Assignee | ||
Updated•7 years ago
|
Keywords: checkin-needed
Assignee | ||
Comment 2•7 years ago
|
||
Please land the Attachment #8903095 [details] [diff] to m-c.
Pushed by ryanvm@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/3442318a1024
Handle RenderTexture lock() failed. r=nical
Keywords: checkin-needed
Comment 4•7 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
status-firefox57:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla57
You need to log in
before you can comment on or make changes to this bug.
Description
•