Implement GLHelperReadbackSupport for GLHelper usage.

content/common/gpu/client/gl_helper_unittest.cc

 // Copyright 2014 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 <stdio.h>
 #include <cmath>
 #include <string>
 #include <vector>
 
 #include <GLES2/gl2.h>
@@ skipping 983 matching lines @@
                                    gfx::Rect(src_size),
                                    pixels,
                                    bitmap_config);
     }
   }
 
   // Test basic format readback.
   bool TestTextureFormatReadback(const gfx::Size& src_size,
                          SkBitmap::Config bitmap_config,
                          bool async) {
-    DCHECK((bitmap_config == SkBitmap::kRGB_565_Config) ||
-           (bitmap_config == SkBitmap::kARGB_8888_Config));
-    bool rgb565_format = (bitmap_config == SkBitmap::kRGB_565_Config);
-    if (rgb565_format && !helper_->CanUseRgb565Readback()) {
-      LOG(INFO) << "RGB565 Format Not supported on this platform";
-      LOG(INFO) << "Skipping RGB565ReadBackTest";
+    if (!helper_->IsReadBackConfigSupported(bitmap_config)) {
+      LOG(INFO) << "Format Not supported on this platform" << bitmap_config
+                << "Skipping the test";

[no sievers, 2014/02/19 19:09:39]

nit: does this need to be spaced? something like
 LOG(INFO) << "Format Not supported on this platform: " << bitmap_config << ".
Skipping the test";

[sivag, 2014/02/21 11:40:51]

Done.

       return true;
     }
     WebGLId src_texture = context_->createTexture();
     SkBitmap input_pixels;
     input_pixels.setConfig(bitmap_config, src_size.width(),
                            src_size.height());
     input_pixels.allocPixels();
     SkAutoLockPixels lock1(input_pixels);
     // Test Pattern-1, Fill with Plain color pattern.
     // Erase the input bitmap with red color.
@@ skipping 650 matching lines @@
 #endif
 #if defined(TOOLKIT_GTK)
   gfx::GtkInitFromCommandLine(*CommandLine::ForCurrentProcess());
 #endif
   gpu::ApplyGpuDriverBugWorkarounds(CommandLine::ForCurrentProcess());
 
   content::UnitTestTestSuite runner(suite);
   base::MessageLoop message_loop;
   return runner.Run();
 }