Don't dump pixel results if we fail to encode the image

content/shell/layout_test_controller_host.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/shell/layout_test_controller_host.h"
 
 #include "base/message_loop.h"
 #include "content/public/browser/render_view_host.h"
 #include "content/shell/shell_messages.h"
 #include "webkit/support/webkit_support_gfx.h"
@@ skipping 121 matching lines @@
   if (actual_pixel_hash != expected_pixel_hash_) {
     std::vector<unsigned char> png;
 
     // Only the expected PNGs for Mac have a valid alpha channel.
 #if defined(OS_MACOSX)
     bool discard_transparency = false;
 #else
     bool discard_transparency = true;
 #endif
 
-    webkit_support::EncodeBGRAPNGWithChecksum(
-        reinterpret_cast<const unsigned char*>(image.getPixels()),
-        image.width(),
-        image.height(),
-        static_cast<int>(image.rowBytes()),
-        discard_transparency,
-        actual_pixel_hash,
-        &png);
-
-    printf("Content-Type: image/png\n");
-    printf("Content-Length: %u\n", static_cast<unsigned>(png.size()));
-    fwrite(&png[0], 1, png.size(), stdout);
+    if (webkit_support::EncodeBGRAPNGWithChecksum(
+            reinterpret_cast<const unsigned char*>(image.getPixels()),
+            image.width(),
+            image.height(),
+            static_cast<int>(image.rowBytes()),
+            discard_transparency,
+            actual_pixel_hash,
+            &png)) {
+      printf("Content-Type: image/png\n");
+      printf("Content-Length: %u\n", static_cast<unsigned>(png.size()));
+      fwrite(&png[0], 1, png.size(), stdout);
+    }
   }
 
   MessageLoop::current()->PostTask(FROM_HERE, MessageLoop::QuitClosure());
 #endif
 }
 
 void LayoutTestControllerHost::OnNotifyDone() {
   if (!wait_until_done_)
     return;
   watchdog_.Cancel();
@@ skipping 36 matching lines @@
     return;
   printf("FAIL: NOT IMPLEMENTED: %s.%s\n",
          object_name.c_str(), property_name.c_str());
   fprintf(stderr, "FAIL: NOT IMPLEMENTED: %s.%s\n",
           object_name.c_str(), property_name.c_str());
   watchdog_.Cancel();
   CaptureDump();
 }
 
 }  // namespace content