Ozone: Dont hardcode format to YUV when using Overlay Composition.

ui/ozone/platform/drm/gpu/overlay_plane.h

 // 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.
 
 #ifndef UI_OZONE_PLATFORM_DRM_GPU_OVERLAY_PLANE_H_
 #define UI_OZONE_PLATFORM_DRM_GPU_OVERLAY_PLANE_H_
 
 #include <vector>
 
 #include "base/memory/ref_counted.h"
+#include "ui/gfx/buffer_types.h"
 #include "ui/gfx/geometry/rect.h"
 #include "ui/gfx/geometry/rect_f.h"
 #include "ui/gfx/overlay_transform.h"
 #include "ui/ozone/ozone_export.h"
 
 namespace ui {
 
 class ScanoutBuffer;
 
 struct OverlayPlane;
@@ skipping 14 matching lines @@
   // Returns the primary plane in |overlays|.
   static const OverlayPlane* GetPrimaryPlane(const OverlayPlaneList& overlays);
 
   scoped_refptr<ScanoutBuffer> buffer;
   int z_order = 0;
   gfx::OverlayTransform plane_transform;
   gfx::Rect display_bounds;
   gfx::RectF crop_rect;
 };
 
+struct OZONE_EXPORT OverlayBufferConfiguration {

[dnicoara, 2015/11/09 20:53:23]

This seems to only be used in DrmWindow. Maybe just declare it in there, in the
anonymous namespace.

[kalyank, 2015/11/09 21:44:05]

Eventually, we want to query this information in GbmBuffer and other places
interested in this information i.e. VaapiDrmPicture.
Currently, we have logic in GbmBuffer as to when scaling is needed and use VPP
hardware unit(in case of Video buffer)for this purpose.
Also, https://codereview.chromium.org/1422563002/ extends support in GbmBuffer
to add YUV support for Overlay when playing Videobuffer. We should consolidate
all this logic in one place and make sure we are using the same during test
commits.

[dnicoara, 2015/11/09 22:26:42]

Acknowledged.

[kalyank, 2015/11/19 16:10:41]

Removed this and saved needed data as part of overlaycheck_params

+  OverlayBufferConfiguration(gfx::BufferFormat buffer_format,
+                             gfx::Size buffer_size,

[dnicoara, 2015/11/09 20:53:23]

nit: const gfx::Size&

+                             bool handle_scaling);
+
+  ~OverlayBufferConfiguration();
+
+  // This is optimal storage format for the buffer.
+  gfx::BufferFormat buffer_format;
+  // This is the required buffer size.
+  gfx::Size buffer_size;
+  // Hardware doesn't have support for plane scaling and the buffer needs to
+  // be scaled using other means i.e. 3D Engine, VPP etc.
+  bool handle_scaling;
+};
+
 }  // namespace ui
 
 #endif  // UI_OZONE_PLATFORM_DRM_GPU_OVERLAY_PLANE_H_