Relocate ozone gbm overlay config cache to manager

ui/ozone/platform/drm/host/drm_overlay_candidates_host.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_HOST_OVERLAY_CANDIDATES_H_
 #define UI_OZONE_PLATFORM_DRM_HOST_OVERLAY_CANDIDATES_H_
 
 #include <stdint.h>
 
 #include <vector>
 
 #include "base/macros.h"
-#include "ui/ozone/common/gpu/ozone_gpu_message_params.h"
-#include "ui/ozone/platform/drm/host/drm_overlay_candidates_host_core.h"
-#include "ui/ozone/public/gpu_platform_support_host.h"
+#include "ui/ozone/platform/drm/host/drm_overlay_candidates_host.h"
 #include "ui/ozone/public/overlay_candidates_ozone.h"
 
 namespace ui {
 
-class DrmGpuPlatformSupportHost;
-class DrmWindowHost;
-class DrmOverlayCandidatesHost;
+class DrmOverlayManagerCore;
 
 // This is an implementation of OverlayCandidatesOzone where the driver is asked
 // about overlay capabilities via IPC. We have no way of querying abstract
 // capabilities, only if a particular configuration is supported or not.
 // Each time we we are asked if a particular configuration is supported, if we
 // have not seen that configuration before, it is IPCed to the GPU via
 // OzoneGpuMsg_CheckOverlayCapabilities; a test commit is then performed and
 // the result is returned in OzoneHostMsg_OverlayCapabilitiesReceived. Testing
 // is asynchronous, until the reply arrives that configuration will be failed.
 //
-// There is a many:1 relationship between this class and
-// DrmGpuPlatformSupportHost, each compositor will own one of these objects.
-// Each request has a unique request ID, which is assigned from a shared
-// sequence number so that replies can be routed to the correct object.
-// TODO(rjkroege): Consider removing the dependency on
-// GpuPlatformSupportHost.
-class DrmOverlayCandidatesHost : public OverlayCandidatesOzone,
-                                 public GpuPlatformSupportHost {
+// All OverlayCandidatesOzone objects share a single cache of tested
+// configurations stored in the overlay manager.
+class DrmOverlayCandidatesHost : public OverlayCandidatesOzone {
  public:
-  DrmOverlayCandidatesHost(DrmGpuPlatformSupportHost* platform_support,
-                           DrmWindowHost* window);
+  DrmOverlayCandidatesHost(DrmOverlayManagerCore* manager_core,
+                           gfx::AcceleratedWidget widget);
   ~DrmOverlayCandidatesHost() override;
 
-  // OverlayCandidatesOzone:
   void CheckOverlaySupport(OverlaySurfaceCandidateList* candidates) override;
 
-  // GpuPlatformSupportHost:
-  void OnChannelEstablished(
-      int host_id,
-      scoped_refptr<base::SingleThreadTaskRunner> send_runner,
-      const base::Callback<void(IPC::Message*)>& sender) override;
-  void OnChannelDestroyed(int host_id) override;
-  bool OnMessageReceived(const IPC::Message& message) override;
-
  private:
-  class OverlayCandidatesIPC : public DrmOverlayCandidatesHostProxy {
-   public:
-    OverlayCandidatesIPC(DrmGpuPlatformSupportHost* platform_support,
-                         DrmOverlayCandidatesHost* parent);
-    ~OverlayCandidatesIPC() override;
-    void RegisterHandler() override;
-    bool IsConnected() override;
-    void UnregisterHandler() override;
-    bool CheckOverlayCapabilities(
-        gfx::AcceleratedWidget widget,
-        const std::vector<OverlayCheck_Params>& new_params) override;
-
-   private:
-    DrmGpuPlatformSupportHost* platform_support_;
-    DrmOverlayCandidatesHost* parent_;
-    DISALLOW_COPY_AND_ASSIGN(OverlayCandidatesIPC);
-  };
-
-  // Entry point for incoming IPC.
-  void OnOverlayResult(bool* handled,
-                       gfx::AcceleratedWidget widget,
-                       const std::vector<OverlayCheck_Params>& params);
-
-  // Sends messages to the GPU thread.
-  scoped_ptr<OverlayCandidatesIPC> sender_;
-
-  // Implementation without messaging functionality.
-  scoped_ptr<DrmOverlayCandidatesHostCore> core_;
+  DrmOverlayManagerCore* manager_core_;  // Not owned.

[kalyank, 2016/01/25 23:43:43]

overlay_manager_ perhaps ?

[rjkroege, 2016/01/26 01:38:40]

Done.

+  gfx::AcceleratedWidget widget_;
 
   DISALLOW_COPY_AND_ASSIGN(DrmOverlayCandidatesHost);
 };
 
 }  // namespace ui
 
 #endif  // UI_OZONE_PLATFORM_DRM_HOST_OVERLAY_CANDIDATES_H_