On gpu, use max(r,g,b) for coverage alpha in LCD and update lcd blend gm.

On gpu, use max(r,g,b) for coverage alpha in LCD and update lcd blend gm.

BUG=skia:

Committed: https://skia.googlesource.com/skia/+/27b6335c71bccb96206ff0495793433bfa982777

[egdaniel, 2015-09-15 18:57:29]

egdaniel@google.com changed reviewers:
+ bsalomon@google.com, jvanverth@google.com

[bsalomon, 2015-09-15 19:08:34]

https://codereview.chromium.org/1339213004/diff/1/gm/lcdblendmodes.cpp
File gm/lcdblendmodes.cpp (right):

https://codereview.chromium.org/1339213004/diff/1/gm/lcdblendmodes.cpp#newcode47
gm/lcdblendmodes.cpp:47: return SkString("lcdblendmodes");
why this change?

[egdaniel, 2015-09-15 19:10:30]

https://codereview.chromium.org/1339213004/diff/1/gm/lcdblendmodes.cpp
File gm/lcdblendmodes.cpp (right):

https://codereview.chromium.org/1339213004/diff/1/gm/lcdblendmodes.cpp#newcode47
gm/lcdblendmodes.cpp:47: return SkString("lcdblendmodes");
On 2015/09/15 19:08:34, bsalomon wrote:
> why this change?

I was confused about the need for sk_tool_utils::major_platform_os_name()), I
was thinking this was needed for Cary's portable text, but after talking to him
it is actually for the cases that explicitly don't want to use the portable
text.

[jvanverth1, 2015-09-15 19:38:01]

Changes to text shaders look good. As discussed, there are some issues with DF
Text, but those don't appear to be related to this change (filed as
https://code.google.com/p/skia/issues/detail?id=4345).

[bsalomon, 2015-09-15 19:40:20]

lgtm

[egdaniel, 2015-09-15 20:07:59]

The CQ bit was checked by egdaniel@google.com

[commit-bot: I haz the power, 2015-09-15 20:08:05]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1339213004/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1339213004/1

[commit-bot: I haz the power, 2015-09-15 20:13:53]

Committed patchset #1 (id:1) as
https://skia.googlesource.com/skia/+/27b6335c71bccb96206ff0495793433bfa982777