Webp encoder now defaults to lossy, if nothing else is specified

[brianpopow]

Prerequisites

• I have written a descriptive pull-request title
• I have verified that there are no overlapping pull-requests open
• I have verified that I am following the existing coding patterns and practice as demonstrated in the repository. These follow strict Stylecop rules 👮.
• I have provided test coverage for my change (where applicable)

Description

To match libwebp behavior, the Webp encoder now defaults to lossy, if nothing else is specified.

[JimBobSquarePants]

LGTM 👍

[JimBobSquarePants]

@brianpopow I'm seeing a lack of alpha channel support when using lossy encoding. Is that expected?

EDIT. Just saw this.
#1802

On a scale of 1 to only you can do it how hard would it be to add support?

[brianpopow]

@brianpopow I'm seeing a lack of alpha channel support when using lossy encoding. Is that expected?

EDIT. Just saw this. #1802

On a scale of 1 to only you can do it how hard would it be to add support?

I dont think its hard to do. The Alpha data just needs to be encoded with the already existing VP8LEncoder or stored as is in an additional ALPH chunk.

[PhyxionNL]

Until alpha support is added, I'm not sure this is a good default as you'd end up with lots of broken images.

[brianpopow]

I am working on alpha encoding. Should not take too long to implement.

[JimBobSquarePants]

@brianpopow HERO!