add a novideo content-type

[totaam]

Issue migrated from trac ticket # 2100

component: encodings | priority: major | resolution: fixed

2019-01-07 22:52:41: nathan_lstc created the issue

---

Some of my users really dislike video frames in xterm. With this patch, users can (1) set a novideo content type and (2) set a guesser rule from an environment variable (from a systemd entry I have):

Environment=XPRA_CONTENT_TYPE_DEFS=class-instance:xterm=novideo

[totaam]

2019-01-07 22:53:03: nathan_lstc uploaded file novideo-content-type.patch (4.8 KiB)

[totaam]

2019-01-08 04:00:23: antoine changed owner from antoine to nathan_lstc

[totaam]

2019-01-08 04:00:23: antoine commented

---

I like the idea.

We could just add a new flag like TEXT_NOVIDEO. This would apply to all applications that are meant to be text.

This does not use lossless:

            return nonvideo(info="lossless content-type")

It will use a non-video encoding, usually at a high quality setting, but not lossless unless you change the quality value with this call.
And if you do, then you might as well call the new content-type "lossless"?

[totaam]

2019-01-08 18:35:30: nathan_lstc commented

---

I had originally gone with lossless and then changed my mind after trying nonvideo and finding myself happy with that. I forgot to change the text is all.

I am okay with either choice. I defer to you.

[totaam]

2019-01-08 18:36:29: nathan_lstc changed type from defect to enhancement

[totaam]

2019-01-08 18:36:29: nathan_lstc changed component from android to encodings

[totaam]

2019-01-09 15:42:07: antoine commented

---

The part adding an env var for specifying content type definitions was merged in r21302 (see #1950#comment:8).
r21303 disables video encoders for windows with the "text" content-type hint.

@nathan_lstc: does this work OK for you? if so, please close.

[totaam]

2019-01-09 17:53:35: nathan_lstc changed status from new to closed

[totaam]

2019-01-09 17:53:35: nathan_lstc set resolution to fixed

[totaam]

2019-01-09 17:53:35: nathan_lstc commented

---

This is great! Thank you.