Fix error handling in DirectWrite with tiny text.

Fix error handling in DirectWrite with tiny text.

If there is an error while trying to determine the metrics,
we need to bail instead of potentially using uninitialized data.

Committed: https://skia.googlesource.com/skia/+/683a37692bc67ef1144922b73a1e871e7e1e842e

[bungeman-skia, 2014-08-28 17:04:17]

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

[bungeman-skia, 2014-08-28 17:19:57]

bungeman@google.com changed reviewers:
+ mtklein@google.com

[mtklein, 2014-08-28 17:25:50]

https://codereview.chromium.org/511783003/diff/1/src/ports/SkScalerContext_wi...
File src/ports/SkScalerContext_win_dw.cpp (right):

https://codereview.chromium.org/511783003/diff/1/src/ports/SkScalerContext_wi...
src/ports/SkScalerContext_win_dw.cpp:477: if (bbox.left == bbox.right ||
bbox.top == bbox.bottom) {
== -> >= for both, and above?

[bungeman-skia, 2014-08-28 18:01:04]

ptal

https://codereview.chromium.org/511783003/diff/1/src/ports/SkScalerContext_wi...
File src/ports/SkScalerContext_win_dw.cpp (right):

https://codereview.chromium.org/511783003/diff/1/src/ports/SkScalerContext_wi...
src/ports/SkScalerContext_win_dw.cpp:477: if (bbox.left == bbox.right ||
bbox.top == bbox.bottom) {
On 2014/08/28 17:25:49, mtklein wrote:
> == -> >= for both, and above?

Done.

[mtklein, 2014-08-28 18:33:52]

lgtm

[bungeman-skia, 2014-08-28 18:34:37]

The CQ bit was checked by bungeman@google.com

[commit-bot: I haz the power, 2014-08-28 18:35:28]

CQ is trying da patch. Follow status at
 https://skia-tree-status.appspot.com/cq/bungeman@google.com/511783003/20001

[commit-bot: I haz the power, 2014-08-28 18:42:34]

Committed patchset #2 (id:20001) as 683a37692bc67ef1144922b73a1e871e7e1e842e