Ozone: HDC should only keep track of planes owned by it’s crtc controllers

ui/ozone/platform/drm/gpu/crtc_controller.cc

 // 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.
 
 #include "ui/ozone/platform/drm/gpu/crtc_controller.h"
 
 #include "base/logging.h"
 #include "base/time/time.h"
 #include "ui/ozone/platform/drm/gpu/drm_device.h"
 #include "ui/ozone/platform/drm/gpu/page_flip_request.h"
 #include "ui/ozone/platform/drm/gpu/scanout_buffer.h"
 
 namespace ui {
 
 CrtcController::CrtcController(const scoped_refptr<DrmDevice>& drm,
                                uint32_t crtc,
                                uint32_t connector)
     : drm_(drm),
       overlay_plane_manager_(nullptr),
       crtc_(crtc),
       connector_(connector) {}
 
 CrtcController::~CrtcController() {
   if (!is_disabled_) {
+    const ScopedVector<HardwareDisplayPlane>& all_planes =
+        drm_->plane_manager()->planes();
+    for (auto* plane : all_planes) {
+      if (plane->owning_crtc() == crtc_) {
+        plane->set_owning_crtc(0);
+        plane->set_in_use(false);
+      }
+    }
+
     SetCursor(nullptr);
     drm_->DisableCrtc(crtc_);
     SignalPageFlipRequest();
   }
 }
 
 bool CrtcController::Modeset(const OverlayPlane& plane, drmModeModeInfo mode) {
   if (!drm_->SetCrtc(crtc_, plane.buffer->GetFramebufferId(),
                      std::vector<uint32_t>(1, connector_), &mode)) {
     PLOG(ERROR) << "Failed to modeset: crtc=" << crtc_
@@ skipping 125 matching lines @@
     // may result in a call to SchedulePageFlip(), which will override
     // page_flip_request_ and possibly release the ref. Stash previous request
     // locally to  avoid deleting the object we are making a call on.
     scoped_refptr<PageFlipRequest> last_request;
     last_request.swap(page_flip_request_);
     last_request->Signal(gfx::SwapResult::SWAP_ACK);
   }
 }
 
 }  // namespace ui