add some asserts for mipmaps, and cleanup fFilterLevel if mipping failed

add some asserts for mipmaps, and cleanup fFilterLevel if mipping failed

BUG=skia:

Committed: http://code.google.com/p/skia/source/detail?r=14241

[reed1, 2014-04-17 15:15:16]

investigating https://code.google.com/p/chromium/issues/detail?id=362154 , so
just trying to harden this code.

[scroggo, 2014-04-17 15:34:48]

On 2014/04/17 15:15:16, reed1 wrote:
> investigating https://code.google.com/p/chromium/issues/detail?id=362154 , so
> just trying to harden this code.

I don't have permission to view that bug :(

lgtm

https://codereview.chromium.org/240633003/diff/20001/src/core/SkBitmapProcSta...
File src/core/SkBitmapProcState.cpp (right):

https://codereview.chromium.org/240633003/diff/20001/src/core/SkBitmapProcSta...
src/core/SkBitmapProcState.cpp:407: // If we are "still" kMedium_FilterLeve,
then then request was not fulfilled by possiblyScale,
nit: kMedium_FilterLevel ('l' missing).

https://codereview.chromium.org/240633003/diff/20001/src/core/SkBitmapProcSta...
src/core/SkBitmapProcState.cpp:407: // If we are "still" kMedium_FilterLeve,
then then request was not fulfilled by possiblyScale,
'then then' -> 'then the'

https://codereview.chromium.org/240633003/diff/20001/src/core/SkBitmapProcSta...
src/core/SkBitmapProcState.cpp:408: // so we downgrade to kLow (so the rest of
the sniffing code can assume that
nit: missing ')'

[reed1, 2014-04-17 15:40:13]

https://codereview.chromium.org/240633003/diff/20001/src/core/SkBitmapProcSta...
File src/core/SkBitmapProcState.cpp (right):

https://codereview.chromium.org/240633003/diff/20001/src/core/SkBitmapProcSta...
src/core/SkBitmapProcState.cpp:407: // If we are "still" kMedium_FilterLeve,
then then request was not fulfilled by possiblyScale,
On 2014/04/17 15:34:49, scroggo wrote:
> nit: kMedium_FilterLevel ('l' missing).

Done.

https://codereview.chromium.org/240633003/diff/20001/src/core/SkBitmapProcSta...
src/core/SkBitmapProcState.cpp:407: // If we are "still" kMedium_FilterLeve,
then then request was not fulfilled by possiblyScale,
On 2014/04/17 15:34:49, scroggo wrote:
> 'then then' -> 'then the'

Done.

https://codereview.chromium.org/240633003/diff/20001/src/core/SkBitmapProcSta...
src/core/SkBitmapProcState.cpp:408: // so we downgrade to kLow (so the rest of
the sniffing code can assume that
On 2014/04/17 15:34:49, scroggo wrote:
> nit: missing ')'

Done.

[reed1, 2014-04-17 15:40:17]

The CQ bit was checked by reed@google.com

[commit-bot: I haz the power, 2014-04-17 15:40:34]

CQ is trying da patch. Follow status at
https://skia-tree-status.appspot.com/cq/reed@google.com/240633003/40001

[commit-bot: I haz the power, 2014-04-17 16:46:09]

Change committed as 14241