Add utils to better quantize grayscale values to three bit indices while

Add utils to better quantize grayscale values to three bit indices while
compressing coverage masks.

Signed-off-by: Pavel Krajcevski <pavel@cs.unc.edu>;

BUG=skia:

Committed: https://skia.googlesource.com/skia/+/47eedcc848e7712aea9607fd489abdc70bc8bcfd

[pavelk, 2014-10-22 21:31:26]

pavel@cs.unc.edu changed reviewers:
+ robertphillips@google.com

[robertphillips, 2014-10-23 12:08:57]

robertphillips@google.com changed reviewers:
+ hcm@google.com

[robertphillips, 2014-10-23 12:10:20]

looks good to me (modulo a nit). Does this slow things down at all?

Adding Heather in case you need to sign some contributor thing before submitting
code.

https://codereview.chromium.org/669243003/diff/1/src/utils/SkTextureCompresso...
File src/utils/SkTextureCompressor_Utils.h (right):

https://codereview.chromium.org/669243003/diff/1/src/utils/SkTextureCompresso...
src/utils/SkTextureCompressor_Utils.h:13: #if 1
Do you intend for these to be static? They're named that way.

[hcm, 2014-10-23 13:26:02]

On 2014/10/23 12:10:20, robertphillips wrote:
> looks good to me (modulo a nit). Does this slow things down at all?
> 
> Adding Heather in case you need to sign some contributor thing before
submitting
> code.

Good point, we do need that now.  Pavel, go here and submit an electronic
individual CLA: https://cla.developers.google.com/clas
...and add your name/external email address to the Skia AUTHORS file as part of
this CL so it will pass the commit checks.
> 
>
https://codereview.chromium.org/669243003/diff/1/src/utils/SkTextureCompresso...
> File src/utils/SkTextureCompressor_Utils.h (right):
> 
>
https://codereview.chromium.org/669243003/diff/1/src/utils/SkTextureCompresso...
> src/utils/SkTextureCompressor_Utils.h:13: #if 1
> Do you intend for these to be static? They're named that way.

[pavelk, 2014-10-23 16:05:03]

I've added my name to the AUTHORS file.

Yes, it does slow things down a tad, but it offers more accurate compression --
I've added a note in the _Utils.h file.

https://codereview.chromium.org/669243003/diff/1/src/utils/SkTextureCompresso...
File src/utils/SkTextureCompressor_Utils.h (right):

https://codereview.chromium.org/669243003/diff/1/src/utils/SkTextureCompresso...
src/utils/SkTextureCompressor_Utils.h:13: #if 1
On 2014/10/23 12:10:20, robertphillips wrote:
> Do you intend for these to be static? They're named that way.

Yep. :)

[robertphillips, 2014-10-23 16:11:29]

lgtm. I have verified that Pavel has signed the CLA.

[robertphillips, 2014-10-23 16:12:04]

The CQ bit was checked by robertphillips@google.com

[commit-bot: I haz the power, 2014-10-23 16:12:33]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/669243003/40001

[commit-bot: I haz the power, 2014-10-23 16:28:54]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-10-23 16:28:55]

Try jobs failed on following builders:
  Test-Ubuntu13.10-GCE-NoGPU-x86_64-Debug-Trybot on client.skia
(http://build.chromium.org/p/client.skia/builders/Test-Ubuntu13.10-GCE-NoGPU-x...)

[pavelk, 2014-10-23 20:09:59]

The CQ bit was checked by pavel@cs.unc.edu

[commit-bot: I haz the power, 2014-10-23 20:10:45]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/669243003/60001

[commit-bot: I haz the power, 2014-10-23 20:18:54]

Committed patchset #4 (id:60001) as 47eedcc848e7712aea9607fd489abdc70bc8bcfd