Add tests (and fix!) for known bad ICO files.

We previously saw crashes decoding bad ICO files. Add tests for
known bad files.

While testing, I learned that one of them still crashes. Check for
large offset and size separately to fix the crash.

BUG=skia:2878

Review URL: https://codereview.chromium.org/712123002
Remove SK_FM_NEW_MATCH_FAMILY_STYLE_CHARACTER.

This removes the old guarded code and enables the new api
introduced with "Update fontMgr to take list of bcp47 language tags."
c20386e3937d3d398ac9b35f9c7d997e972ade98 . Blink on Android is
already using the new code.

Review URL: https://codereview.chromium.org/705843004
SkDocument_PDF does somthing with the trimBox parameter.

Fixes change introduced in http://crrev.com/645563002

This will allow SkDocument to be a drop-in repalcement to
SkPDFDocument/SkPDFDevice in Chromium.

Review URL: https://codereview.chromium.org/711133002
Remove outdated comment in SkImageDecoder.h

The comment was no longer relevant with
https://codereview.chromium.org/483593002

Review URL: https://codereview.chromium.org/715673002
Simplify canvas calls in SkDocument_PDF

As suggested by reed@ in http://crrev.com/711133002

TBR=reed@google.com

Review URL: https://codereview.chromium.org/707533003
tests/Time.cpp: more verbose errors

Review URL: https://codereview.chromium.org/711143002