Remove a DCHECK in BitmapUploader which causes crash in some unittests

components/bitmap_uploader/bitmap_uploader.cc

Index: components/bitmap_uploader/bitmap_uploader.cc
diff --git a/components/bitmap_uploader/bitmap_uploader.cc b/components/bitmap_uploader/bitmap_uploader.cc
index 5c255dbf2f8a2b68dc45626b5ed808b730b0b526..115053f83d29023d3f7a224935e6358075b9484e 100644
--- a/components/bitmap_uploader/bitmap_uploader.cc
+++ b/components/bitmap_uploader/bitmap_uploader.cc
@@ -49,7 +49,8 @@ void BitmapUploader::Init(shell::Connector* connector) {
   gpu_service_->CreateOffscreenGLES2Context(GetProxy(&gles2_client));
   gles2_context_.reset(new mus::GLES2Context(
       std::vector<int32_t>(), gles2_client.PassInterface().PassHandle()));
-  DCHECK(gles2_context_->Initialize());
+  if (!gles2_context_->Initialize())
+    gles2_context_.reset();

[sadrul, 2016/05/26 15:41:35]

Can you add a DCHECK here that if Initialize() returns false, then that's
because gpu_service_ had an error of some sort, i.e. install a
set_connection_error_handler on |gpu_service_|, and DCHECK that the handler was
called if Initialize() returned false?

[Peng, 2016/05/30 14:04:25]

gpu_service_.encountered_error() is not true at here.
But if command_buffer_.encountered_error() is true at
https://code.google.com/p/chromium/codesearch#chromium/src/components/mus/pub...
.

Any thought?

 }
 
 // Sets the color which is RGBA.