Allow index sources to be unpremul.

Allow index sources to be unpremul.

This only affects the PNG image decoder, where we have already created
the color table without premultiplication. Since the RowProcChooser is
just providing a proc that chooses indices into the color table, it can
just return the same RowProc.

Update test_row_proc_choice. It was testing to ensure that we hadn't
changed the behavior from the original version of setPrefConfigTable.
In this case, we deliberately changed the behavior, so we need to
change the test.

BUG=b/12024301

Committed: https://skia.googlesource.com/skia/+/13f701a14d2a303b153ea861a3482e25254a9c3c

[scroggo, 2014-10-23 21:50:17]

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

[reed1, 2014-10-24 14:28:54]

lgtm w/ nit

https://codereview.chromium.org/657863005/diff/1/src/images/SkScaledBitmapSam...
File src/images/SkScaledBitmapSampler.cpp (right):

https://codereview.chromium.org/657863005/diff/1/src/images/SkScaledBitmapSam...
src/images/SkScaledBitmapSampler.cpp:400: // properly with respect to
premultiplication, so premul makes
s/premultiplication/opts.fPremultiplyAlpha

[scroggo, 2014-10-27 15:07:22]

https://codereview.chromium.org/657863005/diff/1/src/images/SkScaledBitmapSam...
File src/images/SkScaledBitmapSampler.cpp (right):

https://codereview.chromium.org/657863005/diff/1/src/images/SkScaledBitmapSam...
src/images/SkScaledBitmapSampler.cpp:400: // properly with respect to
premultiplication, so premul makes
On 2014/10/24 14:28:54, reed1 wrote:
> s/premultiplication/opts.fPremultiplyAlpha

Done.

[scroggo, 2014-10-27 15:07:25]

The CQ bit was checked by scroggo@google.com

[commit-bot: I haz the power, 2014-10-27 15:08:22]

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

[commit-bot: I haz the power, 2014-10-27 15:24:17]

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

[commit-bot: I haz the power, 2014-10-27 15:24:18]

Try jobs failed on following builders:
  Test-Ubuntu13.10-GCE-NoGPU-x86_64-Debug-Trybot on client.skia
(http://build.chromium.org/p/client.skia/builders/Test-Ubuntu13.10-GCE-NoGPU-x...)

[scroggo, 2014-10-27 15:34:53]

The CQ bit was checked by scroggo@google.com

[commit-bot: I haz the power, 2014-10-27 15:35:48]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/657863005/40001

[commit-bot: I haz the power, 2014-10-27 15:43:17]

Committed patchset #3 (id:40001) as 13f701a14d2a303b153ea861a3482e25254a9c3c