Add F16 source to the linear pipelin.

Add F16 source to the linear pipelin.

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

Committed: https://skia.googlesource.com/skia/+/d5f2e2e7429b643943d2f8a532c65e2498223b5b

[herb_g, 2016-04-14 16:28:18]

Description was changed from

==========
Add F16 source to the linear pipelin.

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

to

==========
Add F16 source to the linear pipelin.

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

[herb_g, 2016-04-14 16:29:02]

Description was changed from

==========
Add F16 source to the linear pipelin.

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

to

==========
Add F16 source to the linear pipelin.

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

[herb_g, 2016-04-14 16:29:02]

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

[herb_g, 2016-04-14 16:29:15]

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

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

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

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

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

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

Dry run: This issue passed the CQ dry run.

[reed1, 2016-04-14 17:25:28]

https://codereview.chromium.org/1882893003/diff/1/src/core/SkLinearBitmapPipe...
File src/core/SkLinearBitmapPipeline.cpp (right):

https://codereview.chromium.org/1882893003/diff/1/src/core/SkLinearBitmapPipe...
src/core/SkLinearBitmapPipeline.cpp:658:
sampleStage->initStage<Sampler<PixelHalfSRGB, Blender>>(next, srcPixmap);
why is "SRGB" part of this enum name?

[herb_g, 2016-04-14 17:44:12]

Fix think-o for Half name.

[herb_g, 2016-04-14 17:45:00]

https://codereview.chromium.org/1882893003/diff/1/src/core/SkLinearBitmapPipe...
File src/core/SkLinearBitmapPipeline.cpp (right):

https://codereview.chromium.org/1882893003/diff/1/src/core/SkLinearBitmapPipe...
src/core/SkLinearBitmapPipeline.cpp:658:
sampleStage->initStage<Sampler<PixelHalfSRGB, Blender>>(next, srcPixmap);
On 2016/04/14 17:25:27, reed1 wrote:
> why is "SRGB" part of this enum name?

Think-o on my part. Fixed.

[herb_g, 2016-04-14 17:45:10]

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

[commit-bot: I haz the power, 2016-04-14 17:45:12]

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

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

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

[commit-bot: I haz the power, 2016-04-14 18:03:00]

Dry run: This issue passed the CQ dry run.

[reed1, 2016-04-14 18:15:07]

lgtm

[herb_g, 2016-04-14 18:15:44]

The CQ bit was checked by herb@google.com

[commit-bot: I haz the power, 2016-04-14 18:15:51]

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

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

Description was changed from

==========
Add F16 source to the linear pipelin.

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

to

==========
Add F16 source to the linear pipelin.

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/+/d5f2e2e7429b643943d2f8a532c65e2498223b5b
==========

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

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

[reed1, 2016-04-14 20:15:44]

https://codereview.chromium.org/1882893003/diff/20001/src/core/SkLinearBitmap...
File src/core/SkLinearBitmapPipeline_sample.h (right):

https://codereview.chromium.org/1882893003/diff/20001/src/core/SkLinearBitmap...
src/core/SkLinearBitmapPipeline_sample.h:772: const uint32_t* src =
static_cast<const uint32_t*>(vsrc);
Why is src a uint32_t? I'm not very familiar w/ this code, so perhaps this is
correct...

https://codereview.chromium.org/1882893003/diff/20001/src/core/SkLinearBitmap...
src/core/SkLinearBitmapPipeline_sample.h:780: const uint64_t* src =
static_cast<const uint64_t*>(vsrc);
Do we need to add "index" to src ?