SkScan_Antihair: assert in debug and potentialy lost last pixels

SkScan_Antihair: assert in debug and potentialy lost last pixels

alpha: (uint8_t) 256 is clamped to 255 before static cast to
uint8_t in SkToU8.

BUG=4406
Committed: https://skia.googlesource.com/skia/+/7600183ba7d8c8df6fbba1b32a73e6394216754e

[aleksandar.stojiljkovic, 2015-10-28 22:02:09]

aleksandar.stojiljkovic@intel.com changed reviewers:
+ fmalita@chromium.org, reed@chromium.org

[reed1, 2015-10-29 19:04:44]

reed@google.com changed reviewers:
+ reed@google.com

[reed1, 2015-10-29 19:04:44]

If you want to make this CL real, it should include a test that fails pre-CL,
and pases post-CL.

https://code.google.com/p/skia/issues/detail?id=4406 this bug has a code snippet
that could be turned into a unittests (where passing means it doesn't assert).

[reed1, 2015-11-02 18:54:18]

nice.

lgtm

[aleksandar.stojiljkovic, 2015-11-02 21:04:58]

On 2015/11/02 18:54:18, reed1 wrote:
> nice.
> 
> lgtm

Don't have trybot and commit rights to repository; need to ask a committer to
submit it after review.

[reed1, 2015-11-02 21:17:31]

The CQ bit was checked by reed@google.com

[commit-bot: I haz the power, 2015-11-02 21:17:43]

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

[commit-bot: I haz the power, 2015-11-02 21:28:53]

Committed patchset #2 (id:20001) as
https://skia.googlesource.com/skia/+/7600183ba7d8c8df6fbba1b32a73e6394216754e