Fix undefined behavior in DirectWrite GDI compatible metrics.

Fix undefined behavior in DirectWrite GDI compatible metrics.

GetGdiCompatibleGlyphMetrics checks the matrix passed to it.
If the matrix has any NaNs or INFs or otherwise looks bad, it returns
E_INVALIDARG and so the advances will be zero.

BUG=skia:2579
R=reed@google.com

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

[bungeman-skia, 2014-05-21 17:27:23]

The changes to SkTraceHR were done because we actually always want to log when
SkTraceHR is called. SkHRESULT.h determines whether the call will happen or not,
so it is redundant to also have the content of SkTraceHR be guarded the way it
was. The reason for the change is that if it had been this way to begin with, I
could have found the bug faster.

[reed1, 2014-05-21 17:59:39]

lgtm

[bungeman-skia, 2014-05-21 19:44:42]

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