Reland "Replace gl::trace logging with Chromium style logging"

Removing one usage of FormatString() and its static buffer.
And preparation for Platform logging.

Fix incorrect enabling of ERR() calls in UNIMPLEMENTED() and
UNREACHABLE(), resulting in increased code size and
<iostream> adding 5 static initializers to chrome because of
cerr referenced in statically linked translator.

BUG=angleproject:1660

Change-Id: I7caa18036118d532e0544f75278602559172ae04
Reviewed-on: https://chromium-review.googlesource.com/431457
Reviewed-by: Jamie Madill <jmadill@chromium.org>
Commit-Queue: Jamie Madill <jmadill@chromium.org>
diff --git a/src/libANGLE/renderer/vulkan/FramebufferVk.cpp b/src/libANGLE/renderer/vulkan/FramebufferVk.cpp
index 522ec4f..c6eb3fc 100644
--- a/src/libANGLE/renderer/vulkan/FramebufferVk.cpp
+++ b/src/libANGLE/renderer/vulkan/FramebufferVk.cpp
@@ -217,7 +217,7 @@
     // TODO(jmadill): Handle getRenderTarget error.
     if (errOrResult.isError())
     {
-        ERR("Internal error in FramebufferVk::getImplementationColorReadFormat.");
+        ERR() << "Internal error in FramebufferVk::getImplementationColorReadFormat.";
         return GL_NONE;
     }
 
@@ -231,7 +231,7 @@
     // TODO(jmadill): Handle getRenderTarget error.
     if (errOrResult.isError())
     {
-        ERR("Internal error in FramebufferVk::getImplementationColorReadFormat.");
+        ERR() << "Internal error in FramebufferVk::getImplementationColorReadFormat.";
         return GL_NONE;
     }