Allow pictures to be rendered in the opaque pass
Categories
(Core :: Graphics: WebRender, enhancement, P3)
Tracking
()
Tracking | Status | |
---|---|---|
firefox83 | --- | fixed |
People
(Reporter: nical, Assigned: nical)
References
(Blocks 1 open bug)
Details
Attachments
(2 files)
At the moment picture primitives are rendered in the alpha pass even if the content in the intermediate target is entirely opaque.
During scene building, we can build a conservative estimate of an opaque rect for each picture surface and see if it covers the entire surface rect.
In addition, if the opaque area only covers the picture's surface partially, we could segment the brush around the opaque rect during batching.
Assignee | ||
Comment 1•4 years ago
|
||
Updated•4 years ago
|
Assignee | ||
Comment 2•4 years ago
|
||
This patch tracks tracks a conservative estimated opaque region represented as one rect per picture surface (in picture space). This is computed by first generating an opaque rect per cluster in the cluster's local space and accumulating cluster opaque rects for each picture during the prepare pass.
This information is used during batching to decide whether the content of a picture is entirely opaque in which case it can go in the opaque pass.
A potential followup to this is to segment the opaque and semi-transparent regions via a nine-patch for partially opaque pictures.
Depends on D90580
Comment 4•4 years ago
|
||
Backed out for wrench bustage on screen-space-blit.yaml.
Backout link: https://hg.mozilla.org/integration/autoland/rev/e22e7032c192f66b7f516126b5ba795aeab82337
Push with failures: https://treeherder.mozilla.org/#/jobs?repo=autoland&resultStatus=testfailed%2Cbusted%2Cexception&revision=0481d7e344302703cb635d77c23fc9ff037a1621&searchStr=wrench
Failure log: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=316230853&repo=autoland&lineNumber=13278
Comment 6•4 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/f79f5a702d3d
https://hg.mozilla.org/mozilla-central/rev/afa8408d3933
Assignee | ||
Updated•4 years ago
|
Description
•