Fix undefined behavior in DirectWrite GDI compatible metrics.

src/utils/win/SkHRESULT.cpp

Index: src/utils/win/SkHRESULT.cpp
===================================================================
--- src/utils/win/SkHRESULT.cpp	(revision 14827)
+++ src/utils/win/SkHRESULT.cpp	(working copy)
@@ -9,10 +9,11 @@
 
 #include "SkHRESULT.h"
 
-void SkTraceHR(const char* file, unsigned long line,
-               HRESULT hr, const char* msg) {
-    SkDEBUGCODE(if (NULL != msg) SkDEBUGF(("%s\n", msg)));
-    SkDEBUGF(("%s(%lu) : error 0x%x: ", file, line, hr));
+void SkTraceHR(const char* file, unsigned long line, HRESULT hr, const char* msg) {
+    if (NULL != msg) {
+        SkDebugf("%s\n", msg);
+    }
+    SkDebugf("%s(%lu) : error 0x%x: ", file, line, hr);
 
     LPSTR errorText = NULL;
     FormatMessageA(FORMAT_MESSAGE_ALLOCATE_BUFFER |
@@ -27,9 +28,9 @@
     );
 
     if (NULL == errorText) {
-        SkDEBUGF(("<unknown>\n"));
+        SkDebugf("<unknown>\n");
     } else {
-        SkDEBUGF(("%s", errorText));
+        SkDebugf("%s", errorText);
         LocalFree(errorText);
         errorText = NULL;
     }