Propagate supported scanout formats from driver to SurfaceFactoryOzone.

ui/ozone/platform/drm/client_native_pixmap_factory_gbm.cc

 // Copyright 2015 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 "ui/ozone/platform/drm/client_native_pixmap_factory_gbm.h"
 
 #include <utility>
 
 #include "base/macros.h"
 #include "ui/gfx/native_pixmap_handle_ozone.h"
+#include "ui/ozone/platform/drm/common/drm_util.h"
 #include "ui/ozone/public/client_native_pixmap_factory.h"  // nogncheck
+#include "ui/ozone/public/ozone_platform.h"
 
 #if defined(USE_VGEM_MAP)
 #include "ui/ozone/platform/drm/common/client_native_pixmap_vgem.h"
 #endif
 
 namespace ui {
 
 namespace {
 
 class ClientNativePixmapGbm : public ClientNativePixmap {
@@ skipping 22 matching lines @@
     // It's called in IO thread. We rely on clients for thread-safety.
     // Switching to an IPC message filter ensures thread-safety.
     DCHECK_LT(vgem_fd_.get(), 0);
     vgem_fd_ = std::move(device_fd);
 #endif
   }
   bool IsConfigurationSupported(gfx::BufferFormat format,
                                 gfx::BufferUsage usage) const override {
     switch (usage) {
       case gfx::BufferUsage::GPU_READ:
-      case gfx::BufferUsage::SCANOUT:
-        return format == gfx::BufferFormat::RGBA_8888 ||
-               format == gfx::BufferFormat::RGBX_8888 ||
-               format == gfx::BufferFormat::BGRA_8888 ||
-               format == gfx::BufferFormat::BGRX_8888;
+      case gfx::BufferUsage::SCANOUT: {
+        std::vector<uint32_t> support_formats;
+        ui::OzonePlatform::GetInstance()->GetSupportedFormats(&support_formats);

[spang, 2016/01/18 18:14:31]

The renderer process is allowed to have a ClientNativePixmapFactory, but doesn't
have an OzonePlatform. 

You'll need to use IPC to support this in the renderer case (adding an
initialization handshake between gpu process & renderer/browser).

The gpu process format query probably needs to be moved from
ClientNativePixmapFactory to SurfaceFactoryOzone; After that I don't think the
gpu process will use ClientNativePixmapFactory.

[william.xie, 2016/01/19 04:40:16]

Done.

+        return find(support_formats.begin(), support_formats.end(),
+                    GetFourCCFormatForFramebuffer(format)) !=
+               support_formats.end();
+      }
+
       case gfx::BufferUsage::GPU_READ_CPU_READ_WRITE:
       case gfx::BufferUsage::GPU_READ_CPU_READ_WRITE_PERSISTENT: {
 #if defined(USE_VGEM_MAP)
         return vgem_fd_.is_valid() && format == gfx::BufferFormat::BGRA_8888;
 #else
         return false;
 #endif
       }
     }
     NOTREACHED();
@@ skipping 32 matching lines @@
 #endif
 
   DISALLOW_COPY_AND_ASSIGN(ClientNativePixmapFactoryGbm);
 };
 
 ClientNativePixmapFactory* CreateClientNativePixmapFactoryGbm() {
   return new ClientNativePixmapFactoryGbm();
 }
 
 }  // namespace ui