Add specialized rgba8888 unit sampler.

Add specialized rgba8888 unit sampler.

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

Committed: https://skia.googlesource.com/skia/+/6a5d7139ff17a2dfdf136de35f7dd9c94c1f648b

[herb_g, 2016-04-07 17:34:14]

Description was changed from

==========
Add specialized rgba8888 unit sampler.

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

to

==========
Add specialized rgba8888 unit sampler.

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

[herb_g, 2016-04-07 17:35:41]

Description was changed from

==========
Add specialized rgba8888 unit sampler.

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

to

==========
Add specialized rgba8888 unit sampler.

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

[herb_g, 2016-04-07 17:35:41]

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

[herb_g, 2016-04-07 21:41:12]

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

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

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

[commit-bot: I haz the power, 2016-04-07 21:51:24]

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

[commit-bot: I haz the power, 2016-04-07 21:51:25]

Dry run: This issue passed the CQ dry run.

[f(malita), 2016-04-08 12:58:13]

lgtm

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

https://codereview.chromium.org/1868723003/diff/1/src/core/SkLinearBitmapPipe...
src/core/SkLinearBitmapPipeline.cpp:549: return &fSrc[fWidth * y + x];
Do we need to worry about row padding here or rowbytes never comes into play?

[herb_g, 2016-04-08 13:48:59]

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

https://codereview.chromium.org/1868723003/diff/1/src/core/SkLinearBitmapPipe...
src/core/SkLinearBitmapPipeline.cpp:549: return &fSrc[fWidth * y + x];
On 2016/04/08 12:58:12, f(malita) wrote:
> Do we need to worry about row padding here or rowbytes never comes into play?

I talked with MikeR about this. RowBytes must always be a multiple of pixel
size, so just storing the with in pixels is all you need.

[herb_g, 2016-04-08 13:49:04]

The CQ bit was checked by herb@google.com

[commit-bot: I haz the power, 2016-04-08 13:49:08]

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

[commit-bot: I haz the power, 2016-04-08 13:50:02]

Description was changed from

==========
Add specialized rgba8888 unit sampler.

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

to

==========
Add specialized rgba8888 unit sampler.

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/+/6a5d7139ff17a2dfdf136de35f7dd9c94c1f648b
==========

[commit-bot: I haz the power, 2016-04-08 13:50:03]

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