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

ui/ozone/common/gpu/ozone_gpu_message_params.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_COMMON_GPU_OZONE_GPU_MESSAGE_PARAMS_H_
 #define UI_OZONE_COMMON_GPU_OZONE_GPU_MESSAGE_PARAMS_H_
 
 #include <string>
 #include <vector>
 
@@ skipping 30 matching lines @@
   std::vector<DisplayMode_Params> modes;
   bool has_current_mode = false;
   DisplayMode_Params current_mode;
   bool has_native_mode = false;
   DisplayMode_Params native_mode;
   int64_t product_id = 0;
   std::string string_representation;
 };
 
 struct OZONE_EXPORT OverlayCheck_Params {
+  enum State { kTest, kInvalid, kCompatible, kOverlay };

[dnicoara, 2015/11/23 19:22:54]

nit: Could you please have kInvalid as the first state? It'll better align with
states elsewhere.

+
   OverlayCheck_Params();
   OverlayCheck_Params(
       const OverlayCandidatesOzone::OverlaySurfaceCandidate& candidate);
+  bool operator==(const OverlayCheck_Params& param) const;
   ~OverlayCheck_Params();
 
   gfx::Size buffer_size;
   gfx::OverlayTransform transform = gfx::OVERLAY_TRANSFORM_INVALID;
   gfx::BufferFormat format = gfx::BufferFormat::BGRA_8888;
   gfx::Rect display_rect;
   gfx::RectF crop_rect;
   int plane_z_order = 0;
   // Higher the value, the more important it is to ensure that this
   // overlay candidate finds a compatible free hardware plane to use.
   uint32_t weight;
-  // Will be set in GPU process. These are unique plane ids of primary display
-  // supporting this configuration.
-  std::vector<uint32_t> plane_ids;
+  // state can contain the following values:
+  // kTest      : This is a new configuration which needs to be tested. Usually
+  //              this state is set when OverlayCheck_Params is created by
+  //              taking OverlaySurfaceCandidate in its constructor.
+  // kInvalid   : This configuration has been tested and has proven to be
+  //              incompatible with the current setup.
+  // kCompatible: This configuration has been tested and we have hardware planes
+  //              which can support this configuration. This doesn't mean that
+  //              the overlay candidate should be promoted to use hardware
+  //              planes. This depends on the weight of this candidate,
+  //              available hardware resources and total no of overlay
+  //              candidates.
+  // kOverlay   : This configuration is compatible with the current setup and
+  //              overlay candidate should be promoted to use Hardware Overlay.
+  State state;
+  // The following params are set to appropriate values by DrmOverlayCandidate
+  // after checking and validating the optimal configuration.
+
+  // This is optimal storage format for the buffer.
+  gfx::BufferFormat optimal_format;
+  // This is the required buffer size.
+  gfx::Size required_buffer_size;
 };
 
 }  // namespace ui
 
 #endif  // UI_OZONE_COMMON_GPU_OZONE_GPU_MESSAGE_PARAMS_H_