Jean-Yves Avenard [:jya] Assignee
	Description • 6 years ago

Similar to bug 1469711 but with Web Render

Add support for 10/12 bits YUV image

	Jean-Yves Avenard [:jya] Assignee
	Comment 1 • 6 years ago

Going to have a try at this over the next week or so

	Jean-Yves Avenard [:jya] Assignee
	Comment 2 • 6 years ago

Prevent future ambiguous conversion once a bit depth int is added.
Also more self-explanatory

	Jean-Yves Avenard [:jya] Assignee
	Comment 3 • 6 years ago

Depends on D6662

	Nicolas Silva [:nical]
	Comment 4 • 6 years ago

Comment on attachment 9011446 [details]
Bug 1493198 - P1. Make ImageDescriptor::is_opaque an enum and rename accordingly.

Nicolas Silva [:nical] has approved the revision.

	Jean-Yves Avenard [:jya] Assignee
	Comment 5 • 6 years ago

Depends on D6663

	Jean-Yves Avenard [:jya] Assignee
	Comment 6 • 6 years ago

To be pushed via PR
https://github.com/servo/webrender/pull/3113

	Jean-Yves Avenard [:jya] Assignee
	Comment 7 • 6 years ago

Both D3D11 compositor and WebRender now supports 10/12 bits images.

Depends on D6688

	Matt Woodrow (:mattwoodrow)
	Comment 8 • 6 years ago

Comment on attachment 9011447 [details]
Bug 1493198 - P2. Use enum for describing color depth.

Matt Woodrow (:mattwoodrow) has approved the revision.

	Matt Woodrow (:mattwoodrow)
	Comment 9 • 6 years ago

Comment on attachment 9011545 [details]
Bug 1493198 - P4. Add support for 10/12 bits YUV image to WebRender (bindings)

Matt Woodrow (:mattwoodrow) has approved the revision.

	Jean-Yves Avenard [:jya] Assignee
	Comment 10 • 6 years ago

Depends on D6663

	Nicolas Silva [:nical]
	Comment 11 • 6 years ago

Comment on attachment 9011749 [details]
Bug 1493198 - P3. Fix typo and use move semantics.

Nicolas Silva [:nical] has approved the revision.

	Darkspirit
	Comment 12 • 6 years ago

Debian Testing, KDE, Xorg, Nvidia GTX 1060 (driver 390.87)

Colors seem to be a bit wrong.
(But with Basic compositing I get white backgrounds and everything looks the same.)

https://treeherder.mozilla.org/#/jobs?repo=try&revision=d97bf44347933bd4ad89aa080d83a005b74a9985

mozregression --repo try --launch d97bf44347933bd4ad89aa080d83a005b74a9985 --pref gfx.webrender.all:true -a https://www.avenard.org/htmltests/tests/webm-hdr.html

	Darkspirit
	Comment 13 • 6 years ago

It looks the same on Ubuntu 18.04 LTS, GTX 1060 (390.48), 2560x1440.

	Bryce Seager van Dyk [:bryce] (he/him) - Not reading bugmail
	Comment 14 • 6 years ago

Comment on attachment 9011571 [details]
Bug 1493198 - P5. Don't restrict high bit depth images on Windows

Bryce Seager van Dyk (:bryce) has approved the revision.

	Pulsebot
	Comment 15 • 6 years ago

Pushed by jyavenard@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/fa105d110671
P1. Make ImageDescriptor::is_opaque an enum and rename accordingly. r=nical
https://hg.mozilla.org/integration/autoland/rev/afdc1b5bb3d9
P2. Use enum for describing color depth. r=mattwoodrow
https://hg.mozilla.org/integration/autoland/rev/b571f960e636
P3. Fix typo and use move semantics. r=nical

	Arthur Iakab [arthur_iakab]
	Comment 16 • 6 years ago
bugherder

https://hg.mozilla.org/mozilla-central/rev/fa105d110671
https://hg.mozilla.org/mozilla-central/rev/afdc1b5bb3d9
https://hg.mozilla.org/mozilla-central/rev/b571f960e636

	Pulsebot
	Comment 17 • 6 years ago

Pushed by jyavenard@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/ca2d75deffe6
P4. Add support for 10/12 bits YUV image to WebRender (bindings) r=mattwoodrow
https://hg.mozilla.org/integration/autoland/rev/32bb9ea9e273
P5. Don't restrict high bit depth images on Windows r=bryce

	Stefan Hindli [:stefan_hindli]
	Comment 18 • 6 years ago
bugherder

https://hg.mozilla.org/mozilla-central/rev/ca2d75deffe6
https://hg.mozilla.org/mozilla-central/rev/32bb9ea9e273