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

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

 // 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.
 
 #ifndef UI_OZONE_PLATFORM_DRM_GPU_DRM_OVERLAY_VALIDATOR_H_
 #define UI_OZONE_PLATFORM_DRM_GPU_DRM_OVERLAY_VALIDATOR_H_
 
 #include "base/containers/mru_cache.h"
 #include "ui/ozone/platform/drm/gpu/overlay_plane.h"
 
 namespace ui {
 
 class DrmDevice;
 class DrmWindow;
 class HardwareDisplayController;
 class ScanoutBufferGenerator;
 struct OverlayCheck_Params;
 
 class DrmOverlayValidator {
  public:
-  explicit DrmOverlayValidator(DrmWindow* window);
+  explicit DrmOverlayValidator(DrmWindow* window,
+                               ScanoutBufferGenerator* buffer_generator);
   ~DrmOverlayValidator();
 
   // Tests if configurations |params| are compatible with |window_| and finds
   // which of these configurations can be promoted to Overlay composition
   // without failing the page flip. It expects |params| to be sorted by z_order.
   std::vector<OverlayCheck_Params> TestPageFlip(
       const std::vector<OverlayCheck_Params>& params,
-      const OverlayPlaneList& last_used_planes,
-      ScanoutBufferGenerator* buffer_generator);
+      const OverlayPlaneList& last_used_planes);
 
-  // Tries to predict preferred format supported by hardware planes for |plane|
-  // for the given combination of |plane_list|. Using this format can help
-  // reduce read memory bandwidth during scanout for this plane.
-  uint32_t GetOptimalBufferFormat(const OverlayPlane& plane,
-                                  const OverlayPlaneList& plane_list) const;
+  // Checks if buffers bound to planes can be optimized (i.e. format, scaling)to
+  // reduce Display Read bandwidth. This should be called before an actual page
+  // flip. Expects |planes| to be sorted by Z order.
+  OverlayPlaneList PrepareBuffersForPageFlip(const OverlayPlaneList& planes);
 
   // Clears internal cache of validated overlay configurations. This should be
   // usually called when |window_| size has changed or moved controller.
   void ClearCache();
 
  private:
   // Contains hints which can be used to reduce display read memory bandwidth,
   // for a given OverlayPlane. These are useful in case of video to determine
   // if converting the buffer storage format before composition could lead to
   // any potential bandwidth savings. Other useful hint is to determine if
@@ skipping 11 matching lines @@
     bool handle_scaling;
   };
 
   using OverlayHintsList = std::vector<OverlayHints>;
 
   // Check if we can optimize format for reducing Display controller read
   // bandwidth for |plane_list| and cache the value.
   void UpdateOverlayHintsCache(
       const scoped_refptr<DrmDevice>& drm,
       const OverlayPlaneList& plane_list,
-      ScanoutBufferGenerator* buffer_generator,
       std::vector<scoped_refptr<ScanoutBuffer>>* reusable_buffers);
 
   DrmWindow* window_;  // Not owned.
+  ScanoutBufferGenerator* buffer_generator_;  // Not owned.
 
   // List of all configurations which have been validated.
   base::MRUCacheBase<OverlayPlaneList,
                      OverlayHintsList,
                      base::MRUCacheNullDeletor<OverlayHintsList>>
       overlay_hints_cache_;
 
   DISALLOW_COPY_AND_ASSIGN(DrmOverlayValidator);
 };
 
 }  // namespace ui
 
 #endif  // UI_OZONE_PLATFORM_DRM_GPU_DRM_OVERLAY_VALIDATOR_H_