remove U16 support, just support F16

remove U16 support, just support F16

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1889753002

Committed: https://skia.googlesource.com/skia/+/3dc6aac5da2dbeb56cda025f3699cad72e1a4b4e

[reed1, 2016-04-14 04:50:29]

Description was changed from

==========
remove U16 support, just support F16

BUG=skia:
==========

to

==========
remove U16 support, just support F16

BUG=skia:
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

[reed1, 2016-04-14 14:19:09]

The CQ bit was checked by reed@google.com to run a CQ dry run

[commit-bot: I haz the power, 2016-04-14 14:19:16]

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

[commit-bot: I haz the power, 2016-04-14 14:31:58]

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

[commit-bot: I haz the power, 2016-04-14 14:31:59]

Dry run: This issue passed the CQ dry run.

[reed1, 2016-04-14 14:37:14]

reed@google.com changed reviewers:
+ fmalita@chromium.org, herb@google.com, mtklein@google.com

[reed1, 2016-04-14 14:37:14]

au revoir mes amis

[f(malita), 2016-04-14 15:22:57]

lgtm

https://codereview.chromium.org/1889753002/diff/1/include/core/SkXfermode.h
File include/core/SkXfermode.h (right):

https://codereview.chromium.org/1889753002/diff/1/include/core/SkXfermode.h#n...
include/core/SkXfermode.h:266: typedef void (*LCF16Proc)(uint64_t* dst, const
SkPM4f* src, int count, const uint16_t lcd[]);
nit: LCDF16Proc?  I'm fine with either if intentional.

https://codereview.chromium.org/1889753002/diff/1/src/core/SkXfermodeF16.cpp
File src/core/SkXfermodeF16.cpp (right):

https://codereview.chromium.org/1889753002/diff/1/src/core/SkXfermodeF16.cpp#...
src/core/SkXfermodeF16.cpp:19: static void xfer_u64_1(const SkXfermode* xfer,
uint64_t dst[], const SkPM4f* src, int count,
nit: xfer_f16_1/xfer_f16_n?

[mtklein, 2016-04-14 15:39:59]

lgtm, with same confusion about "LCF16"

[reed1, 2016-04-14 15:52:08]

https://codereview.chromium.org/1889753002/diff/1/include/core/SkXfermode.h
File include/core/SkXfermode.h (right):

https://codereview.chromium.org/1889753002/diff/1/include/core/SkXfermode.h#n...
include/core/SkXfermode.h:266: typedef void (*LCF16Proc)(uint64_t* dst, const
SkPM4f* src, int count, const uint16_t lcd[]);
On 2016/04/14 15:22:56, f(malita) wrote:
> nit: LCDF16Proc?  I'm fine with either if intentional.

Typo, will fix

https://codereview.chromium.org/1889753002/diff/1/src/core/SkXfermodeF16.cpp
File src/core/SkXfermodeF16.cpp (right):

https://codereview.chromium.org/1889753002/diff/1/src/core/SkXfermodeF16.cpp#...
src/core/SkXfermodeF16.cpp:19: static void xfer_u64_1(const SkXfermode* xfer,
uint64_t dst[], const SkPM4f* src, int count,
On 2016/04/14 15:22:57, f(malita) wrote:
> nit: xfer_f16_1/xfer_f16_n?

Done.

[reed1, 2016-04-14 15:52:12]

The CQ bit was checked by reed@google.com to run a CQ dry run

[commit-bot: I haz the power, 2016-04-14 15:52:16]

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

[reed1, 2016-04-14 15:59:23]

The CQ bit was unchecked by reed@google.com

[reed1, 2016-04-14 15:59:41]

The CQ bit was checked by reed@google.com

[reed1, 2016-04-14 15:59:41]

The patchset sent to the CQ was uploaded after l-g-t-m from mtklein@google.com,
fmalita@chromium.org
Link to the patchset: https://codereview.chromium.org/1889753002/#ps20001
(title: "address reviewer comments, mostly fixing class/function names")

[commit-bot: I haz the power, 2016-04-14 15:59:47]

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

[commit-bot: I haz the power, 2016-04-14 16:02:16]

Description was changed from

==========
remove U16 support, just support F16

BUG=skia:
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

to

==========
remove U16 support, just support F16

BUG=skia:
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...

Committed:
https://skia.googlesource.com/skia/+/3dc6aac5da2dbeb56cda025f3699cad72e1a4b4e
==========

[commit-bot: I haz the power, 2016-04-14 16:02:18]

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