Changes to make SkPDiff more robost.

Changes to make SkPDiff more robost.

bitmap_to_cielab now returns a boolean. Instead of asserting that the
config is 8888, copy to 8888 (and return false on failure). This allows
skpdiff to work on Index8 bitmaps (without changing the code that does
the real work). If it returns false, do not attempt to create a pmetric.

In pmetric, exit early if maxLevels is <= 2, since this would crash later
in the function (creation of an ImageL3D if maxLevels is 0, and then the
creation of an array of a negative number of entries). maxLevels is small
if the width or height of the image is small (i.e. a bitmap with width 1
has maxLevels of 0).

R=djsollen@google.com

Committed: https://code.google.com/p/skia/source/detail?r=12239

[scroggo, 2013-11-07 20:30:29]

Derek, I don't know if your other change will obsolete this, but I needed to
make these changes in order to not crash (independent of multithreading).

[djsollen, 2013-11-07 21:44:33]

lgtm

[scroggo, 2013-11-12 14:41:23]

Committed patchset #1 manually as r12239 (presubmit successful).