Catch utility process StartBatchMode failures

chrome/browser/image_decoder.cc

Index: chrome/browser/image_decoder.cc
diff --git a/chrome/browser/image_decoder.cc b/chrome/browser/image_decoder.cc
index cfd887e06a9f4454330f9cd7e4d102506a1d78f3..9e21662088f5ab09a5cd61b71057da235bb6fe24 100644
--- a/chrome/browser/image_decoder.cc
+++ b/chrome/browser/image_decoder.cc
@@ -80,6 +80,16 @@ void ImageDecoder::DecodeImageInSandbox(
   if (!utility_process_host_) {
     StartBatchMode();
   }
+  if (!utility_process_host_) {
+    // Utility process failed to start; notify delegate and return.
+    // Without this check, we were seeing crashes on startup. Further
+    // investigation is needed to determine why the utility process
+    // is failing to start. See crbug.com/472272
+    image_request->task_runner()->PostTask(
+        FROM_HERE, base::Bind(&ImageRequest::OnDecodeImageFailed,
+                              base::Unretained(image_request)));
+    return;
+  }
 
   last_request_ = base::TimeTicks::Now();
   base::AutoLock lock(map_lock_);
@@ -117,7 +127,10 @@ void ImageDecoder::StartBatchMode() {
   utility_process_host_ =
       UtilityProcessHost::Create(this, base::MessageLoopProxy::current().get())
           ->AsWeakPtr();
-  utility_process_host_->StartBatchMode();
+  if (!utility_process_host_->StartBatchMode()) {
+     utility_process_host_.reset();
+     return;
+  }
   batch_mode_timer_.Start(
       FROM_HERE, base::TimeDelta::FromSeconds(kBatchModeTimeoutSeconds),
       this, &ImageDecoder::StopBatchMode);